Changeset 34330

Timestamp:

09/19/2015 06:06:49 PM (9 years ago)

Author:

wonderboymusic

Message:

Add a 'menu_id' argument to wp_page_menu() so that fallback menus have the same attributes as wp_nav_menu().

Props lancewillett, ocean90, wonderboymusic.
Fixes #31656.

File:

• trunk/src/wp-includes/post-template.php (modified) (4 diffs)

trunk/src/wp-includes/post-template.php

@@ -1188 +1188 @@
  *
  * @since 2.7.0
- * @since 4.4.0 Added `container`, `before`, `after`, and `walker` arguments.
+ * @since 4.4.0 Added `menu_id`, `container`, `before`, `after`, and `walker` arguments.
  *
  * @param array|string $args {
@@ -1195 +1195 @@
  *     @type string          $sort_column How to short the list of pages. Accepts post column names.
  *                                        Default 'menu_order, post_title'.
+ *     @type string          $menu_id     ID for the div containing the page list. Default is empty string.
  *     @type string          $menu_class  Class to use for the element containing the page list. Default 'menu'.
  *     @type string          $container   Element to use for the element containing the page list. Default 'div'.
@@ -1212 +1213 @@
     $defaults = array(
         'sort_column' => 'menu_order, post_title',
+        'menu_id'     => '',
         'menu_class'  => 'menu',
         'container'   => 'div',
@@ -1268 +1270 @@
     }
     $container = sanitize_text_field( $args['container'] );
-    $menu = "<{$container} class=\"" . esc_attr( $args['menu_class'] ) . '">' . $menu . "</{$container}>\n";
+    $attrs = '';
+    if ( ! empty( $args['menu_id'] ) ) {
+        $attrs .= ' id="' . esc_attr( $args['menu_id'] ) . '"';
+    }
+
+    if ( ! empty( $args['menu_class'] ) ) {
+        $attrs .= ' class="' . esc_attr( $args['menu_class'] ) . '"';
+    }
+
+    $menu = "<{$container}{$attrs}>" . $menu . "</{$container}>\n";
 
     /**