Closed Bug 455559 Opened 16 years ago Closed 15 years ago

The requestee field should be narrower

Categories

(Bugzilla :: User Interface, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
Attached patch patch, v1 (obsolete) — Splinter Review
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: ui → LpSolit
Status: NEW → ASSIGNED
Attachment #403359 - Flags: review?(guy.pyrzak)
Target Milestone: --- → Bugzilla 3.6
Another welcome side-effect is that it makes the "edit attachment as comment" frame much wider.
Summary: The requestee field should be narrower in show_bug.cgi → The requestee field should be narrower
Attachment #403359 - Flags: review?(guy.pyrzak) → review-
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.
Attached patch patch, v2Splinter Review
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)
Attachment #403506 - Flags: review?(guy.pyrzak) → review+
Flags: approval?
Flags: approval? → approval+
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.

Attachment

General

Created:
Updated:
Size: