648331 - Remove list item bullet size quirk

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

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>)

Comment 1

[j.j.]

> WebKit doesn't have this quirk
Opera removed that quirk (tested with version 12)

Comment 2

[David Baron :dbaron:]

(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

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=974af06fbc27

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=00ec4530de21

Comment 5

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e37adb57f4c

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1848ec478987

Comment 7

[Xidorn Quan [:xidorn] UTC+11]

IE no longer has this quirk either, which makes us the only implementation doing this.

Comment 8

[Xidorn Quan [:xidorn] UTC+11]

Attachment: MozReview Request: Bug 648331 - Remove list item bullet size quirk. r?heycam

Review commit: https://reviewboard.mozilla.org/r/57184/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/57184/

Comment 9

[Xidorn Quan [:xidorn] UTC+11]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b40a438dd31

Comment 10

[Cameron McCormack (:heycam)]

Comment on attachment 8759121 [details]
MozReview Request: Bug 648331 - Remove list item bullet size quirk. r?heycam

https://reviewboard.mozilla.org/r/57184/#review56578

Comment 11

[Xidorn Quan [:xidorn] UTC+11]

https://hg.mozilla.org/integration/mozilla-inbound/rev/948850b9dff58a9af4f8be529cd2638feb2571bb

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/948850b9dff5

Comment 13

[Jean-Yves Perrier [:teoli]]

Updated:
https://developer.mozilla.org/en-US/docs/Mozilla/Mozilla_quirks_mode_behavior
and
https://developer.mozilla.org/en-US/Firefox/Releases/50#CSS