Closed Bug 496060 Opened 15 years ago Closed 15 years ago

'window' is not a font property, but a -moz-appearance or color property

Categories

(Firefox :: Theme, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3.6a1

People

(Reporter: Natch, Unassigned)

References

Details

Attachments

(2 files)

Warning: Error in parsing value for 'font'.  Declaration dropped.
Line: 2113
Source File: chrome://browser/skin/browser.css

STR:

Start up Firefox!
This doesn't change anything in the ui. All it does is change an invalid css declaration to valid. The 'window' thing doesn't work and it isn't a font-face. -moz-window isn't a font-face either, it is an entire 'font' property with its own font-size. Try replacing the current declaration with 'font: -moz-window' and you'll see that the text size changes. the fallback should be builtin anyhow (which is what it was relying on till now anyways), there is now way to fall back to -moz-window or window.

Thanks to bz for pointing this out on irc (I was going to switch window with -moz-window, but that doesn't work).
Assignee: nobody → highmind63
Status: NEW → ASSIGNED
Attachment #381222 - Flags: review?(dao)
Or, something like this can be done to still use the fallback without losing the font-size.
Attachment #381224 - Flags: review?(dao)
Depends on: 465076
Attachment #381222 - Flags: review?(dao)
Attachment #381224 - Flags: review?(dao)
Hah, it's fixed in the "Yet another All Tabs/Ctrl Tab revision", feel free to dupe this I guess.
Assignee: highmind63 → nobody
Status: ASSIGNED → NEW
Let's keep this open to make sure it will get fixed.
Fixed by bug 465076.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
v. woot!

Thanks Dao :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: