Skip to content

[css-grid-1] repeat syntax uses <positive-integer> which is not defined #4479

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
fantasai opened this issue Nov 1, 2019 · 1 comment
Closed

[css-grid-1] repeat syntax uses <positive-integer> which is not defined #4479

fantasai opened this issue Nov 1, 2019 · 1 comment
Labels
Closed Accepted as Obvious Bugfix css-grid-1
Milestone
css-grid-1 CR 201...

Comments

@fantasai
Copy link
Collaborator

fantasai commented Nov 1, 2019 •
edited
Loading

The repeat syntax definition uses <positive-integer> as its first argument; this goes back to the first WD. It's not entirely clear if this was meant to be 1+ or 0+, so that needs to be clarified.

(It should also switch over to the new css-values syntax for range restrictions.)
@fantasai
Copy link
Collaborator Author

fantasai commented Nov 1, 2019

Looks like Gecko and Blink both treat zero as invalid, so going ahead and speccing that. Testcase

fantasai added a commit that referenced this issue Nov 1, 2019
@fantasai
[css-grid-1] Switch <positive-integer> to valid css-values syntax. #4479
ccd94de
@fantasai fantasai closed this as completed Nov 1, 2019
@fantasai fantasai added this to the css-grid-1 CR 2017-12-14+ milestone Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Closed Accepted as Obvious Bugfix css-grid-1
Projects
None yet
Milestone
css-grid-1 CR 2017-12-14+
Development

No branches or pull requests
1 participant
@fantasai