[wp-trac] [WordPress Trac] #19272: Add Filter to Nav Menu Support Themes Text
WordPress Trac
noreply at wordpress.org
Sun Feb 22 08:30:52 UTC 2015
#19272: Add Filter to Nav Menu Support Themes Text
-------------------------------------------------+-------------------------
Reporter: wpsmith | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting
Component: Menus | Review
Severity: trivial | Version:
Keywords: good-first-bug good-first-patch | Resolution:
has-patch needs-testing | Focuses:
-------------------------------------------------+-------------------------
Comment (by bhargavbhandari90):
Replying to [ticket:19272 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..
--
Ticket URL: <https://core.trac.wordpress.org/ticket/19272#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list