[buddypress-trac] [BuddyPress Trac] #7799: Nouveau: Vertical nav toggle doesn't work in Customizer

buddypress-trac noreply at wordpress.org
Thu May 3 15:23:03 UTC 2018

#7799: Nouveau: Vertical nav toggle doesn't work in Customizer
 Reporter:  boonebgorges  |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  3.0
Component:  Templates     |    Version:
 Severity:  normal        |   Keywords:
 Vertical item nav styling (group, user) is currently controlled by *two*
 selectors: `.bp-vertical-navs` on the `buddypress-wrap` element, and
 `.vertical` on the `nav` element. This causes problems in the Customizer,
 because toggling 'Display the Group nav vertically...' triggers a refresh
 where only the `.bp-vertical-navs` class is present, not `.vertical`. As
 such, we have the following situation:

 1. Your saved setting is for horizontal menus
 2. You go into the Customizer and navigate to a group page
 3. You toggle the Group Navigation to 'vertical'
 4. The page refreshes, but the nav is still vertical in the refreshed

 This appears to be due to a bug in
 `bp_nouveau_get_single_item_nav_classes()`, where a strict check `1 ===`
 is being performed on a "temporary setting" that is never an integer.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7799>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac

More information about the buddypress-trac mailing list