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

Syntax section erroneously includes minmax as value #38346

Open
cssinate opened this issue Feb 26, 2025 · 1 comment
Open

Syntax section erroneously includes minmax as value #38346

cssinate opened this issue Feb 26, 2025 · 1 comment
Labels
Content:CSS Cascading Style Sheets docs

Comments

@cssinate
Copy link

MDN URL

https://developer.mozilla.org/en-US/docs/Web/CSS/height

What specific section or headline is this issue about?

Syntax

What information was incorrect, unhelpful, or incomplete?

Neither the level 3, nor level 4 spec allow for minmax as a value of height.

Image

Level 3 spec:
Image

Level 4 spec:
Image

What did you expect to see?

All the same information but with this line removed

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

https://drafts.csswg.org/css-sizing/#preferred-size-properties
https://drafts.csswg.org/css-sizing-4/#sizing-values

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@cssinate cssinate added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Feb 26, 2025
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Feb 26, 2025
@Josh-Cena
Copy link
Member

Josh-Cena commented Feb 26, 2025

These were introduced in #33511.

@chrisdavidmills minmax is not a mathematical function and I don't think it can be used outside of grids (unless some other spec expanded its usage that I'm not aware of; I don't see it in anchor positioning though). Perhaps you meant max, or perhaps we should just:

  1. Remove this minmax() example, since if you change it to min/max, then it's not a primitive construction anymore
  2. We definitely should move the two anchor-size() examples down, because they are keyword values, not <length> values

@Josh-Cena Josh-Cena removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs
Projects
None yet
Development

No branches or pull requests

2 participants