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

ray() syntax is incorrect #722

Closed
pyoor opened this issue Mar 29, 2024 · 1 comment · Fixed by #723
Closed

ray() syntax is incorrect #722

pyoor opened this issue Mar 29, 2024 · 1 comment · Fixed by #723
Labels
idle Issues and pull requests with no activity for three months. needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.

Comments

@pyoor
Copy link
Contributor

pyoor commented Mar 29, 2024

The following was added in #700.

"syntax": "{ <angle> && [closest-side | closest-corner | farthest-side | farthest-corner | sides] && contain? && [at [ left | center | right | top | bottom | <length-percentage> ] | [ left | center | right ] && [ top | center | bottom ] | [ left | center | right | <length-percentage> ] [ top | center | bottom | <length-percentage> ] | [ [ left | right ] <length-percentage> ] && [ [ top | bottom ] <length-percentage> ]]"

This syntax is incorrect and also incorrectly nests the ray-size values as opposed to adding a separate type entry for it.

@caugner caugner added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 29, 2024
@wbamberg
Copy link
Contributor

wbamberg commented Apr 9, 2024

Thanks for filing the issue and a PR to fix it!

I'm interested in knowing how you found this issue. MDN pages don't use the syntax from mdn/data any more, and you can see that the syntax for ray() is correct on MDN: https://developer.mozilla.org/en-US/docs/Web/CSS/ray#formal_syntax.

Is there another place that's exposing the version that's in mdn/data?

@github-actions github-actions bot added the idle Issues and pull requests with no activity for three months. label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idle Issues and pull requests with no activity for three months. needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants