Changeset 14182

Timestamp:

04/21/2010 09:32:08 PM (14 years ago)

Author:

westi

Message:

Allow a theme to set an id on the container object for the wp_nav_menu output. Fixes #13057 props thee17.

File:

• trunk/wp-includes/nav-menu-template.php (modified) (3 diffs)

trunk/wp-includes/nav-menu-template.php

@@ -18 +18 @@
  * format - Whether to format the ul. Defaults to 'div'.
  * fallback_cb - If the menu doesn't exists, a callback function will fire. Defaults to 'wp_page_menu'.
+ * container - Type of container tag. Avalible options div, p, or nav. Defaults to 'div'. 
+ * container_class - Chooses a class for the container.
+ * container_id - Chooses an id for the container.
  * before - Text before the link text.
  * after - Text after the link text.
@@ -31 +34 @@
  */
 function wp_nav_menu( $args = array() ) {
-    $defaults = array( 'menu' => '', 'container' => 'div', 'container_class' => '', 'menu_class' => 'menu', 'echo' => true,
+    $defaults = array( 'menu' => '', 'container' => 'div', 'container_class' => '', 'container_id' => '', 'menu_class' => 'menu', 'echo' => true,
     'fallback_cb' => 'wp_page_menu', 'before' => '', 'after' => '', 'link_before' => '', 'link_after' => '',
     'depth' => 0, 'walker' => '', 'context' => 'frontend' );
@@ -71 +74 @@
     if ( in_array( $args->container, $container_allowedtags ) ) {
         $class = $args->container_class ? ' class="' . esc_attr($args->container_class) . '"' : ' class="menu-'. $menu->slug .'-container"';
-        $nav_menu .= '<'. $args->container . $class .'>';
+        $container_id = $args->container_id ? ' id="' . esc_attr($args->container_id) . '"' : '' ;
+        $nav_menu .= '<'. $args->container . $class . $container_id .'>';
     }