Make WordPress Core

Opened 2 years ago

Last modified 3 weeks ago

#61557 new defect (bug)

When deleting menu items in batches, it is easy to mistakenly click on the delete menu action.

Reported by: iwillhappy1314 Owned by:
Priority: normal Milestone: Awaiting Review
Component: Menus Version: 6.5.5
Severity: normal Keywords: needs-design-feedback has-screenshots has-patch
Cc: Focuses: ui

Description

In the batch delete menu items interface: If there are many menu items, the "Batch Delete" operation link might be obscured by the menu operation toolbar at the bottom, making the "Delete Menu" link more prominent. Although a confirmation prompt appears when clicking this link, accidentally ignoring it could result in the entire menu being mistakenly deleted.

Suggested Solution: I think moving the Delete Entire Menu link in the toolbar to the right will reduce the chances of this error happening.

https://github.com/WordPress/gutenberg/assets/1455683/dcc10098-1236-4e25-8cc5-7089a05afd69

Attachments (1)

2026-07-18 08.29.55.gif (202.0 KB ) - added by maxpertici 3 weeks ago.
Disable nav menu footer sticky position during bulk selection

Download all attachments as: .zip

Change History (9)

#1 @sabernhardt
2 years ago

  • Component GeneralMenus

This ticket was mentioned in PR #12430 on WordPress/wordpress-develop by @maxpertici.


5 weeks ago
#2

  • Keywords has-patch added

styles(admin-nav-menus): space out action buttons while preserving "Delete"

https://github.com/user-attachments/assets/0ec54305-3dc5-49d3-aef5-3aef22ab24e7

#3 @maxpertici
5 weeks ago

Hi,

I just opened a PR to fix this a small issue.

Having made this accidental deletion mistake multiple times myself, I totally agree that moving this button is a great idea.

To prevent this, I've moved the "Delete Entire Menu" link over to the right side of the toolbar.

Version 0, edited 5 weeks ago by maxpertici (next)

#4 @maxpertici
5 weeks ago

  • Keywords has-screenshots added

#5 @joedolson
4 weeks ago

For accessibility reasons, related buttons should be kept in close proximity to each other, and should be proximity to the controls that the are associated with. These were moved in #56594.

Feel free to make suggestions other than spreading the two buttons apart, but that particular change isn't going to happen.

#6 @sabernhardt
4 weeks ago

  • Keywords needs-design has-patch removed

Another possibility is to reconsider the sticky position added in 5.7 (#51631).

#7 @maxpertici
3 weeks ago

Could removing the sticky element while making a multiple selection be a solution? It would cause an interface shift, but maybe it could prevent errors? When using multiple selection to confirm a deletion, you have to scroll to the button at the end of the list anyway.

I think I can propose something for this.
Could this be an issue from an accessibility standpoint?

@maxpertici
3 weeks ago

Disable nav menu footer sticky position during bulk selection

Note: See TracTickets for help on using tickets.