Changeset 5346

Timestamp:

11/23/2011 09:12:18 PM (15 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(),
         ) );