Changeset 13259

Timestamp:

03/15/2022 09:48:27 PM (3 years ago)

Author:

imath

Message:

BP Nouveau: highlight the Messages UI compose nav when clicked

This nav was only set as current when the compose URL was loaded. As soon as the screen display was managed using JavaScript, it wasn't highlighted/set as current when clicked. This commit makes sure it's now the case.

Props devanshijoshi

See #8648 (trunk)

File:

• trunk/src/bp-templates/bp-nouveau/js/buddypress-messages.js (modified) (4 diffs)

trunk/src/bp-templates/bp-nouveau/js/buddypress-messages.js

@@ -2 +2 @@
 /* jshint devel: true */
 /* @since 3.0.0 */
-/* @version 8.0.0 */
+/* @version 10.2.0 */
 window.wp = window.wp || {};
 window.bp = window.bp || {};
@@ -98 +98 @@
         },
 
+        updateNav: function( view ) {
+            var currentView = this.box;
+
+            if ( view ) {
+                currentView = view;
+            }
+
+            // Activate the appropriate nav.
+            $( '#subnav ul li' ).each( function( l, li ) {
+                $( li ).removeClass( 'current selected' );
+            } );
+
+            $( '#subnav a#' + currentView ).closest( 'li' ).addClass( 'current selected' );
+        },
+
         removeTinyMCE: function() {
             if ( typeof tinymce !== 'undefined' ) {
@@ -170 +185 @@
             // Remove all existing views.
             this.clearViews();
+            this.updateNav( 'compose' );
 
             // Create the loop view.
@@ -182 +198 @@
 
         threadsView: function() {
-            // Activate the appropriate nav.
-            $( '#subnav ul li' ).each( function( l, li ) {
-                $( li ).removeClass( 'current selected' );
-            } );
-            $( '#subnav a#' + this.box ).closest( 'li' ).addClass( 'current selected' );
+            this.updateNav();
 
             // Create the loop view.