-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Adds a border radius to modal headers #23799
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are several unrelated changes in this PR.
09111d7
to
7608707
Compare
@mdo I am sorry I had an auto formatter on. 😞 I fixed the PR but it's still showing a strange diff, I don't know why. |
@andresgalante @mdo |
…plied The original bug (twbs/bootstrap#23798) was fixed in Bootstrap V4.beta.2: twbs/bootstrap#23799
@andresgalante Actually a similar bug persists when using a background on modal-footer (it protrudes from the bottom rounded corners) |
…plied (#1433) The original bug (twbs/bootstrap#23798) was fixed in Bootstrap V4.beta.2: twbs/bootstrap#23799
This PR fixes #23798
When a background color is added to the modal header it shows a square corner. This PR is adding border radius to it so it follows the border radius.
Should I add a section on modal documentation about styling it? How to change the color of the border, text and background with utility classes like we do on cards
What do you think?