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

Navigation Block: prevent submenu items from merging #30164

Open
annezazu opened this issue Mar 24, 2021 · 11 comments
Open

Navigation Block: prevent submenu items from merging #30164

annezazu opened this issue Mar 24, 2021 · 11 comments
Labels
[Block] Buttons Affects the Buttons Block [Block] Navigation Affects the Navigation Block Needs Design Feedback Needs general design feedback. [Type] Bug An existing feature does not function as intended

Comments

@annezazu
Copy link
Contributor

Description

If you add multiple sub menu items and then hit backspace at the beginning of the second item, it merges with the first.

Step-by-step reproduction instructions

  1. Add a navigation block
  2. Start new.
  3. Add a Link Block.
  4. Use the toolbar option to add a sub menu item under the link block you just created.
  5. Do this once more.
  6. Go to the very start of the second sub menu item you created and hit the backspace.
  7. Find that the sub menu items merge together.

Expected behaviour

I expect the items to stay separate.

Actual behaviour

They merged :)

Screenshots or screen recording (optional)

merging.mov

WordPress information

  • WordPress version: 5.7
  • Gutenberg version: 10.2.1
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? TT1 Blocks

Device information

  • Device: Desktop
  • Operating system: MacOS
  • Browser: Chrome 88
@annezazu annezazu added [Type] Bug An existing feature does not function as intended [Block] Navigation Affects the Navigation Block labels Mar 24, 2021
@talldan
Copy link
Contributor

talldan commented Mar 24, 2021

Oh no, this one was implemented as a feature 😄

Implemented in #21743.

@talldan
Copy link
Contributor

talldan commented Mar 24, 2021

The button block does the same thing, and I think the two are very comparable and should work similarly.

I suggest we need design feedback on this, as it seems more like a general editor behavior that should be made consistent across blocks.

@talldan talldan added the Needs Design Feedback Needs general design feedback. label Mar 24, 2021
@annezazu
Copy link
Contributor Author

Oh whoa interesting! I think if it were more reversible this would be less jarring :D

@talldan
Copy link
Contributor

talldan commented Mar 24, 2021

You can hit enter to split into two blocks again, but the original link url will be lost from the deletion of the block.

@annezazu
Copy link
Contributor Author

Yeah :) To me, that feels less reversible. Plus once you do that there's no indication that a link is missing.

@jasmussen
Copy link
Contributor

jasmussen commented Mar 24, 2021

Speaking to the "submenu disappears" aspect (circling the cursor in the video), that should be fixed in #29869.

Edit: and if not by that, then by #30169

@annezazu
Copy link
Contributor Author

annezazu commented Feb 7, 2022

Looping back on old issues and returned to this one seeing the same behavior in 5.9 (just to get a current snapshot):

delete.nav.mov

I'd be curious to get more design thoughts on this as, to me, this is a pretty surprising thing to have happen. I can see this was originally implemented as a way to allow for an easier time deleting links. However, you can't easily undo this anymore (unless you hit undo) like you could before by separating the menu items back out as shown above.

In my mind, nothing should happen upon taking this action.

@talldan talldan added the [Block] Buttons Affects the Buttons Block label Feb 7, 2022
@talldan
Copy link
Contributor

talldan commented Feb 7, 2022

However, you can't easily undo this anymore (unless you hit undo) like you could before by separating the menu items back out as shown above.

Yeah, that seems bad, seems like a regression of #18208.

@annezazu
Copy link
Contributor Author

Noting that this came up during the FSE Outreach program's usability testing with main menu items when deleting using 6.0 and Gutenberg 13.6:

menu.items.merging.mov

This felt like a broken experience to the participant.

@jordesign
Copy link
Contributor

Noting that is still present when testing in WP6.3

@annezazu
Copy link
Contributor Author

Still present in 6.7 RC1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Buttons Affects the Buttons Block [Block] Navigation Affects the Navigation Block Needs Design Feedback Needs general design feedback. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

4 participants