[wp-trac] [WordPress Trac] #12844: wp_get_nav_menu UL ID

WordPress Trac wp-trac at lists.automattic.com
Sun Apr 4 14:14:17 UTC 2010


#12844: wp_get_nav_menu UL ID
-------------------------+--------------------------------------------------
 Reporter:  tosak        |       Owner:  ryan                         
     Type:  enhancement  |      Status:  new                          
 Priority:  low          |   Milestone:  3.0                          
Component:  Menus        |     Version:  3.0                          
 Severity:  minor        |    Keywords:  wp_get_nav_menu, id has-patch
-------------------------+--------------------------------------------------
Changes (by nacin):

  * milestone:  Unassigned => 3.0


Comment:

 I spoke with ptahdunbar and we're going to probably combine
 wp_get_nav_menu() and wp_nav_menu(). The latter does allow you to omit the
 div, and it then passes its arguments to wp_get_nav_menu(), which then
 spits out the ul with classes.

 I see it as working like this: optional container element (not just div,
 as many will want to use the new HTML5 nav element), along with an
 optional ID. The id is applied to the container, or if no container, then
 the id is applied to the ul. We'll probably need separate container and ul
 classes though.

 We also need to make sure we're properly escaping attributes.

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


More information about the wp-trac mailing list