[wp-trac] [WordPress Trac] #13525: Improve menus UX: Remove "save menu item", swap "close" for "cancel", etc.

[WordPress Trac]

#13525: Improve menus UX: Remove "save menu item", swap "close" for "cancel", etc.
--------------------------+-------------------------------------------------
 Reporter:  koopersmith   |       Owner:  koopersmith
     Type:  defect (bug)  |      Status:  new        
 Priority:  normal        |   Milestone:  3.0        
Component:  Menus         |     Version:  3.0        
 Severity:  normal        |    Keywords:  needs-patch
--------------------------+-------------------------------------------------
Changes (by koopersmith):

  * keywords:  => needs-patch


Comment:

 Thanks for picking up the slack on this ticket, guys. Here are the issues
 I'm still seeing:

 To Do:

  * When a menu item is added, a draft of the item is created. If the menu
 is not saved, this item should be removed the next time the menu is
 edited. (Another possibility could be to remove draft items entirely.)

 For No-JS:

  * When a menu item is edited and then collapsed, its values are not
 copied into the now-hidden fields. This results in only being able to edit
 one item at once. These values are edited and unsaved, and should likely
 be POSTed as a result.

  * When a menu item position is changed it saves the new positions to the
 DB. This results in menu item positions being updated live, instead of on
 save.

  * The draft issue from above still pertains, but we also must ensure that
 draft menu items are not deleted while the user is mid-editing-session.
 One way of going about this could be to separate the generic 'edit' action
 from an 'editing' action, and remove draft items on the former.

  * When cancel is clicked, all editable values (except position) should be
 restored to their original values. This appears to work now, but may break
 as the above are fixed.

 I'll try and look into search issues if I have a minute, but I'm
 incredibly pressed for time.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13525#comment:23>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software