Closed
Bug 514883
Opened 15 years ago
Closed 15 years ago
Groupbox custom caption content has bold font
Categories
(Toolkit :: Themes, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
status1.9.2 | --- | beta1-fixed |
status1.9.1 | --- | .4-fixed |
People
(Reporter: neil, Assigned: neil)
References
Details
(Keywords: fixed-seamonkey2.0, regression)
Attachments
(1 file)
(deleted),
patch
|
dao
:
review+
beltzner
:
approval1.9.2+
dveditz
:
approval1.9.1.4+
|
Details | Diff | Splinter Review |
It's not uncommon for groupbox captions to contain non-default content. The most popular example is a menulist, which gets used a few times throughout the codebase, most noticeably in font preferences (all three applications). Unfortunately bold menulists look particularly bad as their menuitems are not bold and they therefore fail to size to fit their widest item correctly.
I'd therefore appreciate suggestions as to how to resolve this for gnomestripe. Feel free to suggest other ideas, but three to get you thinking are a) just make menulists use the same font as menuitems b) only apply bold to the default caption label class c) apply bold to any label in a caption.
Comment 1•15 years ago
|
||
(In reply to comment #0)
> Unfortunately bold menulists look particularly bad as their menuitems are not
> bold and they therefore fail to size to fit their widest item correctly.
This sounds like menulists shouldn't inherit font-weight.
Assignee | ||
Comment 2•15 years ago
|
||
I've had a look at our only in-tree user of a custom <caption> that doesn't use a menulist; this is Venkman's Breakpoint Properties dialog. Its bold checkbox caption does not seem too much out of place.
Assignee | ||
Comment 3•15 years ago
|
||
Updated•15 years ago
|
Attachment #398937 -
Flags: review?(dao) → review+
Assignee | ||
Comment 4•15 years ago
|
||
Pushed changeset 0d056cf12c63 to mozilla-central.
I guess I'll call this fixed then.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment on attachment 398937 [details] [diff] [review]
Possible patch
Would like to get this fix into SM2.0 and SM2.1 - low risk fix.
Attachment #398937 -
Flags: approval1.9.2?
Attachment #398937 -
Flags: approval1.9.1.4?
Comment 6•15 years ago
|
||
Comment on attachment 398937 [details] [diff] [review]
Possible patch
Approved for 1.9.1.4, a=dveditz for release-drivers
Attachment #398937 -
Flags: approval1.9.1.4? → approval1.9.1.4+
status1.9.1:
--- → .4-fixed
Keywords: fixed-seamonkey2.0
Updated•15 years ago
|
Attachment #398937 -
Flags: approval1.9.2? → approval1.9.2+
Comment 8•15 years ago
|
||
Comment on attachment 398937 [details] [diff] [review]
Possible patch
a192=beltzner
Assignee | ||
Comment 9•15 years ago
|
||
Pushed changeset 8a96df13cf59 to releases/mozilla-1.9.2
status1.9.2:
--- → beta1-fixed
Comment 10•15 years ago
|
||
Is there a way to manually verify this fix?
Assignee | ||
Comment 11•15 years ago
|
||
Sure, just compare font preferences before and after.
You need to log in
before you can comment on or make changes to this bug.
Description
•