search

mdn / data

This repository contains general data for Web technologies
https://developer.mozilla.org
Creative Commons Zero v1.0 Universal
724 stars 189 forks source link

`overflow-clip-margin` syntax seems incorrect #729

Closed salmenf closed 3 months ago

salmenf commented 5 months ago

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

github-actions[bot] commented 5 months ago

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.

caugner commented 3 months ago

@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: https://github.com/w3c/csswg-drafts/issues/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!

salmenf commented 3 months ago

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.