Unordered lists are displayed as ordered lists

RESOLVED FIXED in Firefox 38

Status

()

Toolkit
Reader Mode
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: agibson, Assigned: Gijs)

Tracking

(Blocks: 1 bug)

unspecified
mozilla40
Points:
1
Dependency tree / graph
Bug Flags:
firefox-backlog +
in-testsuite -
qe-verify -

Firefox Tracking Flags

(firefox38 fixed, firefox39 fixed, firefox40 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
STR:

1.) In Nightly, visit: http://alistapart.com/article/taminglists/#figure1
2.) Click the Reader View icon in the address bar.
3.) In Reader View scroll down to #figure1

Expected results:

The unordered list items should be preceded by a disc.

Actual Results:

The unordered list items and ordered numerically.

As pointed out in the original GitHub issue I opened [1], it looks lime there is a typo in the Reader View CSS style sheet:

.content ul {
    list-style: outside none disk;
}

should in fact be:

.content ul {
    list-style: outside none disc;
}

[1] https://github.com/mozilla/readability/issues/93
(Assignee)

Comment 1

3 years ago
I was cheeky and just landed this because, srsly. :-)

remote:   https://hg.mozilla.org/integration/fx-team/rev/19c7476c47de
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 40.1 - 13 Apr
Points: --- → 1
Flags: qe-verify-
Flags: in-testsuite-
Flags: firefox-backlog+
Hardware: x86 → All
(Assignee)

Updated

3 years ago
Blocks: 558882
status-firefox38: --- → affected
status-firefox39: --- → affected

Updated

3 years ago
Blocks: 1132074
https://hg.mozilla.org/mozilla-central/rev/19c7476c47de
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40

Comment 3

3 years ago
Created attachment 8588100 [details] [diff] [review]
fix silly typo in list styles

Approval Request Comment
[Feature/regressing bug #]: reader view
[User impact if declined]: wrong bullet styles in reader view
[Describe test coverage new/current, TreeHerder]: landed on nightly
[Risks and why]: low-risk simple CSS change
[String/UUID change made/needed]: none
Attachment #8588100 - Flags: approval-mozilla-beta?
Attachment #8588100 - Flags: approval-mozilla-aurora?
Comment on attachment 8588100 [details] [diff] [review]
fix silly typo in list styles

should be in 38 beta 2
Attachment #8588100 - Flags: approval-mozilla-beta?
Attachment #8588100 - Flags: approval-mozilla-beta+
Attachment #8588100 - Flags: approval-mozilla-aurora?
Attachment #8588100 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/bd0bf4503443
status-firefox39: affected → fixed
https://hg.mozilla.org/releases/mozilla-beta/rev/3d380257da88
status-firefox38: affected → fixed
You need to log in before you can comment on or make changes to this bug.