Accessibility improvement, title for popup close button

Apparently it read as "times" in screen readers.

diff --git a/js/util.js b/js/util.js
index cd982bbb179c279d50fe28945e23654a011b459d..3e4da433afd3198f808d2dee5ec8382359ded90c 100644 (file)
--- a/js/util.js
+++ b/js/util.js
@@ -771,7 +771,7 @@ var SN = { // StatusNet
 
             form
                 .addClass('dialogbox')
-                .append('<button class="close">&#215;</button>')
+                .append('<button class="close" title="' + SN.msg('popup_close_button') + '">&#215;</button>')
                 .closest('.notice-options')
                     .addClass('opaque');
 

diff --git a/lib/action.php b/lib/action.php
index 7f3aea9c1a74115d2bbaccf4787faff1ad1ac4fc..4a815c148360efa6d0064c411cc6b06a669a245e 100644 (file)
--- a/lib/action.php
+++ b/lib/action.php
@@ -460,6 +460,7 @@ class Action extends HTMLOutputter // lawsuit
 
             // TRANS: Localized tooltip for '...' expansion button on overlong remote messages.
             $messages['showmore_tooltip'] = _m('TOOLTIP', 'Show more');
+            $messages['popup_close_button'] = _m('TOOLTIP', 'Close popup');
 
             $messages = array_merge($messages, $this->getScriptMessages());