'Members' menu selected when current page user profile page

[kaffeeguru]

In attached screen shot you can see that the current page selected is "My Profile" URL "/members/chris/profile/"

But also menu "members" URL "/members" was selected.

I checked that the current menu selection was done by wordpress core in file /wp-includes/nav-menu-template.php.
So I'm not 100% sure if this could be fixed within buddypress.

Seems like as "my Profile" also contains 'members' it selects the members link too.

[kaffeeguru]

screenshot of the two menu items selected at same time.

[johnjamesjacoby]

Ran out of time for 2.2, but I have a suspicion this will need addressing in 2.2.1.

[r-a-y]

This is kind of expected behavior.

We've been highlighting the directory page even when on a single item page since v1.5. I originally sought not to do this in r8821.

But then, the issue came about that we wanted to keep the highlighting for backward compatibility, see:
​https://buddypress.trac.wordpress.org/ticket/5810#comment:2

(Also see the follow-up ticket to highlight the menus using WP nav menu and page menu filters - #5997.)

If we're okay with not highlighting the directory page when on a single page, then 01.patch is an attempt to fix this. Instead of adding the current_page_item CSS class while on a BP single item page, it adds the current_page_parent CSS class, which is more conventional to WP.

If we want to do a hack to keep directory pages highlighted while on a single page, but not highlighted when we're using a BP nav menu member page item, then we'll need to approach this another way.

[johnjamesjacoby]

If this is expected behavior (meaning nothing has changed) let's move this to 2.3 and reconsider changes later.

[r-a-y]

Definitely not a regression, moving to 2.3.

I've tagged this dev-feedback, because I'm not sure what we want to do.

[boonebgorges]

Is it possible for us to detect whether the menu would have more than one highlighted item, and then ensure that only one gets highlighted, in a predictable way? The behavior described here, for example, only comes up when there's a 'My Profile' link in the menu. Seems to me that if such a link is present, we should give it the current_page_item class (since it's more specific than the directory page); but in that case, we'd want to remove the class from the directory page. Too hacky?