Skip to content

Responsive Card Decks alternative#29274

Closed
MartijnCuppens wants to merge 5 commits into
masterfrom
row-cols-mc
Closed

Responsive Card Decks alternative#29274
MartijnCuppens wants to merge 5 commits into
masterfrom
row-cols-mc

Conversation

@MartijnCuppens

@MartijnCuppens MartijnCuppens commented Aug 18, 2019

Copy link
Copy Markdown
Member

Alternative implementation of #29073. In draft to show difference.

The .row-cols-* modifier now targets all children, and the .col- classes can still be used to override this behaviour if needed. This implementation has cards of 100%, which makes them have equal heights by default.

TODOs:

  • Make the card height variable.

mdo and others added 4 commits August 16, 2019 13:36
Trying to find a new way to do responsive card decks while not locking ourselves into the cards themselves. My thinking here is we can easily control the column (.col) width by the parent, but I don't know how many we need (have 0-5 now) across each breakpoint. This works for cards so far, and I think could get us equal height, too.
- Rename and move the variable to variables file
- Move code to the grid file
- Use the mixin to generate our own classes
- Wrap in a grid classes enabled conditional
- Document the mixin
@MartijnCuppens

MartijnCuppens commented Aug 27, 2019

Copy link
Copy Markdown
Member Author

The height: 100% (.card) will be a breaking change for v4, so maybe I should change that in a configurable null variable.

@ysds ysds mentioned this pull request Aug 28, 2019
@MartijnCuppens

Copy link
Copy Markdown
Member Author

#29317 makes more sense, closing this

@XhmikosR XhmikosR deleted the row-cols-mc branch August 28, 2019 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants