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

When using the same classic menu at more than 1 instance on the FSE, the menu duplicates itself #38034

Closed
audrasjb opened this issue Jan 17, 2022 · 4 comments
Labels
[Block] Navigation Affects the Navigation Block [Type] Bug An existing feature does not function as intended

Comments

@audrasjb
Copy link
Contributor

Description

Note: this issue was originally report on Trac. See https://core.trac.wordpress.org/ticket/54792.

When using the same classic menu at more than 1 instance on the FSE, the menu duplicates itself

Step-by-step reproduction instructions

  1. Create a classic menu before activating block theme
  2. Add a Nav Menu Block -> select the classic menu
  3. Add a new Nav Menu Block -> select the same classic menu selected above. (Repeat the same step however times you need)
  4. Go to Manage Menu. You'll find the same classic menu duplicated.

The same happens every time 'start empty' is used in Nav Menu.

Screenshots, screen recording, code snippet

xuJp8Kn

Environment info

WordPress 5.9 RC 2 without Gutenberg plugin installed.

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@audrasjb audrasjb added [Block] Navigation Affects the Navigation Block Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta labels Jan 17, 2022
@annezazu
Copy link
Contributor

I can replicate this! Going to flag in the navigation block feature channel for good measure.

@annezazu annezazu added the [Type] Bug An existing feature does not function as intended label Jan 17, 2022
@noisysocks noisysocks removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Jan 17, 2022
@talldan
Copy link
Contributor

talldan commented Jan 18, 2022

When a classic menu is selected, it's copied to the new navigation block format and the original menu is kept in place. The same will also happen a second time. I think this is probably mostly a case of unclear UI, but there has also been discussion about how to better handle menus in #36307 and #36311.

@annezazu
Copy link
Contributor

Thanks for connecting the dots. The fact that every time I select the classic menu (no matter how many times), it recreates the new block format resulting in a new menu seems like something to resolve before 5.9. I imagine many folks will want to first use their classic menu and might continue to select it out of habit/lack of knowledge.

In any case, +1 on unclear UI.

@talldan talldan added Needs Design Needs design efforts. and removed Needs Design Needs design efforts. labels Jan 18, 2022
@talldan
Copy link
Contributor

talldan commented Jan 18, 2022

I definitely see that.

It's now too late to include anything in 5.9, the final backports from Gutenberg have already been made. So 5.9.1 is the best we can do.

Some options have been outlined in #36311. I'm not sure there's value in keeping two very similar issues open, so I think it's worth closing this as a duplicate of #36307 and I'll add some more comments over there.

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

No branches or pull requests

4 participants