Closed
Bug 455559
Opened 16 years ago
Closed 15 years ago
The requestee field should be narrower
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: LpSolit, Assigned: LpSolit)
References
Details
Attachments
(1 file, 1 obsolete file)
1.85 KB,
patch
|
guy.pyrzak
:
review+
|
Details | Diff | Splinter Review |
As part of bug 420288, we should limit the width of the requestee field (and why not also limit the length of the requester) to avoid (too much) horizontal scroll.
Assignee | ||
Comment 1•15 years ago
|
||
I just checked on landfill, and based on 28400 email addresses, the median is around 20 characters, and with most addresses shorter than ~25 characters. With size="20", and considering that many addresses have letters such as "i" or "l" in them, we are able to see up to 20-25 characters at once without problem. I prefer to have some very long addresses being truncated a bit in the requestee field than to waste whitespace in 90% of cases.
Assignee | ||
Updated•15 years ago
|
Target Milestone: --- → Bugzilla 3.6
Assignee | ||
Comment 2•15 years ago
|
||
Another welcome side-effect is that it makes the "edit attachment as comment" frame much wider.
Assignee | ||
Updated•15 years ago
|
Summary: The requestee field should be narrower in show_bug.cgi → The requestee field should be narrower
Updated•15 years ago
|
Attachment #403359 -
Flags: review?(guy.pyrzak) → review-
Comment 3•15 years ago
|
||
Comment on attachment 403359 [details] [diff] [review] patch, v1 I'd prefer to use CSS to make this change, mostly b/c we want to move towards using CSS for formatting stuff like this. We should test it in older browsers but i think using CSS will allow greater flexibility to stylers.
Assignee | ||
Comment 4•15 years ago
|
||
OK, now using CSS. Tested on IE 6, IE8, Firefox, Safari, Opera, Google Chrome and Konqueror.
Attachment #403359 -
Attachment is obsolete: true
Attachment #403506 -
Flags: review?(guy.pyrzak)
Updated•15 years ago
|
Attachment #403506 -
Flags: review?(guy.pyrzak) → review+
Updated•15 years ago
|
Flags: approval?
Assignee | ||
Updated•15 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 5•15 years ago
|
||
Checking in skins/standard/global.css; /cvsroot/mozilla/webtools/bugzilla/skins/standard/global.css,v <-- global.css new revision: 1.67; previous revision: 1.66 done Checking in template/en/default/flag/list.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/flag/list.html.tmpl,v <-- list.html.tmpl new revision: 1.37; previous revision: 1.36 done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•