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

Have same margin for <hN> as a <div> on .card-headers #19048

Merged
merged 2 commits into from
Sep 12, 2016
Merged

Have same margin for <hN> as a <div> on .card-headers #19048

merged 2 commits into from
Sep 12, 2016

Conversation

mkroeders
Copy link
Contributor

Related #18726

When you use a <hN> as .card-header there is margin-bottom: .5rem added. The changes set the margin-bottom to 0

Related #18726

When you use a `<hN>` as `.card-header` there is `margin-bottom: .5rem` added. The changes set the margin-bottom to 0
@@ -71,6 +71,7 @@
.card-header {
@include clearfix;
padding: $card-spacer-y $card-spacer-x;
margin-bottom: 0; // Removes the default margin-bottom of <hN>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line contains trailing whitespace

@mdo mdo added this to the v4.0.0-alpha.5 milestone Sep 12, 2016
@mdo mdo merged commit 53a20c1 into twbs:v4-dev Sep 12, 2016
@mdo
Copy link
Member

mdo commented Sep 12, 2016

Merged this for now, but I may come back to revert this and similar changes in favor of margin utilities that can easily remove the margin-bottom for you.

@jamesharvard
Copy link

Hey, @mkroeders & @mdo, given the default appearance seems to require zero margin, might it be preferable to just set all margins with margin: 0;?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants