1457744 - Fix serialization order of the list-style shorthand.

Status: RESOLVED INVALID

(Core :: CSS Parsing and Computation, enhancement)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1457744: Fix serialization order of list-style.

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

Comment 2

[Xidorn Quan [:xidorn] UTC+11]

Comment on attachment 8971840 [details]
Bug 1457744: Fix serialization order of list-style.

https://reviewboard.mozilla.org/r/240584/#review246276

The spec is probably wrong here. `list-style-type` accepts arbitrary identifier, which can overlap with values of `list-style-position`, and thus it is checked at last. Serializing it first means we may get different result in some case.

Please consider declaration like `list-style: inside outside;`. It currently means `list-style-position: inside; list-style-type: outside;` because `list-style-type` is parsed later, but this change would make us serialize it as `list-style: outside inside;` which changes the original meaning.

We should change the canonical order of `list-style` to make `list-style-type` the last one, just like what we've done with `animation` shorthand.

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Hmm, indeed. Good point. Filed https://github.com/w3c/csswg-drafts/issues/2624.