Re: [csswg-drafts] [css-gcpm-4] Fix <content-level> definition (#9323)

Do you think it can be updated to `<attr()>` rather than preserving `attr(<identifier>)`?

[`<identifier>`](https://drafts.csswg.org/css2/#value-def-identifier) is only used in CSS 2 for [`content`](https://drafts.csswg.org/css2/#propdef-content), [`counter-reset`](https://drafts.csswg.org/css2/#propdef-counter-reset), [`counter-increment`](https://drafts.csswg.org/css2/#propdef-counter-increment).

It seems inappropriate for `counter-increment` and `counter-reset` because [CSS Lists 3](https://drafts.csswg.org/css-lists-3/#propdef-counter-increment) and browsers do not accept CSS-wide keywords or `default` (even when followed by an explicit integer), as well as in [`counter()`](https://drafts.csswg.org/css-lists-3/#typedef-counter).

CSS Content 3 redefines [`content`](https://drafts.csswg.org/css-content-3/#propdef-content) without `attr(<identifier>)` and implicitly accepts the global `<attr()>`, and `string-set` accepts `<attr()>` in CSS GCPM 3.

---

Similarly, do you think it would be fine to replace `<content-list>` by `<content-list> | <attr()>` in the value definition of [`string-set`](https://drafts.csswg.org/css-gcpm-3/#propdef-string-set) in CSS GCPM 3, and to remove its production rule of `<content-list>`, so that it references [the production rule in CSS Lists 3](https://drafts.csswg.org/css-content-3/#typedef-content-content-list)? The goal is to remove duplicate `<content-list>` production rules (cf. #9318).

Note that it would mean that `string-set` accepts eg. `<image>` as `<content-list>`, whereas it is not accepted in CSS GCPM 3.

-- 
GitHub Notification of comment by cdoublev
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/pull/9323#issuecomment-1820478060 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 21 November 2023 08:48:29 UTC