-
Notifications
You must be signed in to change notification settings - Fork 661
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
[css-transforms-1] Simplify transform function syntaxes #4054
Conversation
Marked as non substantive for IPR from ash-nazg. |
Needs an entry in the Changes section, e.g. “Simplified grammar of transform functions. (No normative impact.)”, but otherwise it looks correct afaict. |
Simplified grammar of transform functions. (No normative impact.)
@fantasai Done. Btw I'm not sure such entries is needed in that case. At least no entries were added in my other PRs with similar fixes. |
Yeah, non-normative changes aren't always explicitly tracked. But it's still good practice to do so. ^_^ |
@tabatkins Thank you! Could you take a look at the relevant PR #4053? (it's just about |
Are the changes to translate (and others correct? Isn’t it possible to write |
(I’d also appreciate to wait for the feedback of one of the spec editors before merging. The editors agreed not to merge PRs w/o feedback of other editors given the current state of the spec.) |
https://www.w3.org/TR/css-values-4/#component-types
|
As this was a non-substantial grammar fix, and I'm one of the editors of the spec defining said grammar, I figured it was fine to do a merge. I wouldn't merge anything subject-matter-related without one of y'all editors approving. ^_^ |
Simplify syntaxes for
matrix()
,translate()
,scale()
andskew()