Changeset 7545 for trunk/bp-notifications/bp-notifications-adminbar.php

Timestamp:

11/09/2013 03:16:01 PM (11 years ago)

Author:

johnjamesjacoby

Message:

No humility. Bump notifications menu and navigation position to below Profile, and ensure that custom notifications toolbar menu now links to new notifications page for logged in user. See #5148.

File:

• trunk/bp-notifications/bp-notifications-adminbar.php (modified) (3 diffs)

trunk/bp-notifications/bp-notifications-adminbar.php

@@ -29 +29 @@
     $alert_class   = (int) $count > 0 ? 'pending-count alert' : 'count no-alert';
     $menu_title    = '<span id="ab-pending-notifications" class="' . $alert_class . '">' . $count . '</span>';
+    $menu_link     = trailingslashit( bp_loggedin_user_domain() . bp_get_notifications_slug() );
 
     // Add the top-level Notifications button
@@ -35 +36 @@
         'id'        => 'bp-notifications',
         'title'     => $menu_title,
-        'href'      => bp_loggedin_user_domain(),
+        'href'      => $menu_link,
     ) );
 
@@ -52 +53 @@
             'id'     => 'no-notifications',
             'title'  => __( 'No new notifications', 'buddypress' ),
-            'href'   => bp_loggedin_user_domain(),
+            'href'   => $menu_link,
         ) );
     }