Closed Bug 514883 Opened 15 years ago Closed 15 years ago

Groupbox custom caption content has bold font

Categories

(Toolkit :: Themes, defect)

x86
Linux
defect
Not set
normal

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)

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.
(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.
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.
Attached patch Possible patch (deleted) — Splinter Review
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #398937 - Flags: review?(dao)
Attachment #398937 - Flags: review?(dao) → review+
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 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+
Attachment #398937 - Flags: approval1.9.2? → approval1.9.2+
Pushed changeset 8a96df13cf59 to releases/mozilla-1.9.2
Is there a way to manually verify this fix?
Sure, just compare font preferences before and after.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: