Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#13338 closed defect (bug) (fixed)

Deleting a menu should land you on the next available menu instead of the + tab

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by: koopersmith's profile koopersmith
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords: has-patch
Focuses: Cc:

Description

In software editors that support tabs (or in browsers, for that matter), killing a tab results in your being navigated to the next available tab (the one to the right, if any, else the one to the left).

WP should not behave any differently when you delete menus.

Attachments (1)

13338.diff (1.4 KB) - added by koopersmith 14 years ago.

Download all attachments as: .zip

Change History (10)

#1 @koopersmith
14 years ago

  • Keywords ux-feedback added; ui ux removed
  • Severity changed from critical to normal

Agreed. Marking as ux-feedback, but patches welcome.

#2 @jane
14 years ago

Agreed.

#3 @ryan
14 years ago

(In [14601]) Go to first tab instead of + after deleting a menu. see #13338

#4 @ryan
14 years ago

Quick fix to go to the first tab, which is some improvement.

#5 @koopersmith
14 years ago

I can't remember which ticket this came up in, but do we want the tabs to be rearrangeable? I have an idea for a quick patch if they're not, but it'll break if they are.

#6 @koopersmith
14 years ago

  • Keywords ux-feedback removed

Given that we're implementing menu slots in #13378, I see no need to implement rearrangeable tabs for 3.0 (since users no longer have to rely on the first menu being the one automatically used by a theme).

#7 @koopersmith
14 years ago

  • Owner set to koopersmith
  • Status changed from new to accepted

@koopersmith
14 years ago

#8 @koopersmith
14 years ago

  • Keywords has-patch added

Patch selects the next tab (alphabetically, because they're sorted alphabetically). If the deleted menu was the last tab, it selects the last tab (ie the previous tab). If there are no more menus, it selects create menu. It also fixes the bug where no menu tab will be selected if the delete url is refreshed.

#9 @ryan
14 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

(In [14758]) Make the next available tab active after deleting a tab. Props koopersmith. fixes #13338

Note: See TracTickets for help on using tickets.