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

WordPress Trac noreply at wordpress.org
Tue Feb 3 04:25:54 UTC 2015


#31218: nav-menu.js menu item added event
-------------------------+------------------------------
 Reporter:  welcher      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Menus        |     Version:  trunk
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:  javascript
-------------------------+------------------------------

Comment (by adamsilverstein):

 Hey Ryan!

 Thanks for the patch. Interesting idea for a plugin, the menu system
 certainly gets unwieldy when long (or always :)).

 I think triggering an event on document is perfectly reasonable, although
 if you are going to reuse the $(menuMarkup) selector it should be cached.
 I also see this as another argument for standardizing
 [https://core.trac.wordpress.org/ticket/21170 Actions and Filters in
 JavaScript] in core; until that bolder budges, triggering on document is
 the way to go.

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


More information about the wp-trac mailing list