[csswg-drafts] Pull Request: [various] Replace optional group with implicitly optional comma

cdoublev has just submitted a new pull request for https://github.com/w3c/csswg-drafts:

== [various] Replace optional group with implicitly optional comma ==
Editorial change replacing explicit optional groups with implicitly optional commas, ie. `... [ , ... ]?` with `... , ...?`.

I would appreciate this change because parsing against these two syntaxes with the CSS parser I am using, produces two different results (ie. `List(..., optionalList(comma, ...))` vs. `List(..., comma, ...)`), and the second is easier to handle.

I searched for all occurrences with `/\[\s*,.+\]\s*\?/`. I did not replace the result found in the value definition of [`copy-into`](https://drafts.csswg.org/css-gcpm-4/#propdef-copy-into) (CSS GCPM 4, not ready for implementation): the whole alternation can be omitted, which seems to be a bug.

  > **Name:** `copy-into`
  > **Value**: `none | [ [ <custom-ident> <content-level>] [, <custom-ident> <content-level>]* ]?`

It should probably be `none | [ <custom-ident> <content-level> ]#`.

See https://github.com/w3c/csswg-drafts/pull/8480


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

Received on Tuesday, 21 February 2023 13:01:42 UTC