-
Notifications
You must be signed in to change notification settings - Fork 42
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
fix(alert): prevent content from overflowing alert #138
fix(alert): prevent content from overflowing alert #138
Conversation
👋 @ashleyryan,
Thank you, 🤖 Clarity Release Bot |
- flexbox has some quirks with overflow - setting min-width: 0 prevents the content from overflowing fixes vmware-clarity#137
97d29e3
to
d8d403b
Compare
I made the changes that I could make
🎉 This PR is included in version 6.1.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary. |
fixes #137
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Content with overflow-x: auto that would otherwise render a scrollbar in a small container are instead overflowing the alert
Issue Number: #137
What is the new behavior?
add min-width: 0 to allow content to be smaller than the implied width: https://css-tricks.com/flexbox-truncated-text/
Does this PR introduce a breaking change?
Other information