Skip to content

[css-borders-4] border-color property's grammar doesn't agree with its example #5103

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.

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

Closed
litherum opened this issue May 22, 2020 · 3 comments
Closed

Comments

@litherum
Copy link
Contributor

From https://drafts.csswg.org/css-backgrounds-4/#the-border-color:

Name: border-color
Value: <color>#{1,4}

The #{} syntax is defined as such:

A hash mark (#) indicates that the preceding type, word, or group occurs one or more times, separated by comma tokens ... It may optionally be followed by the curly brace forms, above, to indicate precisely how many times the repetition occurs, like <length>#{1,4}.

But then css-backgrounds-4 goes on to list an example:

border-color: skyblue orange yellowgreen indianred, black yellow;

I don't think this example conforms to the property's grammar according to the definition listed above.

@SebastianZ
Copy link
Contributor

So it looks like the syntax should rather be <color>{1,4}# or [ <color>{1,4} ]#.

Sebastian

@fantasai
Copy link
Collaborator

@litherum ... Just so you know, I don't think this is a spec that should be referenced by anybody right now. ^_^;; It's got a "don't implement yet" warning and isn't FPWD for a reason.

Agreed there's an error here, but I think the resolution was to change the grammar to background-color: [ <color> | <image-1D> ]{1,4}.

@SebastianZ SebastianZ changed the title [css-backgrounds-4] border-color property's grammar doesn't agree with its example [css-borders-4] border-color property's grammar doesn't agree with its example Aug 12, 2023
@SebastianZ
Copy link
Contributor

Agreed there's an error here, but I think the resolution was to change the grammar to background-color: [ <color> | <image-1D> ]{1,4}.

It's border-color, but yes. It got changed to [ <color> | <image-1D> ]{1,4} meanwhile. So this issue is solved now.

Sebastian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants