Skip to content
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

COMMA delimiter missing in <text-shadow> #720

Closed
palemieux opened this issue Apr 27, 2018 · 1 comment
Closed

COMMA delimiter missing in <text-shadow> #720

palemieux opened this issue Apr 27, 2018 · 1 comment
Labels
bug pr merged substantive
Milestone
1ED-CR2

Comments

@palemieux
Copy link
Contributor

As currently specified, the <text-shadow> syntax cannot distinguish between a list of three shadows each with two components, and a list of two shadows each with three components.

Suggest specifying the following, which matches CSS:

  : "none"
  | <shadow> (<lwsp>? "," <lwsp>? <shadow>)*
@nigelmegitt
Copy link
Contributor

+1 to the proposed fix.

@skynavga skynavga added this to the CR2 milestone May 10, 2018
skynavga added a commit that referenced this issue May 12, 2018
@skynavga
Use COMMA as separator in a list of text shadows (#720).
e8ea777
@skynavga skynavga self-assigned this May 12, 2018
@skynavga skynavga mentioned this issue May 12, 2018
Merged
skynavga added a commit that referenced this issue May 25, 2018
@skynavga
Merge pull request #737 from w3c/issue-0720-text-shadow-separator
2065751 
Use COMMA as separator in a list of text shadows (#720).
@skynavga skynavga removed their assignment May 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug pr merged substantive
Projects
None yet
Milestone
1ED-CR2
Development

No branches or pull requests
3 participants
@skynavga @palemieux @nigelmegitt