Closed Bug 549290 Opened 14 years ago Closed 14 years ago

Title of common dialogs no longer bold on Mac

Categories

(Toolkit :: Themes, defect)

All
macOS
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.9.3a3

People

(Reporter: florian, Assigned: florian)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
Before the patch from bug 459457, there was this CSS rule in formatting.css:
 #commonDialog .dialogTitle {
   font-weight: bold !important;
 }

After the patch, it became this in commonDialog.css:
 .dialogTitle {
   font-weight: bold;
 }

Without !important, the rule has no effect because there's another rule just above it which causes the font attributes to use system values:
 #info\.title,
 #info\.header,
 #info\.body {
   font: menu;
   line-height: 16px;
   margin-bottom: 6px;
 }
Attachment #429502 - Flags: review?(dao)
Can this just use #info\.title instead? Does dialogTitle need to exist at all?
Attached patch Patch v2Splinter Review
(In reply to comment #1)
> Can this just use #info\.title instead?
Yes.

>Does dialogTitle need to exist at all?
No... or at least I couldn't find any use for it with mxr. I removed it in this second patch. Let's hope it won't break any theme.
Attachment #429502 - Attachment is obsolete: true
Attachment #429505 - Flags: review?(dao)
Attachment #429502 - Flags: review?(dao)
Attachment #429505 - Flags: review?(dao) → review+
http://hg.mozilla.org/mozilla-central/rev/20fce9e19782
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: