-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 Convert to Regular Blocks button to ellipsis Dropdown #32310
Add Convert to Regular Blocks button to ellipsis Dropdown #32310
Conversation
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.
Thanks for working on this @thisissandip !
While the code here looks good, this changed was introduced here: https://github.com/WordPress/gutenberg/pull/28318/files#diff-3e104de22b2ea1fac4d9f2fd42f07a1c8ff39bcb9d0bfa43b056538e74e1c558R124
and I'm wondering if there had been related discussion about this and this change happened.
--cc @mtias @jasmussen
Also related: #30993 |
...s/reusable-blocks/src/components/reusable-blocks-menu-items/reusable-blocks-manage-button.js
Outdated
Show resolved
Hide resolved
Co-authored-by: Nik Tsekouras <[email protected]>
I am linking in this Trac comment: https://core.trac.wordpress.org/ticket/52779#comment:25 |
Thanks for the PR! I don't have a strong opinion here: the menu item is redundant duplicate UI to the button in the toolbar (which by the way is an icon button as the reusable block name can and often will grow very long). Redundant UI can be okay. I would follow the sentence case menu item standards, though, so it should be "Convert to regular blocks" not "Convert to Regular Blocks". |
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 👍 - thanks!
Let's land with a green CI.
Be sure to also get this included:
Thanks for approving and merging this PR, Nik! |
@peterwilsoncc Peter. We have a possible merged PR that can be backported to WP 5.7.3. |
It's included yes. |
Thank you for reviewing and merging this PR! @ntsekouras @paaljoachim @jasmussen |
@youknowriad Should we backport this PR into WP 5.8? |
As mentioned on #34393, I've decided to not include this in 5.8.1 as this is more of a bug fix by way of enhancement. I think adding UI elements and controls should only be done in minor releases when absolutely necessary. |
Fixes #32217
Description
Added convert to regular blocks button to ellipsis dropdown for reusable blocks
How has this been tested?
Screenshots
convert_regular_blocks.mov
Types of changes
Bug fix
Checklist: