Skip to content

Remove .btn-block for utilities#30587

Closed
mdo wants to merge 3 commits intomasterfrom
v5-btn-block-as-utils
Closed

Remove .btn-block for utilities#30587
mdo wants to merge 3 commits intomasterfrom
v5-btn-block-as-utils

Conversation

@mdo
Copy link
Member

@mdo mdo commented Apr 15, 2020

Alternate fix for #30376, which closes #30223.

Alternate fix for #30376, which closes #30223
@mdo mdo requested a review from a team as a code owner April 15, 2020 04:33

### Buttons

- Removed `.btn-block` as it duplicated functionality of utilities, but in a more limited way. With a utilities based approach, "block buttons" are now easily made responsive.
Copy link
Member

Choose a reason for hiding this comment

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

I think we should mention the exact replacement classes to make things easier for people 🙂

@MartijnCuppens
Copy link
Member

I don't think we need the .d-block class, only .w-100 should be ok. We can even ditch the mb-2, since the inline blocks already have some spacing.

@mdo
Copy link
Member Author

mdo commented Apr 15, 2020

Might not go this route after all actually... requires more utility classes to undo the responsive blocks (assuming block on mobile, not block on say medium). 🤔

@mdo
Copy link
Member Author

mdo commented Jun 16, 2020

Closing per my last comment.

@mdo mdo closed this Jun 16, 2020
@mdo mdo deleted the v5-btn-block-as-utils branch June 16, 2020 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for block buttons based on specific screen widths, eg. btn-xs-block

3 participants