Changeset 4641

Timestamp:

07/10/2011 03:23:12 PM (13 years ago)

Author:

boonebgorges

Message:

Changes main Notifications WP Admin Bar menu id, to avoid clashing with Settings > Notifications. Fixes #3294. Props johnjamesjacoby

Location:

trunk

Files:

• bp-core/css/admin-bar.css (modified) (1 diff)
• bp-members/bp-members-adminbar.php (modified) (3 diffs)

trunk/bp-core/css/admin-bar.css

@@ -123 +123 @@
 }
 
-#wpadminbar .quicklinks li#wp-admin-bar-notifications #ab-pending-notifications {
+#wpadminbar .quicklinks li#wp-admin-bar-bp-notifications #ab-pending-notifications {
     background: #eee;
     color: #333;

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

@@ -168 +168 @@
     // Add the top-level Notifications button
     $wp_admin_bar->add_menu( array(
-        'id'    => 'notifications',
+        'id'    => 'bp-notifications',
         'title' => $menu_title,
         'href'  => bp_loggedin_user_domain()
@@ -176 +176 @@
         foreach ( (array)$notifications as $notification ) {
             $wp_admin_bar->add_menu( array(
-                'parent' => 'notifications',
+                'parent' => 'bp-notifications',
                 'id'     => 'notification-' . $notification->id,
                 'title'  => $notification->content,
@@ -184 +184 @@
     } else {
         $wp_admin_bar->add_menu( array(
-            'parent' => 'notifications',
+            'parent' => 'bp-notifications',
             'id'     => 'no-notifications',
             'title'  => __( 'No new notifications', 'buddypress' ),