WordPress.org

Make WordPress Core

Opened 7 years ago

Last modified 2 months ago

#13910 reviewing feature request

Get Menu name with wp_nav_menu()

Reported by: jowo Owned by: chriscct7
Milestone: 4.9 Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords: has-patch has-unit-tests commit
Focuses: Cc:

Description

There is no way to get the actual "Menu name" in wp_nav_menu()

For example if you want to create a left sidebar menu you want a header to go with it. In previous versions you could do use wp_list_pages() with "title_li".

With wp_nav_menu() you have to hard code <h3>Static menu name</h3> into the template before calling the function.

If you could get the "Menu name" as defined in backend interface from the wp_nav_menu() it would create the menu title automatically.

Attachments (5)

13910.diff (1.2 KB) - added by wojtek.szkutnik 7 years ago.
13910-2.diff (1.6 KB) - added by christophherr 17 months ago.
Turns markoheijnen's comment into a patch. Adds filter for the title of a navigation menu.
13910.tests.patch (892 bytes) - added by SpencerFinnell 10 months ago.
Add tests for wp_nav_menu_title()
13910.patch (1.1 KB) - added by sagarprajapati 4 months ago.
13910.2.patch (1.1 KB) - added by welcher 3 months ago.
Combined patch

Download all attachments as: .zip

Change History (27)

#1 @ocean90
7 years ago

  • Milestone changed from 3.0 to Future Release

#2 @pavelevap
7 years ago

  • Cc pavelevap@… added

#3 @wojtek.szkutnik
7 years ago

  • Cc wojtek.szkutnik@… added
  • Keywords has-patch needs-testing added; wp_nav_menu menu name removed

#4 @wojtek.szkutnik
7 years ago

  • Keywords gsoc added

#5 @dangayle
6 years ago

+1 I've needed this a few times, having to resort to a similar solution to http://www.wptavern.com/forum/themes-templates/1563-list-available-wp_nav_menus.html

#6 @markoheijnen
5 years ago

  • Cc marko@… added
  • Keywords dev-feedback added; gsoc removed

The patch looks wrong. title shouldn't be an argument and it should have been $menu->name.

Also a function like this should be in core

<?
function wp_nav_menu_title( $theme_location ) {
	$title = '';
	if ( $theme_location && ( $locations = get_nav_menu_locations() ) && isset( $locations[ $theme_location ] ) ) {
		$menu = wp_get_nav_menu_object( $locations[ $theme_location ] );
			
		if( $menu && $menu->name ) {
			$title = $menu->name;
		}
	}

	return apply_filters( 'wp_nav_menu_title', $title, $theme_location );
}
?>

#7 @DrewAPicture
5 years ago

  • Cc xoodrew@… added

#8 @chriscct7
2 years ago

There was about 10 comments from the same user that were moderated as spam (which I confirmed they were). Reviewed and deleted them.

This ticket was mentioned in Slack in #core by chriscct7. View the logs.


21 months ago

@christophherr
17 months ago

Turns markoheijnen's comment into a patch. Adds filter for the title of a navigation menu.

#10 @christophherr
17 months ago

  • Keywords needs-unit-tests added

Following what chriscct7 said in Slack, 13910-2.diff turns markoheijnen's comment into a patch.

@SpencerFinnell
10 months ago

Add tests for wp_nav_menu_title()

#11 @SpencerFinnell
10 months ago

  • Keywords has-unit-tests 4.8-early added; needs-testing needs-unit-tests removed

#12 @chriscct7
10 months ago

  • Keywords 4.8-early removed
  • Milestone changed from Future Release to 4.8

#13 @chriscct7
10 months ago

  • Owner set to chriscct7
  • Status changed from new to reviewing

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


5 months ago

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


5 months ago

#16 @jbpaul17
5 months ago

  • Milestone changed from 4.8 to 4.8.1

Punting to 4.8.1 per today's bug scrub.

#17 @sagarprajapati
4 months ago

  • Keywords dev-feedback removed

Hi All,

I have checked the 13910-2.diff patch and it's working as expected. I have updated @since as 4.8.1 in refreshed patch because its milestone is changed to 4.8.1

Thanks

This ticket was mentioned in Slack in #core by sagarprajapati. View the logs.


4 months ago

#19 @welcher
3 months ago

  • Keywords commit added

Unit tests passed for me. Adding combined patch for the feature and the tests. I think this is good to go.

@welcher
3 months ago

Combined patch

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


3 months ago

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


2 months ago

#22 @jbpaul17
2 months ago

  • Milestone changed from 4.8.1 to 4.9

Punting per today's bug scrub and since 4.8.1 is focused on regressions.

Note: See TracTickets for help on using tickets.