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

Revert "Allow individual grid classes to override .row-cols" #34612

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

mdo
Copy link
Member

@mdo mdo commented Jul 28, 2021

Reverts #33621.

Rather than go through the motions of splitting our grid, no generating some classes, or generating different classes in different places, I'm proposing we revert this outright and address it properly when we can devote the time to it. Right now this breaks too many things and I'm not sure what the best solution is just yet.

Fixes #34611, fixes #34564, fixes #34335, fixes #34363.

Copy link
Member

@ffoodd ffoodd left a comment

Choose a reason for hiding this comment

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

Idea felt good.

I think we'll need a comprehensive test case with all those examples before trying to refactor the grid.

@danielkauffman
Copy link

Because #33621 causes such a significant regression when going from 5.0.1 to 5.0.2, I think the reversion should be a hot fix, perhaps released by itself as 5.0.3, and not delayed for 5.1.0.

@XhmikosR XhmikosR merged commit 906a990 into main Jul 29, 2021
@XhmikosR XhmikosR deleted the revert-33621-RowColConsistency branch July 29, 2021 03:29
marvin-hinkley-vortx pushed a commit to Vortx-Inc/bootstrap that referenced this pull request Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants