wp_nav_menu treats menu_class differently for fallback

[Nonchalant]

I've found references to this issue, but no actual solution or prior ticket. Issue:

When using wp_nav_menu, the output manner of menu_class is different depending if a fallback is being used or not.

Menu:

$wp_nav_header = array(
    'theme_location' => 'primaryheader',
    'menu'            => '',
    'container'       => false,
    'container_class' => '',
    'container_id'    => '',
    'menu_class'      => 'sf-menu',
    'menu_id'         => '',
    'fallback_cb'     => 'wp_page_menu',
    'before'          => '',
    'after'           => '',
    'link_before'     => '',
    'link_after'      => '',
    'depth'           => 0,);
wp_nav_menu( $wp_nav_header);

Output without Fallback (Correct output):

<ul id="menu-default" class="sf-menu"><li...

Output with Fallback (Incorrect output):

<div class="sf-menu"><ul><li...

So despite setting 'sf-menu' in menu_class, it creates a container (which I had explicitly disabled). It also does not populate the unordered list with the mentioned class.

Thank you,

Noel

[filosofo]

The <div> element is being generated by your fallback, in this case wp_page_menu. You can expect the markup to vary from non-fallback to the extent that the fallback differs, and wp_page_menu does not have an argument to remove the wrapping <div> element.

Some possible ways to address this:

• Use a fallback of your own creation that more closely resembles typical menu markup.
• Open a ticket to make wp_page_menu to print markup more like standard wp_nav_menu.

[Nonchalant]

Replying to filosofo:

The <div> element is being generated by your fallback, in this case wp_page_menu. You can expect the markup to vary from non-fallback to the extent that the fallback differs, and wp_page_menu does not have an argument to remove the wrapping <div> element.

Some possible ways to address this:

• Use a fallback of your own creation that more closely resembles typical menu markup.
• Open a ticket to make wp_page_menu to print markup more like standard wp_nav_menu.

Thank you for the explanation! I tried looking through the Codex too and hence was stumped by this. What I'll do is make a codex contribution once I've created a fallback of my own creation.

[filosofo]

Thanks for improving the Codex. I'm sure others will benefit from the explanation.

[SergeyBiryukov]

Related: #18842