[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