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

overflow-clip-margin syntax seems incorrect #729

Closed
salmenf opened this issue Apr 15, 2024 · 3 comments
Closed

overflow-clip-margin syntax seems incorrect #729

salmenf opened this issue Apr 15, 2024 · 3 comments
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

@salmenf
Copy link

salmenf commented Apr 15, 2024

What information was incorrect, unhelpful, or incomplete?

The syntax for the CSS property overflow-clip-margin seems incorrect:

"syntax": "<visual-box> || <length [0,∞]>" (https://github.com/mdn/data/blob/main/css/properties.json#L7578)

What did you expect to see?

I expected it to be "syntax": "<visual-box> || <length>", since no other property in the whole data set uses that range syntax [0,∞].

Do you have any supporting links, references, or citations?

The range syntax is not explained here at all: https://developer.mozilla.org/en-US/docs/Web/CSS/Value_definition_syntax
Although, the current syntax comes straight from the spec, it seems: https://drafts.csswg.org/css-overflow/#overflow-clip-margin

Do you have anything more you want to share?

No response

@salmenf salmenf added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Apr 15, 2024
Copy link
Contributor

It looks like this is your first issue. Welcome! 👋 One of the project maintainers will be with you as soon as possible. We appreciate your patience. To safeguard the health of the project, please take a moment to read our code of conduct.

@github-actions github-actions bot added the idle Issues and pull requests with no activity for three months. label May 17, 2024
@caugner
Copy link
Contributor

caugner commented Jun 14, 2024

@salmenf Thanks for reporting. As you mention, the MDN page correctly reproduces the syntax from the CSS specification, so there is no issue on our side that needs addressing and I'm closing this issue as "won't fix".

Note that there seems to be a somewhat related issue reported in the csswg-drafts repo: w3c/csswg-drafts#8381

If you believe this is an issue in the specification, please consider creating a new csswg-drafts issue, providing as much detail as you can. Thanks!

@caugner caugner closed this as not planned Won't fix, can't repro, duplicate, stale Jun 14, 2024
@salmenf
Copy link
Author

salmenf commented Jun 15, 2024

Thanks for the reply! Looking back on this, indeed the issue is more that the new range syntax is not included in the syntax summary on MDN. I'll open an issue there.

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

No branches or pull requests

2 participants