- From: Guillaume via GitHub <sysbot+gh@w3.org>
- Date: Wed, 01 Feb 2023 08:56:02 +0000
- To: public-css-archive@w3.org
cdoublev has just created a new issue for https://github.com/w3c/csswg-drafts: == [css-values-5] Minor problems on `toggle()` and `random-*()` productions == **[`toggle()`](https://drafts.csswg.org/css-values-5/#funcdef-toggle)** `toggle()` should be declared with a proper production rule in order to be extracted by `w3c/reffy`: `toggle() = toggle(...)` or `<toggle()> = toggle(...)`. I was editing a PR to fix this but created this issue instead because I do not know if you want to declare it as a `function` or `type`. I do not know what difference it makes. `<mix()>` production was recently edited in CSS Values 4 as a `type`, but `attr()` production is currently declared as a `function` in CSS Values 5, whereas `<random-*()>` productions are declared as `type`s. I was also replacing `<toggle-value>` by `<declaration-value>` to fix #8091. Note the current text is *"where `<toggle-value>` is any CSS value"*, but since it should be semicolon-separated values, it cannot be `<any-value>`: > The `<any-value>` production is identical to `<declaration-value>`, but also allows top-level `<semicolon-token>` tokens --- **[`random-item()`](https://drafts.csswg.org/css-values-5/#funcdef-random-item)** Similarly, I think `<any-value>` is inappropriate in `<random-item()>`, and should be replaced by `<declaration-value>`: > `<random-item()> = random-item( <random-caching-options> ';' <any-value> [ ';' <any-value> ]* )` --- **[`random()`](https://drafts.csswg.org/css-values-5/#funcdef-random)** Finally, the trailing `;` in the production for `<random()>` seems to be a typo: > `<random()> = random( <random-caching-options>? , <calc-sum>, <calc-sum>, [by <calc-sum>]? );` Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/8384 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Wednesday, 1 February 2023 08:56:04 UTC