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

Wrong syntax of column-width, not actually implemented #36001

Closed
Eccenux opened this issue Sep 22, 2024 · 3 comments
Closed

Wrong syntax of column-width, not actually implemented #36001

Eccenux opened this issue Sep 22, 2024 · 3 comments
Labels
Content:CSS Cascading Style Sheets docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.

Comments

@Eccenux
Copy link
Contributor

Eccenux commented Sep 22, 2024

MDN URL

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

What specific section or headline is this issue about?

Formal syntax

What information was incorrect, unhelpful, or incomplete?

column-width = 
  auto                                |
  <length [0,∞]>                      |
  min-content                         |
  max-content                         |
  fit-content( <length-percentage> )  

<length-percentage> = 
  <length>      |
  <percentage>  

What did you expect to see?

column-width = 
  auto                                |
  <length [0,∞]>                      |

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

Here is the current syntax:
https://www.w3.org/TR/css-multicol-1/#propdef-column-width

This is an autocomplete in Chrome (similar in Firefox):
screen from Chrome devtools

As you can see, auto | <length [0,∞]> is what's actually implemented in browsers.

Note that additions to column-width were already considered at-risk in Level 3: https://www.w3.org/TR/css-sizing-3/

Do you have anything more you want to share?

No response

MDN metadata

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

Josh-Cena commented Sep 22, 2024

We don't use the published w3 specs as source, because those are arbitrarily cut versions. Check the latest syntax at https://drafts.csswg.org/css-sizing/#column-sizing. It's a known issue that sometimes this may get too new and thus unimplemented anywhere but that's a big project to solve.

@Eccenux
Copy link
Contributor Author

Eccenux commented Sep 23, 2024

Well in that case the compat tabel is wrong (browsers clearly don't implement additional sizing):
mdn/browser-compat-data#24493

@Eccenux
Copy link
Contributor Author

Eccenux commented Sep 23, 2024

Sorry to bother you, but I also noticed that other section is wrong, assuming this is correct: min-content, max-content, fit-content( <length-percentage> ).

These values are not included in the Syntax section, either in the syntax examples or in the value descriptions.

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 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