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.

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

[cssom] Color properties should use "used value" as "resolved value" #566

Closed
upsuper opened this issue Oct 4, 2016 · 5 comments
Closed
Assignees
Labels

Comments

@upsuper
Copy link
Member

upsuper commented Oct 4, 2016

In the new CSS Color spec, computed value of <color> could be either a numeric color or currentcolor keyword. However, color properties currently always return numeric color value, and web content may depend on that behavior. I think we should probably make those properties use "used value" as "resolved value" for getComputedStyle.

@upsuper
Copy link
Member Author

upsuper commented Dec 2, 2016

Also that matches behavior of browsers which have implemented computed-value time currentcolor keyword.

@svgeesus svgeesus added the css-color-4 Current Work label Dec 7, 2016
@frivoal
Copy link
Collaborator

frivoal commented Dec 14, 2016

Agenda+ because w3c/csswg-test#1161 is blocked by this.

@tabatkins
Copy link
Member

WG resolved to accept this.

@tabatkins tabatkins added Needs Edits and removed Agenda+ css-color-4 Current Work labels Dec 21, 2016
@svgeesus
Copy link
Contributor

also clarified on call, no impact of this change on css3-color or css4-color.

@svgeesus svgeesus changed the title [cssom][css-color] Color properties should use "used value" as "resolved value" [cssom] Color properties should use "used value" as "resolved value" Dec 21, 2016
@frivoal
Copy link
Collaborator

frivoal commented Dec 21, 2016

The WG resolved (ah!) in favor of making <color> properties use used values as resolved values.

@therealglazou @zcorpan I think you guys are the editors, can you do the update?

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

No branches or pull requests

5 participants