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

[motion-1] Invalid offset-path value syntax #410

Comments

Copy link

The ray() and path() functions in the offset-path value definition must be wrapped in <> to identify them as CSS types.

Value: none | ''ray()'' | <a href="#offsetpath-pathfunc">path()</a> | <<url>> | [ <<basic-shape>> && <<coord-box>>? ] | <<coord-box>>

The correct syntax should be:
none | <ray()> | <path()> | <url> | [ <basic-shape> && <coord-box>? ] | <coord-box>

@jihyerish
Copy link

I think you're right. I couldn't find the exact guide of value definition syntax for functional values in https://drafts.csswg.org/css-values-3/#value-defs.
But most of the functional values such as url(), *-gradient() are wrapped in <>.

@SebastianZ
Copy link
Contributor

The definition in https://drafts.csswg.org/css-values-4/#component-types (note that level 4 is the current version worked on) doesn't explicitly mention how functional values have to look like, i.e. it doesn't say that their names need to be followed by some parentheses. Though as their definition is somewhere else, they are non-terminal values. And by that they are required to be surrounded by < and >.

Bikeshed also has a note about that at https://tabatkins.github.io/bikeshed/#autolink-wrappers.

Sebastian

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

Successfully merging a pull request may close this issue.

3 participants