[wp-trac] [WordPress Trac] #31218: nav-menu.js menu item added event

WordPress Trac noreply at wordpress.org
Fri Dec 9 19:27:09 UTC 2016


#31218: nav-menu.js menu item added event
------------------------------------+-----------------------------
 Reporter:  welcher                 |       Owner:  chriscct7
     Type:  enhancement             |      Status:  reviewing
 Priority:  normal                  |   Milestone:  Future Release
Component:  Menus                   |     Version:  4.2
 Severity:  trivial                 |  Resolution:
 Keywords:  has-patch next-release  |     Focuses:  javascript
------------------------------------+-----------------------------
Changes (by adamsilverstein):

 * keywords:  has-patch dev-feedback next-release => has-patch next-release


Comment:

 @welcher in [attachment:31218.5.diff] I updated the patch with another
 event right before items are removed from the menu, again passing a jQuery
 object handle to the menu DOM element itself. @jorbin pointed out this
 might be useful and I agree.

 I took the brackets out of the data passed in your trigger calls as it was
 passing the element as a single element array.

 Can you review and see if this still works for your use case?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31218#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list