Add a Manage in Customizer link to the menus admin

[celloexpressions]

To match what we have for widgets, for discover-ability.

This was patched on the tracking ticket, but splitting out here so it doesn't get missed.

[SergeyBiryukov]

In 32985:

Menus: Add a Manage in Customizer link, like we did for Widgets screen in [30459].

props celloexpressions.
fixes #32808.

[swissspidy]

The link points to the wrong panel so it doesn't focus on the nav menu panel.

[swissspidy]

Fix panel parameter

[rabmalin]

#32836 was marked as a duplicate.

[obenland]

In 33004:

Use correct panel id for menus when linking to the customizer.

Props swissspidy.
Fixes #32808.

[ryan]

This is one reason why I want desktop and mobile screenshots to be commit requirements.

[obenland]

Patch makes it look like another tab on mobile.

We could just prevent it from wrapping but that doesn't really look very elegant.
@helen, what do you think?

[obenland]

Just with preventing wrapping.

[obenland]

Per discussion with @hugobaeta in Slack (see above), we'll go for the small solution.

[obenland]

In 33040:

Prevent link to customizer from wrpping on small screens.

Fixes #32808.

[jeremyfelt]

Is there a way we can break this out above the Edit Menus and Manage Locations tabs on mobile? It still looks very broken in its current state. I'm hoping to translate this pattern to a patch on #32525 as well.

[ryan]

iPhone 6+

[celloexpressions]

Manage in Customizer link on custom header (and background) admin page.

[celloexpressions]

Current status here looks like we're trying to figure out a better styling for these links, but without much success so far. I'd like this to be a persistent link, because users may land on these screens and mean to go to the Customizer instead, so a dismiss-able notice probably won't work. A more subtle link that initially has a dismiss-able feature pointer could work though.

I added a screenshot of the custom header admin page. It, and the custom background page, still exist, primarily for no-js users, but they aren't linked anywhere in core anymore for users that can access the Customizer. On those screens, we have the info-style notice with a descriptive sentence that links to the Customizer version. I think we went with the alternate styling for widgets to be less invasive, but I do think that this is probably the most appropriate way to include a link to the Customizer version on the admin screens, especially promoting the ability to live-preview changes.

See #28032 for more history.

Also just got another idea - what if when the Customize admin bar link is present in the admin, when on this screen we deep-link it to menus? Make it contextual to the current admin context when possible, like it's contextual to the current front-end context.

[obenland]

@jeremyfelt, any news here? Should we tackle that as an enhancement in 4.4 maybe?

[jeremyfelt]

Replying to obenland:

@jeremyfelt, any news here? Should we tackle that as an enhancement in 4.4 maybe?

There was some ​discussion in #design a bit ago, but I don't think any consensus. #32525 went a different route with a couple text links rather than a button.

I'd prefer leaving this open until a design decision is made. If we do need to postpone it to 4.4, we should hide "Manage in Customizer" on mobile as it looks really out of place.

I'll bring it up in #core-customizer and #design tomorrow and see if there are new thoughts.

[obenland]

@hugobaeta, could you weigh in on this as well?

[obenland]

Hiding it on mobile would work for me too.

[jeremyfelt]

32808.3.diff adds an <h1>Menus... to match the other admin screens and then places "Manage in Customizer" alongside it. This follows the pattern from Widgets, but still feels really weird. It does translate to mobile though.

The H1 for this page is then "Menus Manage in Customizer". The H1 for Widgets is "Widgets Manage in Customizer".

[jeremyfelt]

Large view after pending commit of 32808.3.diff

[jeremyfelt]

Mobile view after pending commit of 32808.3.diff

[jeremyfelt]

In 33355:

Nav Menus: Add a Menus title with Manage in Customizer button to nav-menus.php

This aligns the <h1> on wp-admin/nav-menus.php with that of other admin screens and aligns the "Manage in Customizer" button with that on wp-admin/widgets.php.

Fixes #32808.

[jeremyfelt]

See #26601 as a ticket for future improvements to <h1> titles on these admin pages.