WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#24115 closed defect (bug) (fixed)

'Manage Locations' tab should be hidden for zero locations/no menus support

Reported by: DrewAPicture Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Menus Version: 3.6
Severity: normal Keywords: has-patch 3.6-menus
Cc: Focuses:

Description

If you're working with a theme that doesn't have any registered menu locations, the 'Manage Locations' tab should be hidden. Patch attached.

Attachments (2)

24115.diff​ (1.6 KB ) - added by DrewAPicture 13 years ago.
24115.2.diff​ (2.4 KB ) - added by DrewAPicture 13 years ago.

Download all attachments as: .zip

Change History (8)

#1 @DrewAPicture
13 years ago

24115.diff​ also fixes display of the 'This theme does not natively support menus' message based on number of theme locations rather than number of menus.

@DrewAPicture
13 years ago

#2 @DrewAPicture
13 years ago

It's worth mentioning that if we don't hide the tab for this case, they'll visit the tab, it'll say 'Your theme supports 0 menus ...' and the list table will be empty, which is just bad UX.

Last edited 13 years ago by DrewAPicture (previous) (diff)

#3 @DrewAPicture
13 years ago

  • Summary 'Manage Locations' tab should be hidden for zero menus and/or zero locations → 'Manage Locations' tab should be hidden for zero locations/no menus support

#4 @SergeyBiryukov
13 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 24217:

Don't show "Manage Locations" tab if the theme has no menu locations defined. props DrewAPicture. fixes #24115.

#5 @DrewAPicture
13 years ago

24115.2.diff​ brings the no-support notice text more in line with the help tab, with the Widgets screen linked:

"Your theme does not natively support menus, but you can use them in sidebars by adding a Custom Menus widget on the ​Widgets screen."

@DrewAPicture
13 years ago

#6 @SergeyBiryukov
13 years ago

In 24218:

Make the notice on Edit Menus screen more in line with the help text. props DrewAPicture. see #24115.

Note: See TracTickets for help on using tickets.