Make WordPress Core

Opened 10 years ago

Closed 15 months ago

Last modified 14 months ago

#39352 closed defect (bug) (invalid)

Customize: menu widgets have two edit shortcuts

Reported by: sirbrillig Owned by:
Priority: normal Milestone:
Component: Customize Version: 4.7
Severity: normal Keywords: close
Cc: Focuses:

Description

r39622 fixes the edit shortcut for menus inside widgets, but that still leaves two edit shortcuts for those partials, generally with one on top of the other. Right now the menu shortcut is on top of the widget shortcut, so I was thinking it might be worth while to hide one, but I wonder which one is the better experience for users?

Clicking on the menu shortcut brings up the editor for that menu.

https://cldup.com/7lTAzaHguo.png

Clicking on the widget shortcut brings up the widget settings, which allow changing which menu is displayed there.

https://cldup.com/mWvyLgbTc1.png

It seems to me that both could be what a user might be looking for, but I think we can really only have one. Since you can click the "Edit Menu" button to get from the widget settings to the menu settings and not the other way around, I suggest we hide the menu shortcut, but I'm open to a dissenting opinion.

cc: @folletto and @melchoyce for a UX perspective

Split from #39101

Attachments (3)

menu-widget-menu-control.png (50.2 KB ) - added by sirbrillig 10 years ago.
Menu widget menu control
menu-widget-widget-control.png (41.9 KB ) - added by sirbrillig 10 years ago.
Menu widget widget control
39352.diff (866 bytes ) - added by sirbrillig 10 years ago.
Prevent adding edit shortcut for nav menus inside widgets

Download all attachments as: .zip

Change History (9)

@sirbrillig
10 years ago

Menu widget menu control

@sirbrillig
10 years ago

Menu widget widget control

#1 @sirbrillig
10 years ago

  • Summary Customize: menu widget have two edit shortcutsCustomize: menu widgets have two edit shortcuts

#2 @folletto
10 years ago

Since you can click the "Edit Menu" button to get from the widget settings to the menu settings and not the other way around, I suggest we hide the menu shortcut, but I'm open to a dissenting opinion.

I'd wait for Mel's review too, but my opinion is that a better solution would require more design and rework, so this is probably the best approach we have right now for the reason you have outlined.
It's also slightly less incongruent having all the "widget" items to have the same kind of interaction to start with.

tl;dr: hide the menu shortcut.

#3 @sirbrillig
10 years ago

tl;dr: hide the menu shortcut.

I'll attach the patch that does that in case we decide to go that route.

@sirbrillig
10 years ago

Prevent adding edit shortcut for nav menus inside widgets

#4 @celloexpressions
5 years ago

  • Keywords needs-design-feedback added

I'm not personally bothered by the double icons since they're both visible in a nested way... but I know exactly why there are two and what each does. It would be good to get feedback from users that may be less familiar with what's actually happening here.

I also wonder navigation blocks might replace most instances of the menu widget moving forward.

#5 @callumbw95
15 months ago

  • Keywords close added; needs-design-feedback removed
  • Resolutioninvalid
  • Status newclosed

Hi all,
As the way widgets work within WordPress have changed in favour of Gutenberg blocks, the UI/UX of this issue has changed now. As of such the issues raised in this ticket are no longer relevant to the way WordPress handles menus Appearance Customizer. Because of this I believe this ticket can be safely closed.

#6 @peterwilsoncc
14 months ago

  • Milestone Awaiting Review
Note: See TracTickets for help on using tickets.