[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