[buddypress-trac] [BuddyPress] #2258: Wrong highlight CSS class in bp_get_loggedin_user_nav()

[buddypress-trac at lists.automattic.com]

#2258: Wrong highlight CSS class in bp_get_loggedin_user_nav()
--------------------+-------------------------------------------------------
 Reporter:  21cdb   |       Owner:                                               
     Type:  defect  |      Status:  new                                          
 Priority:  minor   |   Milestone:  1.2.4                                        
Component:  Core    |    Keywords:  highlight CSS class, bp-core-templatetags-php
--------------------+-------------------------------------------------------
 If Theme Developers play around with bp_get_loggedin_user_nav() in there
 custom theme and make it visible all time, also if you are on the Groups
 Directory page, then it happens that the highlight class is applied on the
 "Groups (x)" menu item within bp_get_loggedin_user_nav() when you are on
 the Groups Directory Page. This is because both have "groups" within the
 url and item CSS id.

 The issue can be prevent by adding !bp_is_directory check to
 bp_get_loggedin_user_nav()

 File: bp-core-templatetags.php
 Line: 429

 Original:

 if ( $bp->active_components[$bp->current_component] == $nav_item['css_id']
 ) {

 Patch:

 if ( !bp_is_directory() && $bp->active_components[$bp->current_component]
 == $nav_item['css_id'] ) {

 Small issue, but annoying if someone wants to build its own navigation
 structure, would be great if this could be fixed.

-- 
Ticket URL: <https://trac.buddypress.org/ticket/2258>
BuddyPress <http://buddypress.org/>
BuddyPress