Changeset 11518

Timestamp:

03/21/2017 02:14:45 AM (8 years ago)

Author:

boonebgorges

Message:

Better current-item determination in bp_nav_menu().

Current component *and* action should both be matched when
determining current item, so as to avoid clashes when two
top-level items have children by the same name.

Props lakrisgubben.
Fixes #4986.

File:

• trunk/src/bp-core/bp-core-template.php (modified) (1 diff)

trunk/src/bp-core/bp-core-template.php

@@ -3609 +3609 @@
 
                 // If we're viewing this item's screen, record that we need to mark its parent menu to be selected.
-                if ( $sub_menu->slug == bp_current_action() ) {
+                if ( bp_is_current_action( $sub_menu->slug ) && bp_is_current_component( $nav_menu->slug ) ) {
                     $menu->class[]    = 'current-menu-parent';
                     $submenu->class[] = 'current-menu-item';