[csswg-drafts] [css-images-3][css-images-4] Typos in the expanded definition of radial gradient (#6425)

cdoublev has just created a new issue for https://github.com/w3c/csswg-drafts:

== [css-images-3][css-images-4] Typos in the expanded definition of radial gradient ==
I think there is a typo and a missing range in [3.2.1. radial-gradient() Syntax](https://drafts.csswg.org/css-images-3/#radial-gradient-syntax) (the note at the very bottom of the section, css-images-3), for which I suggest the following changes:

```diff
  Expanded with the above definitions, the grammar becomes:

  radial-gradient() = radial-gradient(
-   [ [ circle               || <length> ]                          [ at <position> ]? , |
-     [ ellipse              || <length-percentage>{2} ]            [ at <position> ]? , |
+   [ [ circle               || <length [0,∞]> ]                    [ at <position> ]? , |
+     [ ellipse              || <length-percentage [0,∞]>{2} ]      [ at <position> ]? , |
      [ [ circle | ellipse ] || <extent-keyword> ]                  [ at <position> ]? , |
      at <position> ,
    ]?
-   <color-stop> [ , <color-stop> ]+
+   <color-stop-list>
  )
  <extent-keyword> = closest-corner | closest-side | farthest-corner | farthest-side
```

Ie.:

- `<size>` should be positive
- a `<color-stop-list>` should come after `[ <ending-shape> || <size> ]? [ at <position> ]?` instead of `<color-stop> [ , <color-stop> ]+` because the latter is currently expanded to a definition without a `<color>`*, or I suggest to change it [there](https://drafts.csswg.org/css-images-4/#color-stop-syntax) (css-images-4):

```diff
- <color-stop> = <color-stop-length> | <color-stop-angle>
+ <color-stop> = <linear-color-stop> | <angular-color-stop>
```

*: `<color-stop-length> = <length-percentage>{1,2}`, and `<color-stop-angle> = <angle-percentage>{1,2}`

Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6425 using your GitHub account


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

Received on Tuesday, 6 July 2021 12:50:54 UTC