Changeset 13925

Timestamp:

06/13/2024 05:40:43 PM (9 months ago)

Author:

imath

Message:

Keep the Notifications Admin Bar menu before the "My Account" one

This commit’s goal is to make sure people testing WordPress >= 6.6-beta2 during BP 14.0.0 beta testing period won’t believe this is a bug we missed. We’ll need to perform another commit once stable WP 6.6 will be released for 12.0 branch.

See #9182
Closes ​https://github.com/buddypress/buddypress/pull/313

File:

• trunk/src/bp-members/bp-members-adminbar.php (modified) (1 diff)

trunk/src/bp-members/bp-members-adminbar.php

@@ -268 +268 @@
     }
 }
-add_action( 'admin_bar_menu', 'bp_members_admin_bar_notifications_menu', 90 );
+
+/**
+ * Keep the Notification toolbar menu at the left of the "My Account" one.
+ *
+ * @since 12.6.0
+ */
+function bp_members_admin_bar_notifications_menu_priority() {
+    /*
+     * WordPress 6.6 edited the WP Admin style & removed the right float.
+     * See: https://core.trac.wordpress.org/changeset/58215/
+     */
+    if ( bp_is_running_wp( '6.6-beta2', '>=' ) ) {
+        bp_members_admin_bar_notifications_menu();
+    } else {
+        add_action( 'admin_bar_menu', 'bp_members_admin_bar_notifications_menu', 90 );
+    }
+}
+add_action( 'admin_bar_menu', 'bp_members_admin_bar_notifications_menu_priority', 6 );
 
 /**