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

Impending name clash with overflow-clip #4622

Closed
valtism opened this issue Jun 11, 2021 · 1 comment · Fixed by #5630
Closed

Impending name clash with overflow-clip #4622

valtism opened this issue Jun 11, 2021 · 1 comment · Fixed by #5630

Comments

@valtism
Copy link
Contributor

valtism commented Jun 11, 2021

Reproduction repository

www.IUnderstandWhyYouRequireThisButInMyCaseItsNotReallyNecessary.com

Describe your issue

There is a new overflow type being adopted by browsers, overflow: "clip". You can see it picking up adoption here: https://caniuse.com/mdn-css_properties_overflow_clip

The idea of it is to fix issues with overflow: "hidden" where it does a similar thing but does not break any position: "sticky"'d elements inside it.

RIght now, tailwind defines the class of overflow-clip as text-overflow: "clip", so you can see that this will become a problem when the overflow spec is more widely adopted and used.

@reinink
Copy link
Member

reinink commented Sep 28, 2021

Hey thanks for making us aware of this, we've gone ahead and renamed overflow-clip to text-clip, and now are using overflow-clip for overflow: clip 👍

#5630

@reinink reinink closed this as completed Sep 28, 2021
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 a pull request may close this issue.

2 participants