1881517 - The counter does not reset in <ol> nested list if you use the CSS property `all: unset`. It only affects Firefox, the other browsers have `counter-reset: list-item` for their default <ol> stylesheet.

Status: RESOLVED INVALID

(Core :: Layout: Generated Content, Lists, and Counters, defect)

Description

[Yahuhanan Yukiya Sese Cuneta [:Yuki] (:YourOnly.One)]

Attachment: testpage.html

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:124.0) Gecko/20100101 Firefox/124.0

Steps to reproduce:

1. Create a nest <ol> list.
2. Add this CSS reset:

*:where(:not(html, iframe, canvas, img, svg, video, audio):not(svg *, symbol *)) {
    all: unset;
    display: revert;
}

Actual results:

1. Level 1 Item 1
2. Level 1 Item 2
   3. Item 2's Level 2 Item 1
   4. Item 2's Level 2 Item 2

and so on.

Expected results:

1. Level 1 Item 1
2. Level 1 Item 2
   1. Item 2's Level 2 Item 1
   2. Item 2's Level 2 Item 2

and so on.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Graphics: Canvas2D' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Alice0775 White]

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=1032b24d9925074b94ec1a0dc2c91166766a9277&tochange=b0a13898111c6548427a451993ce69e0d8eeb749

Suspect:
fd29ba131d13728be2b10453636e8f76d968ec33 Emilio Cobos Álvarez — Bug 933562 - Implement complex :not(). r=jfkthame

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 933562

:emilio, since you are the author of the regressor, bug 933562, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 4

[Emilio Cobos Álvarez [:emilio]]

Yeah, this is not really a regression from that change, and this is really invalid. Firefox implements CSS lists using CSS counters as per spec, so you're really resetting https://searchfox.org/mozilla-central/rev/a8cc31504a2379bcf8ba395d2da7bb632b5521d6/layout/style/res/html.css#570.

The issue is that other browsers don't implement html lists using CSS.

Comment 5

[Alice0775 White]

Attachment: testpage (removed complicated :not())

Regression window:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=5dc0652cd024fef30217ac3e6087b7a32f16967f&tochange=9bad25ccc8d0414e21d3c8d4dcf2178a97bb07c3

Comment 6

[Yahuhanan Yukiya Sese Cuneta [:Yuki] (:YourOnly.One)]

Ahh! I see what's happening. Thank you for the clarification, much appreciated!