- Notifications
You must be signed in to change notification settings - Fork 4.6k
Remove ungroup option for section blocks #73183
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
Conversation
| Size Change: -3 B (0%) Total Size: 2.42 MB
ℹ️ View Unchanged
|
| Looks good! Thanks for getting it started. What do you think about "Paste styles"? Should users be able to paste random styles into a pattern? |
andrewserong left a comment
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.
Good idea curating these! I think this PR could be good to go if you just want to deal with the "ungroup" case?
This PR is testing nicely for me, and for section blocks, the "Disconnect pattern" option is still available in the menu which is good (and works nicely for folks who might have been looking for Ungroup):
Synced patterns just removes copy/paste altogether 😄 @scruffian has a PR up to add "Edit pattern" to the context menu as well That reminds me, perhaps we should update the copy of "Unlock design" to "Edit pattern" in the sidebar button as well. 🤔 |
We also use it for template parts, though, don't we? |
Good point! |
| The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
| Marking this ready for review and adjusting the title as I think we should get it in as a prequel to #73199. |
andrewserong left a comment
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.
I agree. Let's get it in!
| Nice work! |

What?
Fixes #72764.
Removes the "ungroup" option for section wrappers.
Testing Instructions