The default bug view has changed. See this FAQ.

Remove list item bullet size quirk

RESOLVED FIXED in Firefox 50

Status

()

Core
Layout
RESOLVED FIXED
6 years ago
9 months ago

People

(Reporter: Ms2ger, Assigned: xidorn)

Tracking

(Blocks: 1 bug, {dev-doc-complete})

Trunk
mozilla50
dev-doc-complete
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

(URL)

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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>)
(Reporter)

Updated

5 years ago
Blocks: 747485

Comment 1

5 years ago
> 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)
(Reporter)

Updated

4 years ago
Assignee: nobody → Ms2ger
Flags: needinfo?(Ms2ger)

Updated

4 years ago
Keywords: dev-doc-needed
(Reporter)

Comment 3

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=974af06fbc27
(Reporter)

Comment 4

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=00ec4530de21
(Reporter)

Comment 5

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e37adb57f4c
(Reporter)

Comment 6

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1848ec478987
(Assignee)

Comment 7

10 months ago
IE no longer has this quirk either, which makes us the only implementation doing this.
(Assignee)

Comment 8

10 months ago
Created attachment 8759121 [details]
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/
Attachment #8759121 - Flags: review?(cam)
(Assignee)

Comment 9

10 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b40a438dd31
Assignee: Ms2ger → bugzilla
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+
(Assignee)

Updated

9 months ago
Keywords: checkin-needed
(Assignee)

Comment 11

9 months ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/948850b9dff58a9af4f8be529cd2638feb2571bb
Keywords: checkin-needed

Comment 12

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/948850b9dff5
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Updated:
https://developer.mozilla.org/en-US/docs/Mozilla/Mozilla_quirks_mode_behavior
and
https://developer.mozilla.org/en-US/Firefox/Releases/50#CSS
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.