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.

Sign up for GitHub

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

Jump to bottom

All major browsers now support display: contents #32605

Closed
yin1999 opened this issue Mar 8, 2024 · 4 comments
Closed

All major browsers now support display: contents #32605

yin1999 opened this issue Mar 8, 2024 · 4 comments
Labels
Content:CSS Cascading Style Sheets docs goal: up-to-date (Experimental label) Issues about outdated content. help wanted If you know something about this topic, we would love your help!

Comments

@yin1999
Copy link
Member

yin1999 commented Mar 8, 2024

MDN URL

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

What specific section or headline is this issue about?

https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_flexible_box_layout/Relationship_of_flexbox_to_other_layout_methods#flexbox_and_display_contents

What information was incorrect, unhelpful, or incomplete?

The document states that current browsers have limited support for display: contents. This description is outdated as all major browsers now support this value.

What did you expect to see?

update this statement

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

caniuse: https://caniuse.com/css-display-contents

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@yin1999 yin1999 added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 8, 2024
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Mar 8, 2024
@yin1999 yin1999 mentioned this issue Mar 8, 2024
Merged
@caugner caugner added the goal: up-to-date (Experimental label) Issues about outdated content. label Mar 15, 2024
@Josh-Cena Josh-Cena added help wanted If you know something about this topic, we would love your help! and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jun 5, 2024
@evelinabe
Copy link
Contributor

I was looking into this with the intention of helping with the issue. However, from my understanding the note is still correct. It specifies that the current implementations in most browsers interferes with accessibility. This is also what is being stated on canius when hovering the statuses (see image below).

My suggestion is to close this issue without further action.

screenshot of css display:contents on canius.com
@yin1999
Copy link
Member Author

yin1999 commented Jun 12, 2024

The the implementation issue is already stated above this paragraph (see the warning), and the issue exists in both Firefox and Chrome. It is still inappropriate to say "Firefox supports display: contents already, the value is being implemented in Chrome" here.
@evelinabe evelinabe mentioned this issue Jun 27, 2024
Closed
@evelinabe
Copy link
Contributor

evelinabe commented Jun 27, 2024 •
edited
Loading

You're right, added a PR to update this statement.

Issue has been solved in #34149 , please close this issue.
@yin1999
Copy link
Member Author

yin1999 commented Jun 28, 2024

You're right, added a PR to update this statement.

Issue has been solved in #34149 , please close this issue.

Yes, the relevant content has been updated! I'm going to close this :)
@yin1999 yin1999 closed this as completed Jun 28, 2024
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 goal: up-to-date (Experimental label) Issues about outdated content. help wanted If you know something about this topic, we would love your help!
4 participants
@caugner @yin1999 @evelinabe @Josh-Cena