Changeset 7545

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.

Location:

trunk/bp-notifications

Files:

• bp-notifications-adminbar.php (modified) (3 diffs)
• bp-notifications-loader.php (modified) (2 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,
         ) );
     }

trunk/bp-notifications/bp-notifications-loader.php

@@ -27 +27 @@
             BP_PLUGIN_DIR,
             array(
-                'adminbar_myaccount_order' => 100
+                'adminbar_myaccount_order' => 30
             )
         );
@@ -109 +109 @@
             'name'                => sprintf( __( 'Notifications <span>%d</span>', 'buddypress' ), number_format_i18n( $count ) ),
             'slug'                => $this->slug,
-            'position'            => 60,
+            'position'            => 30,
             'screen_function'     => 'bp_notifications_screen_unread',
             'default_subnav_slug' => 'unread',