remove broken 2-column breakpoint from columns block#13605
remove broken 2-column breakpoint from columns block#13605drdogbot7 wants to merge 1 commit intoWordPress:masterfrom
Conversation
|
Thanks so much for keeping on this, your help fixing up the columns stuff has been tremendously valuable. Here's a GIF: In this PR, there's 1 column until after the medium breakpoint, where the user-set amount of columns are shown. This is certainly simpler — it simplifies the CSS to be more understandable, and it avoids us having to stack things. However in light of the arguments in #12199 (comment), and the creation of #13363 to track additional responsive enhancements — is this PR something we should land or will it be subsumed by other improvements? |
#13363 is really cool. It would give a lot more control to the end-user via the GUI. So the end user could set up columns however they want without fussing with code. If this is implemented, it will mean completely re-writing a lot of the css anyway. #13605 (this PR) takes the opposite approach and keeps the columns block as simple as possible, and leaves it up to the theme designer to add any intermediate breakpoints. I don't know what is better. I think it really depends on what the role of the core gutenberg blocks is. |
I will add that one of the intentions of #13363 is to continue putting the onus on theme (or core) developers to build in the best-case recommended behavior for these breakpoints. If the end user chooses to override that, we'd drop whatever smart responsive behavior we've built in, and let them manually adjust behavior across a few key breakpoints. So from our perspective here, I think the main question is: will dropping the middle-breakpoint for the columns block result in the smartest behavior for this block? I'd lean towards saying no... after all, this is a "Columns" block. I'd expect to see more than one column on most devices. |
|
I'm going to go ahead and close this. |
|
Can I just say thank you for all the work you've put into this? Please consider adding yourself to https://github.com/WordPress/gutenberg/blob/master/CONTRIBUTORS.md, I will approve the PR in a heartbeat. Or, if you'll permit me to add you, you totally should be there. |
+1. Thank you, @drdogbot7! You've really helped sort out some confusing details around this. |
|
Thanks guys! I'll be excited to see where this goes. Glad to be part of the discussion! |
|
Created #13724. |

Description
Removes the non-working 2-column breakpoint and resolves margin-related bugs caused by it.
This is discussed extensively in #12199 and #12816, both of which attempt to fix the bugs while retaining the 2-column breakpoint.
This is an alternative approach that simplifies the code significantly and which should make things easier on theme-creators in the future.
How has this been tested?
Locally using Docker environment. Chrome, Safari, Firefox on Mac. Need to test IE11/Edge.
Types of changes
SCSS changes only. Almost exclusively removing code that didn't work properly anyway. This may cause some minor disruption in themes (including Twenty-Nineteen) that already implemented their own fixes to this block.
Checklist: