Closed Bug 151648 Opened 21 years ago Closed 21 years ago

'QA Contact' column presented with useqacontact = Off

Categories

(Bugzilla :: User Accounts, defect)

2.17
x86
Linux
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: MatsPalmgren_bugz, Assigned: gerv)

References

()

Details

Attachments

(1 file)

OVERVIEW:
In our local installation of Bugzilla 2.17 (updated today from CVS) we are not
using the QA field. So we have useqacontact = Off.

ACTUAL BEHAVIOUR:
User email prefs still presents a 'QA Contact' column.

EXPECTED BEHAVIOUR:
No 'QA Contact' column.
-> gerv
Assignee: justdave → gerv
Component: Bugzilla-General → User Accounts
Attached patch Patch v.1Splinter Review
Indeed, you are right. Good catch. This should fix it.

Gerv
Comment on attachment 87651 [details] [diff] [review]
Patch v.1

>-    <td colspan="5" align="center" width="50%">
>+    <td colspan="[% useqacontact ? '5' : '4' %]" align="center" width="50%">
>       <b>When my relationship to this bug is:</b>
>     </td>
>     <td rowspan="2" width="50%">

The patch works, but you need to make td widths in this part adapt to the
useqacontact changes. Specifically, the first td should have a 40% width when
!useqacontact, and in that case the latter td should have 60%.
Jouni: does it really matter? I didn't see any layout problems when I tried it.
It just seems to me like it complicates the template to make the two versions
look more similar, when no-one will actually care.

Gerv
Can someone review this? myk - you may well want it on b.m.o.

Gerv
Status: NEW → ASSIGNED
Keywords: patch, review
Target Milestone: --- → Bugzilla 2.18
Comment on attachment 87651 [details] [diff] [review]
Patch v.1

Oh all right. Maybe the td widths are not that important. Per my previous
tests, 2xr=jouni.
Attachment #87651 - Flags: review+
Fixed.

Checking in template/en/default/account/prefs/email.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/account/prefs/email.html.tmpl,v
 <--  email.html.tmpl
new revision: 1.1.2.3; previous revision: 1.1.2.2
done

Checking in template/en/default/account/prefs/email.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/account/prefs/email.html.tmpl,v
 <--  email.html.tmpl
new revision: 1.5; previous revision: 1.4
done

Gerv
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.