Opened 14 years ago
Closed 7 years ago
#13910 closed feature request (fixed)
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)
Change History (31)
#3
@
14 years ago
- Cc wojtek.szkutnik@… added
- Keywords has-patch needs-testing added; wp_nav_menu menu name removed
#6
@
12 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 ); } ?>
#8
@
9 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.
9 years ago
@
8 years ago
Turns markoheijnen's comment into a patch. Adds filter for the title of a navigation menu.
#10
@
8 years ago
- Keywords needs-unit-tests added
Following what chriscct7 said in Slack, 13910-2.diff turns markoheijnen's comment into a patch.
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
7 years ago
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
7 years ago
#17
@
7 years 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.
7 years ago
#19
@
7 years 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.
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
7 years ago
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
7 years ago
#22
@
7 years 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.
+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