-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Add equal column mixin #32155
Add equal column mixin #32155
Conversation
a7dbc6b
to
78a68ae
Compare
I think I can dig this. Put together a little Codepen: https://codepen.io/emdeoh/pen/JjRbXYr. Since this only changes the mixin, I don't see any problems for us. Wondering if we need to update any docs? |
Do you need me to make edits to that codepen? It looks like what I have in mind. I can add a quick note for the docs about the default behavior. |
No codepen edits needed. Just some docs updates I think to mention the behavior. Not sure where myself just yet, so feel free to suggest! |
9aa086c
to
0fe5300
Compare
Make equal cols the default Add documentation
0fe5300
to
cdf4606
Compare
Ok, thank you! Added a docs edit and rebased 👍🏻 |
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.
LGTM, pretty nice idea 👌
Add equal column mixin so that we can do@include make-col-eq
instead of having to extend the.col
class or use it in markup.Make the
make-col
mixin default to equal columns when not providing a column size.