Closed
Bug 652405
Opened 15 years ago
Closed 15 years ago
All user fields (assignee, QA contact, Add CC) have the page title as the "title" attribute
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: deleeuw+bugzilla, Assigned: LpSolit)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
2.85 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:6.0a1) Gecko/20110423 Firefox/6.0a1
Build Identifier:
All details in summary.
Reproducible: Always
Steps to Reproduce:
1. Open any bug report
2. Click the edit link for the CC List
3. Hover on the "Add" text box
Actual Results:
– in title attribute
Expected Results:
- in title attribute
|
Reporter | |
Updated•15 years ago
|
Blocks: bmo-regressions
|
|
Reporter | |
Updated•15 years ago
|
OS: Windows XP → All
Hardware: x86 → All
Version: unspecified → 4.0
|
Assignee | |
Comment 1•15 years ago
|
||
You are correct. 3.6 doesn't have this problem. So something regressed this.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
Target Milestone: --- → Bugzilla 4.0
|
|
Assignee | |
Comment 2•15 years ago
|
||
The problem is that one parameter used in global/userselect.html.tmpl is "title". But "title" is already used to define the title of the HTML page, and so this parameter is implicitly passed to global/userselect.html.tmpl every time this template is called. This is a regression due to bug 578254.
Depends on: 578254
Summary: "Add" Textbox in CC List (click edit link to activate) has – in the title attribute → All user fields (assignee, QA contact, Add CC) have the page title as the "title" attribute
|
|
Assignee | |
Comment 3•15 years ago
|
||
I also fixed the wrong usage of ". You cannot use it here as the title is HTML filtered, and so is displayed as-is instead of being converted into a double quote.
|
||
Comment 4•15 years ago
|
||
Comment on attachment 527996 [details] [diff] [review]
patch, v1
Review of attachment 527996 [details] [diff] [review]:
Looks good to me. r=dkl
Attachment #527996 -
Flags: review?(dkl) → review+
|
|
||
Updated•15 years ago
|
Flags: approval?
Flags: approval4.0?
|
|
Assignee | |
Updated•15 years ago
|
Flags: approval?
Flags: approval4.0?
Flags: approval4.0+
Flags: approval+
|
|
Assignee | |
Comment 5•15 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/global/userselect.html.tmpl
modified template/en/default/request/queue.html.tmpl
Committed revision 7787.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/
modified template/en/default/global/userselect.html.tmpl
modified template/en/default/request/queue.html.tmpl
Committed revision 7576.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Comment 6•15 years ago
|
||
deployed to production on bmo
You need to log in
before you can comment on or make changes to this bug.
Description
•