[wp-trac] [WordPress Trac] #13669: Nav menu parameters not working

WordPress Trac wp-trac at lists.automattic.com
Tue Jun 1 19:44:05 UTC 2010


#13669: Nav menu parameters not working
--------------------------+-------------------------------------------------
 Reporter:  vteixeira     |        Owner:          
     Type:  defect (bug)  |       Status:  reopened
 Priority:  normal        |    Milestone:  3.0     
Component:  Menus         |      Version:  3.0     
 Severity:  normal        |   Resolution:          
 Keywords:  has-patch     |  
--------------------------+-------------------------------------------------
Changes (by nacin):

  * keywords:  needs-patch => has-patch


Comment:

 I think I'd rather kill all container code and just let the theme choose
 to include a container around their menu call. The only complexity that
 introduces is if the user declines to assign a menu to that location,
 which seems like a job for has_nav_menu().

 IMO, unless it's a <nav> element purely for semantics, there's almost
 never a reason for an extra container div. I consider it cruft.

 Nonetheless, has-patch.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13669#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list