WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#13330 closed feature request (fixed)

Is the AYS? really needed for menu items?

Reported by: Denis-de-Bernardy Owned by: nacin
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords: ux-feedback
Focuses: Cc:

Description

The AYS? behavior when menu items are removed is inconsistent with that of, say, widgets.

Change History (9)

comment:1 @filosofo5 years ago

Almost everything else about the UI behavior differs, too. Why does this particular difference matter?

comment:2 @koopersmith5 years ago

  • Keywords ux-feedback added; ui ux removed

comment:3 @Denis-de-Bernardy5 years ago

Well, it seems to me that there's little need to enforce it when adding/removing menu items. They can readily be re-added if necessary. Plus, picture ourselves keeping the AYS while enabling drag/drop of menu items to add/remove them. Clearly it makes no sense in that case.

comment:4 @jane5 years ago

Widgets don't get removed and lost, as of 2.8. Widget removal used to have an AYS, but when we created the "Inactive Widgets" area, it rendered AYS unnecessary, since nothing was lost by removing a widget. Since menu items are lost on removal, that's why there's an AYS, just like on screens where we have delete instead of trash (excluding actual Trash screens).

comment:5 @jane5 years ago

  • Type changed from defect (bug) to feature request

comment:6 @nacin5 years ago

I would tend to agree we don't need an AYS for menu items.

That said, widgets have a Delete link only pixels away from the Close link. The Delete link has no AYS and deletes the instance entirely -- it is not moved to inactive widgets. Maybe the best fix for that, for now, would be to hide the close link if not JS.

comment:7 @nacin5 years ago

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

comment:8 @ryan5 years ago

I think it was decided to remove this menu item AYS and use just the menu level AYS.

comment:9 @nacin5 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed
Note: See TracTickets for help on using tickets.