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

Timestamp:

11/09/2013 03:16:01 PM (13 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,
                 ) );
         }