Inconsistent GUI between Frinds and Messaging

Guys, just a suggestion but there seem to be a bit of an inconsistency between the way Friends and Messaging plugins notify users about new events. Consider the following screenshots:

https://premium.wpmudev.org/wp-content/uploads/2011/04/friends61.jpg
https://premium.wpmudev.org/wp-content/uploads/2011/04/message61.jpg

Messaging plugin seem to update it's menu item using WP's update classes to notify user about new mail, while Friends simply lists current number of new requests.

Considering how tightly integrated this two plugins are, wouldn't it be nicer if the Friends plugin updated menu item similarly to Messaging plugin? It would also prevent users from overlooking friends requests when email notification is disabled.

    UmkaDK

    Hi guys,

    I've had a bit of time on my hands, so did a little patch on the Friends plugin to bring users notification events inline with Messaging plugin. I've tried to keep the patch as small as possible, and to mimic existing coding style of the two plugins, so the changes I've introduced should be quite obvious.

    INSTALL:
    1. Install Friends plugin version 1.1.7
    2. Save attached patch into friends.patch file in your /wp-content/plugins directory.
    3. Apply patch with cd /wp-content/plugins && patch -p0 < friends.patch.
    4. Remove, now redundant, friends.patch file.

    diff -ur friends/friends.php friends/friends.php
    --- friends/friends.php	2011-10-27 01:32:48.000000000 -0700
    +++ friends/friends.php	2012-01-30 05:30:09.000000000 -0800
    @@ -144,13 +144,22 @@
    
     function friends_plug_pages() {
     	global $wpdb, $user_ID, $friends_enable_approval;
    +    $count_output = '';
    
    -    add_menu_page( __('Friends', 'friends' ), __('Friends', 'friends' ), 'read', 'friends', 'friends_output' );
    +    if ( $friends_enable_approval ) {
    +		$tmp_friend_requests_count = $wpdb->get_var("SELECT COUNT(*) FROM " . $wpdb->base_prefix . "friends WHERE friend_user_ID = '" . $user_ID . "' AND friend_approved = '0'");
    +
    +        if ($tmp_friend_requests_count > 0) {
    +            $count_output = '&nbsp;<span class="update-plugins"><span class="updates-count count-' . $tmp_friend_requests_count . '">' . $tmp_friend_requests_count . '</span></span>';
    +        }
    +    }
    +
    +    add_menu_page( __('Friends', 'friends' ), __('Friends', 'friends' ) . $count_output, 'read', 'friends', 'friends_output' );
    +    add_submenu_page( 'friends', __('Friends', 'friends' ), __('Friends', 'friends' ), 'read', 'friends', 'friends_output' );
         add_submenu_page( 'friends', __('Friends', 'friends' ), __('Find Friends', 'friends' ), 'read', 'find-friends', 'friends_find_output' );
    
     	if ( $friends_enable_approval ) {
    -		$tmp_friend_requests_count = $wpdb->get_var("SELECT COUNT(*) FROM " . $wpdb->base_prefix . "friends WHERE friend_user_ID = '" . $user_ID . "' AND friend_approved = '0'");
    -		add_submenu_page( 'friends', __( 'Friends', 'friends' ), __( 'Friend Requests', 'friends' ) . ' (' . $tmp_friend_requests_count . ')', 'read', 'friend-requests', 'friends_requests_output' );
    +		add_submenu_page( 'friends', __( 'Friends', 'friends' ), __( 'Friend Requests', 'friends' ) . $count_output, 'read', 'friend-requests', 'friends_requests_output' );
     	}
     	add_submenu_page( 'friends', __( 'Friends', 'friends' ), __( 'Notifications', 'friends' ), 'read', 'friend-notifications', 'friends_notifications_output' );
     }
    @@ -937,4 +946,4 @@
     add_action( 'network_admin_notices', 'wdp_un_check', 5 );
     endif;
    
    -?>
    \ No newline at end of file
    +?>