Closed
Bug 192385
Opened 22 years ago
Closed 21 years ago
Bug ID wordwraps in change-several-bugs page if window is narrow
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: justdave, Assigned: justdave)
References
Details
Attachments
(1 file, 1 obsolete file)
1.08 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
The checkbox and the Bug ID appear to be in the same table cell in the change-several screen, which causes the Bug ID to wrap onto a second line if your window is narrow, which basically makes the screen look funny. The checkbox probably ought to be in a separate table cell by itself so the bug ID doesn't wrap.
Comment 1•22 years ago
|
||
Or you could use the <nobr> stuff (or whitespace: <something>; in css)
Assignee | ||
Comment 2•21 years ago
|
||
OK, the nowrap stuff doesn't work all the time if the window is really narrow... for some reason the end boundary of an <input> is considered whitespace even if there isn't any whitespace there. Dunno if it's a bug in Moz, but it's enough reason to make it a cell instead. Had to fix this for Zippy, patch coming up.
Assignee: endico → justdave
Assignee | ||
Comment 3•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #128221 -
Flags: review?(myk)
Comment 4•21 years ago
|
||
Comment on attachment 128221 [details] [diff] [review] Patch >+ [% IF dotweak %] >+ <th colspan="[% splitheader ? 3 : 2 %]" align="center"> Looks good, except for the align="center", which doesn't actually align the ID header to the data in its column. Have you tried a blank TH tag on top of the checkbox column instead of adding extra colspan to the ID header?
Assignee | ||
Comment 5•21 years ago
|
||
That's a good idea. done.
Attachment #128221 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #128221 -
Flags: review?(myk)
Assignee | ||
Updated•21 years ago
|
Attachment #129275 -
Flags: review?(myk)
Comment 6•21 years ago
|
||
Comment on attachment 129275 [details] [diff] [review] Patch v2 Looks good, works. r=myk
Attachment #129275 -
Flags: review?(myk) → review+
Assignee | ||
Comment 7•21 years ago
|
||
Checking in template/en/default/list/table.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/list/table.html.tmpl,v <-- table.html.tmpl new revision: 1.14; previous revision: 1.13 done
Status: NEW → RESOLVED
Closed: 21 years ago
Flags: approval+
Resolution: --- → FIXED
Assignee | ||
Updated•21 years ago
|
Target Milestone: --- → Bugzilla 2.18
Assignee | ||
Comment 8•21 years ago
|
||
*** Bug 228346 has been marked as a duplicate of this bug. ***
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•