Open Bug 747464 Opened 12 years ago Updated 2 years ago

Drop font size quirk

Categories

(Core :: CSS Parsing and Computation, defect)

defect

Tracking

()

People

(Reporter: zcorpan, Unassigned, Mentored)

References

(Blocks 1 open bug)

Details

(Whiteboard: [lang=c++])

http://mxr.mozilla.org/mozilla-central/source/layout/style/nsStyleUtil.cpp#64

IE and Opera don't have this quirk, which means it should be possible to align quirks mode and standards mode font sizes.

http://software.hixie.ch/utilities/js/live-dom-viewer/saved/1481 in Opera, Preferences->Webpages (change both normal font size and monospace font size; restart.

http://software.hixie.ch/utilities/js/live-dom-viewer/saved/1484 in IE9 I didn't find anywhere to change the default font size, but it appears to make no difference between compat quirks vs compat standards or ie9 quirks vs ie9 standards.

WebKit has this quirk, but I'll file a bug on them to drop it too.
(In reply to Simon Pieters from comment #0)
> http://mxr.mozilla.org/mozilla-central/source/layout/style/nsStyleUtil.cpp#64

This presumably pointed to:
http://hg.mozilla.org/mozilla-central/file/add05f884131/layout/style/nsStyleUtil.cpp#l64

A current link would be:
http://hg.mozilla.org/mozilla-central/file/d136c8999d96/layout/style/nsRuleNode.cpp#l2587

Is there actually interop on these sizes?  I think if we're going to change them we should change them in the direction of being more interoperable rather than just changing the quirks sizes to the strict sizes.  (What if the quirks sizes are closer to the interoperable numbers?)

There are also a number of open bugs suggestion changes to these sizes for other reasons.  It's worth considering those if interop allows it.  (bug 187256, maybe others)

I think the bulk of the work involved in fixing this bug is investigation of browser interoperability, what we should do to improve both interoperability and behavior.
Whiteboard: [mentor=dbaron]
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [mentor=dbaron] → [mentor=dbaron][lang=c++]
Mentor: dbaron
Whiteboard: [mentor=dbaron][lang=c++] → [lang=c++]
I am interested to work on this bug.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.