Changeset 5346

Timestamp:

11/23/2011 09:12:18 PM (13 years ago)

Author:

johnjamesjacoby

Message:

Tweaks to WP toolbar menus:

• WP3.3 compat
• Move Notification bubble to right, next to my-account
• Make my-account-buddypress a menu-group, so dependent children always follow
• See #3596

Location:

trunk

Files:

• bp-core/bp-core-adminbar.php (modified) (2 diffs)
• bp-members/bp-members-adminbar.php (modified) (1 diff)

trunk/bp-core/bp-core-adminbar.php

@@ -20 +20 @@
  * @return If doing ajax
  */
-function bp_admin_bar_my_account_secondary() {
+function bp_admin_bar_my_account_root() {
     global $wp_admin_bar;
 
@@ -32 +32 @@
         // Add secondary parent item for all BuddyPress components
         $wp_admin_bar->add_menu( array(
-            'parent' => 'my-account',
-            'id'     => 'my-account-buddypress',
-            'title'  => ' ',
-            'meta'   => array(
-                'class' => 'secondary',
+            'parent'    => 'my-account',
+            'id'        => 'my-account-buddypress',
+            'title'     => __( 'My Account' ),
+            'group'     => true,
+            'meta'      => array(
+                'class' => 'ab-sub-secondary'
             )
         ) );
     }
 }
-add_action( 'admin_bar_menu', 'bp_admin_bar_my_account_secondary', 9999 );
+add_action( 'admin_bar_menu', 'bp_admin_bar_my_account_root', 100 );
 
 /**

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

@@ -190 +190 @@
     // Add the top-level Notifications button
     $wp_admin_bar->add_menu( array(
-        'id'    => 'bp-notifications',
-        'title' => $menu_title,
-        'href'  => bp_loggedin_user_domain(),
-        'meta'  => array(
-            'class' => 'opposite',
-        )
+        'parent'    => 'top-secondary',
+        'id'        => 'bp-notifications',
+        'title'     => $menu_title,
+        'href'      => bp_loggedin_user_domain(),
     ) );