Re: [csswg-drafts] Apply the range definition notation (#7400)

> - [ ] [css-fonts-4] [`font-style`](https://drafts.csswg.org/css-fonts-4/#font-style-prop): `normal | italic | oblique <angle [-90,90]>?`

I think this requires units, i.e., `<angle [-90deg,90deg]>`

> - [ ] [css-animations-1] [`animation-duration`](https://drafts.csswg.org/css-animations-1/#animation-duration): `<time [0,∞]>#`
> - [ ] [css-animations-1] [`animation-iteration-count`](https://drafts.csswg.org/css-animations-1/#animation-duration): `<time [0,∞]>#`
> - [ ] [css-transitions-1] [`transition-duration`](https://drafts.csswg.org/css-transitions-1/#transition-duration-property): `<time [0,∞]>#`

Likewise, I think these three require units since `0` is not a valid `<time>`, so they should be `<time [0s,∞]>`.

> - [ ] [css-images-4] [`<cf-image>`](https://drafts.csswg.org/css-images-4/#typedef-cf-image): `<percentage [0,100]>? && [<image> | <color>]`

I think we should probably also use units for percentages, although I'm less confident of that.  That is, `<percentage [0%,100%]>`.

> ## Multiple interval syntax
> - [ ] [css-grid-1][css-grid-2] [`<grid-line>`](https://drafts.csswg.org/css-grid-1/#typedef-grid-row-start-grid-line): `auto | <custom-ident> | [<integer [-∞,-1] [1,∞]> && <custom-ident>?] | [span && [ <integer [1,∞]> || <custom-ident>]]`

I think you could write `[ <integer [-∞,-1]> | <integer [1,∞]> ]` for this one case.

-- 
GitHub Notification of comment by dbaron
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/7400#issuecomment-1163191465 using your GitHub account


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

Received on Wednesday, 22 June 2022 14:39:47 UTC