Closed Bug 648331 Opened 9 years ago Closed 4 years ago

Remove list item bullet size quirk

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: Ms2ger, Assigned: xidorn)

References

(Blocks 1 open bug, )

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

As mentioned in $URL, WebKit doesn't have this quirk, which suggests that it isn't necessary. Note that the test case in bug 97351 comment 0 is fixed by the new HTML parser, which produces this DOM tree:

ul
  font color="red" size="30pt"
  li
    font color="red" size="30pt"
      This text should be ...

If nobody objects, I can fix. (<http://mxr.mozilla.org/mozilla-central/source/layout/style/quirk.css#74>)
> WebKit doesn't have this quirk
Opera removed that quirk (tested with version 12)
(In reply to :Ms2ger from comment #0)
> If nobody objects, I can fix.

Sounds good to me.  Still up to fixing?

> (<http://mxr.mozilla.org/mozilla-central/source/layout/style/quirk.css#74>)

A persistent version of that link is:
http://hg.mozilla.org/mozilla-central/file/152fe0de3c82/layout/style/quirk.css#l74
Flags: needinfo?(Ms2ger)
Assignee: nobody → Ms2ger
Flags: needinfo?(Ms2ger)
Keywords: dev-doc-needed
IE no longer has this quirk either, which makes us the only implementation doing this.
Comment on attachment 8759121 [details]
MozReview Request: Bug 648331 - Remove list item bullet size quirk. r?heycam

https://reviewboard.mozilla.org/r/57184/#review56578
Attachment #8759121 - Flags: review?(cam) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/948850b9dff5
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.