Add an "add new menu" button to the menu locations section in the customizer

[celloexpressions]

When there are no menus, or even if there are menus but a new menu needs to be created to add to a location, the flow from being on the locations section and realizing that a new menu should be created to go back and then add a new menu is awkward. Let's add the ability to add a new menu from the menu locations section directly.

A button that focusing the new menu name control is likely the best approach, although this would still animate back out of the locations section, it still gives the option easier access and better flow.

[westonruter]

Would the “Add Menu” button appear next to each location so that upon creating the new menu it would be assigned to that location? Or should there be just one button appearing after all of the location dropdowns for creating a new menu? Or should the nav menu locations section just be hidden until a nav menu is created? This gets into some of the UX problems with how menus are managed and how it's not intuitive for novices how theme locations relate to nav menus.

[celloexpressions]

I would think one button at the end. That being said, it would probably be a good idea to also hide the section when there are no menus, since that places more visual emphasis on the add menu button in the first place. I see value in having an add menu button in the section even when there are existing menus, though, since you could easily realize there's a need for a new menu when looking at the options in this section (say you just switched to a theme with a social menu location, for example, and wanted to set one up).

[Travel_girl]

Mock-up Customizer

[Travel_girl]

Mock-up Customizer

[Travel_girl]

For the Customer I have follow suggestion:

https://core.trac.wordpress.org/attachment/ticket/36279/Mokup_customizer.jpg

The link should go to the "Add Menu" Option in the Customizer. When people add a menu, it would be good, if there go back to Menu Location Setting in the Customzier.

I also talked to the accessibility-team and we agreed on making the links underline. For a better consistency and accessibility, it would be also good, to make also the "use new menu" in the Admin-Dashboard also underlined.

Do we open a new ticket for the other "use new menu".
We should also update the version for this ticket.

[celloexpressions]

Note that using a button for each section would conflict with #36795 and cause things to start to get fairly cluttered. That's why I think a single button would probably be better, since that could then be added to any location.

[Travel_girl]

Edit + Add New

[Travel_girl]

The Edit Button would not change the problem, that you can´t add a new Menu.

What about to add the "Add new Button" as well? https://core.trac.wordpress.org/attachment/ticket/36279/Slice%201.png

It could be a bit cluttered, but I think the functionality to add a new menu is a bit more needed on this place, than editing the menu, because you can´t see the details of the menu, but you can see the existing menus.

[karmatosed]

I'm not convinced about the need for edit and new in this case. I feel that's veered a little from the original intention too of this ticket. I think the original argument for a new button is solid but not convinced about edit and feel that gets super noisy fast in this section visually.

[celloexpressions]

Yeah, I think with the edit links, if we add add-menu UI to this section it should probably be as a single button at the bottom, similar to what's shown in the menus panel. The user would still need to assign the menu to a location (or multiple locations) after adding it, so repetition of an add-menu button would have less functional benefit than the repeated edit menu buttons.

[Travel_girl]

Right now the "Edit" link is visible.

@karmatosed do you suggest, that we add the "add new" Button and remove the edit Button?

Because I think, that would be the better way.

[Travel_girl]

Current view

[melchoyce]

#42171 was marked as a duplicate.

[Presskopp]

It's already there!

In class-wp-customize-nav-menu-location-control.php, L.76 it gets the 'hidden' class:

if ( ! $this->value() ) { echo ' hidden'; }

If you delete this, the "Edit Menu" appears.

But that's where I'm on my wit's end

[bpayton]

Initial work for this ticket is here:
​https://github.com/xwp/wordpress-develop/pull/285

Before wrapping up, I need to:

• Add unit tests
• Confirm what we want to do with the existing Edit Menu button. @melchoyce, what are your thoughts?

[melchoyce]

Replying to bpayton:

Before wrapping up, I need to:

• Confirm what we want to do with the existing Edit Menu button. @melchoyce, what are your thoughts?

What's the issue? AFAIK it should stay the same.

[bpayton]

Replying to melchoyce:

Replying to bpayton:

Before wrapping up, I need to:

• Confirm what we want to do with the existing Edit Menu button. @melchoyce, what are your thoughts?

What's the issue? AFAIK it should stay the same.

Based on my reading of the discussion, I thought editing menus from All Locations was being reconsidered, but we cleared up my misunderstanding in #core-customize.

Because the client-side api.Menus.MenuLocationsControl relies on both server-side and client-side rendering, I am not sure it is reasonable to add qunit tests. @westonruter, is there anything you'd suggest here?

Not including unit tests, this change is ready for review here:
​https://github.com/xwp/wordpress-develop/pull/285

[westonruter]

This tests well for me. @melchoyce @celloexpressions is the functionality in line with your expectations? I've deployed to ​http://trac-39896-xwp-testbed.pantheonsite.io/wp-admin/customize.php

[melchoyce]

Replying to westonruter:

This tests well for me. @melchoyce @celloexpressions is the functionality in line with your expectations? I've deployed to ​http://trac-39896-xwp-testbed.pantheonsite.io/wp-admin/customize.php

Yeah, works great. Let's get this in.

[obenland]

In 41899:

Customize: Allow menu creation in locations pane

Adds a link to the menu creation workflow from the locations selector in the nav menu control.

Props bpayton, westonruter, Travel_girl, melchoyce, celloexpressions.
Fixes #36279.