Format wp_list_pages and wp_page_menu without <li> tags

[zialingua]

Would that be possible to have the functions wp_list_pages and wp_page_menu with an option to display pages in a "flat" format like the functions wp_list_bookmarks or wp_tag_cloud or wp_get_archives?

[stevegrunwell]

Adds 'format', 'before', and 'after' arguments to wp_list_pages()

[stevegrunwell]

These have limited use (especially when 'title_li' is present), but with the patch we can:

1. Flatten the output (without re-ordering)

   <ul>
     <?php
       $args = array(
         'format' => 'custom',
         'before' => '<li class="%s">',
         'after' => '</li>',
         'title_li' => null
       );
       wp_list_pages( $args );
     ?>
   </ul>
   
   <!-- produces -->
   <ul>
     <li class="page_item page-item-1"><a href="http://src.wordpress-develop.dev?p=1">Parent Page</a></li>
     <li class="page_item page-item-3"><a href="http://src.wordpress-develop.dev?p=3">Child Page</a></li>
     <li class="page_item page-item-2"><a href="http://src.wordpress-develop.dev?p=2">Second Parent</a></li>
    </ul>
   

2. Wrap links in other elements

   <?php
     $args = array(
       'format' => 'custom',
       'before' => '<span class="%s">',
       'after' => '</span>',
       'title_li' => null
     );
     wp_list_pages( $args );
   ?>
   
   <!-- produces -->
   <span class="page_item page-item-1"><a href="http://src.wordpress-develop.dev?p=1">Parent Page</a></span>
   <span class="page_item page-item-3"><a href="http://src.wordpress-develop.dev?p=3">Child Page</a></span>
   <span class="page_item page-item-2"><a href="http://src.wordpress-develop.dev?p=2">Second Parent</a></span>
   

2. Add icons, entities, or whatever other items (that should most likely be CSS pseudo-elements):

   <?php
     $args = array(
       'format' => 'custom',
       'before' => '*&nbsp;',
       'title_li' => null
     );
     wp_list_pages( $args );
   ?>
   
   <!-- produces -->
   *&nbsp;<a href="http://src.wordpress-develop.dev?p=1">Parent Page</a>
   *&nbsp;<a href="http://src.wordpress-develop.dev?p=3">Child Page</a>
   *&nbsp;<a href="http://src.wordpress-develop.dev?p=2">Second Parent</a>
   

Patch also adds tests for wp_page_menu().

[wonderboymusic]

Seems like wp_page_menu() needs to be able to accept 'walker' and remove some of the hardcoded HTML bits. wp_list_pages() already accepts 'walker'

[wonderboymusic]

In 34200:

Allow wp_page_menu() to accept 'walker', 'before', and 'after' to allow custom markup.

Fixes #11095.

[DrewAPicture]

Documentation for the three new arguments needs to be added to the hash notation for $args. Also, a changelog entry, please :-)

[stevegrunwell]

Adds inline docs to code committed by @wonderboymusic in 34200

[SergeyBiryukov]

In 34243:

Docs: Update the DocBlock for wp_page_menu() to include 'before', 'after', and 'walker' arguments added in [34200].

Align the $defaults array and include the default value for 'show_home'.

Props stevegrunwell.
Fixes #11095.

[wonderboymusic]

In 34653:

After [34200], force 'before' and 'after' to <ul> and </ul> when wp_nav_menu() falls back to wp_page_menu().

See #11095.

[DrewAPicture]

In 34654:

Tests: Add a test for the 'before' and 'after' arguments in wp_page_menu() when used as a fallback for wp_nav_menu().

When wp_page_menu() is used as a fallback for wp_nav_menu(), the before and after arguments should be set and output as <ul> and </ul>, respectively.

See #11095. See [34653].

[obenland]

[34243] broke back compat for themes adding a home link by checking ! isset( $args['show_home'] ), including Twenty Ten and Twenty Eleven.

I think we should leave the docs but remove the actual declaration in the default array for show_home.

[obenland]

In 35737:

Template: Defining a default value for show_home breaks back compat.

To add a home link to the fallback menu output many themes only check if that
argument is set. Including Twenty Ten and Twenty Eleven. They check with
isset() so child themes and other instances using wp_page_menu() have a
chance to disable the home link by setting it to false.

Fixes #11095.