Closed
Bug 278711
Opened 20 years ago
Closed 20 years ago
An empty entry is sometimes not useful and should be removed
Categories
(Bugzilla :: Bugzilla-General, defect)
Bugzilla
Bugzilla-General
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: goobix, Assigned: john)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.33 KB,
patch
|
goobix
:
review+
|
Details | Diff | Splinter Review |
Bug 255428 caused to show a multiple selection list where the first entry is an empty line. This is not practical and the first line (the empty one) should be removed in some cases, like in the case of "Add CC".
| Reporter | ||
Updated•20 years ago
|
Flags: blocking2.20?
Target Milestone: --- → Bugzilla 2.20
| Assignee | ||
Comment 1•20 years ago
|
||
Ah yes, the usage of userselect.html.tmpl in bug/edit.html.tmpl shouldn't use emptyok, should it...
| Assignee | ||
Comment 2•20 years ago
|
||
A patch to remove the blank line in the multiselect for "Add CC" on the edit bug page.
| Assignee | ||
Comment 3•20 years ago
|
||
Oh, and strictly the blank line was there before my patch in bug 255428, so this wasn't a regression.
| Assignee | ||
Comment 4•20 years ago
|
||
Actually, no, I understand now. When the addcc was a single dropdown, it needed emptyok, now that's it's a multiple select it doesn't. I get it now.
| Assignee | ||
Comment 5•20 years ago
|
||
Second attempt at a patch to remove unnecessary blank lines in multiselects for CCs. There shouldn't be a blank line in the CC field in the create bug page either.
Attachment #171509 -
Attachment is obsolete: true
Attachment #171511 -
Flags: review?
| Reporter | ||
Updated•20 years ago
|
Attachment #171511 -
Flags: review? → review+
| Reporter | ||
Updated•20 years ago
|
Assignee: general → john
Flags: approval?
Updated•20 years ago
|
Whiteboard: patch awaiting approval
Updated•20 years ago
|
Flags: blocking2.20?
Flags: approval?
Flags: approval+
Comment 6•20 years ago
|
||
Checking in template/en/default/bug/edit.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl,v <-- edit.html.tmpl new revision: 1.51; previous revision: 1.50 done Checking in template/en/default/bug/create/create.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tm pl,v <-- create.html.tmpl new revision: 1.40; previous revision: 1.39 done
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: patch awaiting approval
Comment 8•20 years ago
|
||
*** Bug 286550 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•