[wp-trac] [WordPress Trac] #13615: Confusing behavior adding items to nonexistent menu
WordPress Trac
wp-trac at lists.automattic.com
Sat May 29 13:31:43 UTC 2010
#13615: Confusing behavior adding items to nonexistent menu
--------------------------+-------------------------------------------------
Reporter: filosofo | Owner: markjaquith
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 3.0
Component: Menus | Version: 3.0
Severity: normal | Keywords: has-patch
--------------------------+-------------------------------------------------
Comment(by filosofo):
I think your reasoning about the UX is sound and agree that it's a better
strategy.
As an aside, I'm really curious why you'd change the line in my patch from
`if ( document.getElementById('menu-to-edit') )`
to
`if ( 0 != $('#menu-to-edit').length )`
Would you please explain?
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13615#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list