-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
Dismissible alert - long text overlaps close button #24469
Comments
Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and OS details. |
Here is a fiddle - https://jsfiddle.net/ggefb28f/2/ |
they are no issue in your JSFiddle 😲 |
Try to resize browser window ;-) I clearly see that text doesn't wrap when reaching the close button |
yes true .. it's wrap but too late .... |
Would love to contribute to bootstrap and have submitted a PR to resolve this. #24478 |
Yeah, we knew of this when we merged it: #23673 @Kopyov on that PR you'll also find the explanation of why the absolute position. @MattJLeach I'll check your PR, thanks! |
Quick workaround:
Not sure though if ($close-font-size / 2) is a good solution. |
@Kopyov you are right! we have a parent selector! That was the exact fix I had in my head. It does create a larger padding to the right, but I'd be ok with it, Do you want to PR it? |
.alert-dismissible {
padding-right: ($close-font-size + $alert-padding-x);
} |
@andresgalante added your version, since it saves some space 😊 |
Not sure i understand why close button now has absolute positioning in Beta2, but now if text in alert is too long, it overlaps close button. I think we need to add extra right padding to dismissible alert to ensure text and button are always visible.
The text was updated successfully, but these errors were encountered: