Fixes #4170: Messages icon on top bar doesn't have a title attribute

(tooltip)

diff --git a/mod/messages/languages/en.php b/mod/messages/languages/en.php
index 308c3d5194a5c18e86dd1cef72ff36178f2902d1..7732a9dce4e2522f40601ad62ee73c07abf60b6c 100644 (file)
--- a/mod/messages/languages/en.php
+++ b/mod/messages/languages/en.php
@@ -11,6 +11,7 @@ $english = array(
        */
 
        'messages' => "Messages",
+       'messages:unreadcount' => "%s unread",
        'messages:back' => "back to messages",
        'messages:user' => "%s's inbox",
        'messages:posttitle' => "%s's messages: %s",

diff --git a/mod/messages/start.php b/mod/messages/start.php
index e4ef4d43a53b5b8b01affce4e8d8d21851272dcc..1a27093248c350712e77dd93e094a3740d383a3a 100644 (file)
--- a/mod/messages/start.php
+++ b/mod/messages/start.php
@@ -125,11 +125,13 @@ function messages_notifier() {
        if (elgg_is_logged_in()) {
                $class = "elgg-icon elgg-icon-mail";
                $text = "<span class='$class'></span>";
-
+               $tooltip = elgg_echo("messages");
+               
                // get unread messages
                $num_messages = (int)messages_count_unread();
                if ($num_messages != 0) {
                        $text .= "<span class=\"messages-new\">$num_messages</span>";
+                       $tooltip .= " (" . elgg_echo("messages:unreadcount", array($num_messages)) . ")";
                }
 
                elgg_register_menu_item('topbar', array(
@@ -137,6 +139,7 @@ function messages_notifier() {
                        'href' => 'messages/inbox/' . elgg_get_logged_in_user_entity()->username,
                        'text' => $text,
                        'priority' => 600,
+                       'title' => $tooltip,
                ));
        }
 }