Add Filter to Nav Menu Support Themes Text

[wpsmith]

Frameworks could use a filter here to customize the message: _n('Your theme supports %s menu. Select which menu you would like to use.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ). For example, it may be the child theme that doesn't support the menus. Also, if none are supported (say via add_theme_support), then when it's zero, it says: "Your theme supports 0 menus. Select which menu appears in each location." (which doesn't make much sense). So adding a filter can enable theme developers to further customize.

An example use case:

add_filter( 'nav_menu_theme_support_text' , 'wps_nav_menu_theme_support_text' );
function wps_nav_menu_theme_support_text ( $num_locations ) {
	if ( $num_locations == 0 )
		$text = 'Your child theme does not support custom menus.';
	else
		$text = _n('Your theme supports %s menu. Select which menu you would like to use.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations );
	return $text;
}

[chriscct7]

Needs to be refreshed because of #24945

[bhargavbhandari90]

Check with this. As we applying filter to a string so add

[bhargavbhandari90]

Replying to wpsmith:

Frameworks could use a filter here to customize the message: _n('Your theme supports %s menu. Select which menu you would like to use.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ). For example, it may be the child theme that doesn't support the menus. Also, if none are supported (say via add_theme_support), then when it's zero, it says: "Your theme supports 0 menus. Select which menu appears in each location." (which doesn't make much sense). So adding a filter can enable theme developers to further customize.

An example use case:
add_filter( 'nav_menu_theme_support_text' , 'wps_nav_menu_theme_support_text' );
function wps_nav_menu_theme_support_text ( $num_locations ) {

if ( $num_locations == 0 )

$text = 'Your child theme does not support custom menus.';

else

$text = _n('Your theme supports %s menu. Select which menu you would like to use.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations );

return $text;

}

As we are applying filters to string, so add filter will contain two arguments so $num_locations should be 2nd argument..

[DrewAPicture]

As of [31951] we're using the decoupled strings from the Customizer here, which actually means we're using one of two different strings. @SergeyBiryukov: What would be your best suggestion if we wanted to make this text filterable? My first thought was to package them into an array and make that filterable, but I'm not sure I like that.

Currently:

<?php
if ( 1 == $num_locations ) {
        echo '<p>' . __( 'Your theme supports one menu. Select which menu you would like to use.' ) . '</p>';
} else {
        echo '<p>' .  sprintf( _n( 'Your theme supports %s menu. Select which menu appears in each location.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ), number_format_i18n( $num_locations ) ) . '</p>';
}

[DrewAPicture]

Moving off the milestone until we can figure out a plan here.

[DrewAPicture]

Assigning ownership to mark the good-first-bug as "claimed".

Trac ticket: https://core.trac.wordpress.org/ticket/19272

Trac ticket: https://core.trac.wordpress.org/ticket/19272

Add a filter nav_menu_theme_support_text

Trac ticket: https://core.trac.wordpress.org/ticket/19272