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

v4: Card improvements #19885

Merged
merged 6 commits into from
May 12, 2016
Merged

v4: Card improvements #19885

merged 6 commits into from
May 12, 2016

Conversation

mdo
Copy link
Member

@mdo mdo commented May 12, 2016

WIP.

  • 17911: Explicitly remove margin-bottom from cards in .card-deck.

    Here we're doing some margin swapping, so it looks a little funky. All this does is match the margin implementation and rendering across our table and flex versions of card decks.

  • 17959, Cards in .card-deck do not always have same width #19650: Improve non-flexbox card deck behavior so columns are always the same width.

  • 19840: Invert the .card-subtitle color as well for .card-inverse.

  • Update card variants:

  • Update card header and footer

    • Remove the box-shadow and switch back to border to match .card basics
    • Update the header nav margin override—since we restored the border, we need that default negative margin at the bottom

mdo added 5 commits May 11, 2016 20:22
- Remove the box-shadow and switch back to border to match .card basics
- Update the header nav margin override--since we restored the border, we need that default negative margin at the bottom
- Remove background-color from header and footer in inverse cards (fixes #19841)
- Update border override for header and footer in cards; only need to declare a new color
- Update the selector for targetting blockquote footers in cards
Here we're doing some margin swapping, so it looks a little funky. All this does is match the margin implementation and rendering across our table and flex versions of card decks.
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.

1 participant