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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: goobix, Assigned: john)

References

Details

Attachments

(1 file, 1 obsolete file)

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".
Depends on: 255428
Flags: blocking2.20?
Target Milestone: --- → Bugzilla 2.20
Ah yes, the usage of userselect.html.tmpl in bug/edit.html.tmpl shouldn't use
emptyok, should it...
Attached patch Bugzilla patch for tip (obsolete) — Splinter Review
A patch to remove the blank line in the multiselect for "Add CC" on the edit
bug page.
Oh, and strictly the blank line was there before my patch in bug 255428, so this
wasn't a regression.
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.
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?
Attachment #171511 - Flags: review? → review+
Assignee: general → john
Flags: approval?
Whiteboard: patch awaiting approval
Flags: blocking2.20?
Flags: approval?
Flags: approval+
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
yes, thank you.
*** 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.

Attachment

General

Creator:
Created:
Updated:
Size: