Changeset 7545

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.

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',