Skip to content

Update display.js to include display:contents utility class #2023

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

Merged
merged 1 commit into from
Aug 15, 2020

Conversation

mtimbs
Copy link
Contributor

@mtimbs mtimbs commented Jul 14, 2020

Add utility class for display: contents. This is useful for flex and grid. You can read more here https://css-tricks.com/get-ready-for-display-contents/

Add utility class for `display: contents`. This is useful for flex and grid. You can read more here https://css-tricks.com/get-ready-for-display-contents/
@adamwathan adamwathan merged commit 94325f5 into tailwindlabs:master Aug 15, 2020
@adamwathan
Copy link
Member

Thanks!

@knowler
Copy link

knowler commented Aug 18, 2020

One should use display: contents very cautiously as there is an accessibility impact involved with its effect despite its apparent usefulness for other applications. I'd go as far as not including it as an out-of-the-box Tailwind utility and leave it up to userland to add via config or plugin if needed. It is just one rule, but given Tailwind's pervasiveness and the general lack of accessibility knowledge amongst developers (especially in regards to strange cases like this), this one would be best left out of immediate reach (for now).

workashutosh pushed a commit to workashutosh/tailwindcss that referenced this pull request Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants