From d6e56924fe24a032394bf09b7889280fdc32ed4c Mon Sep 17 00:00:00 2001
From: Mikael Nordfeldth <mmn@hethane.se>
Date: Mon, 28 Sep 2015 00:47:16 +0200
Subject: [PATCH] Accessibility improvement, title for popup close button

Apparently it read as "times" in screen readers.
---
 js/util.js     | 2 +-
 lib/action.php | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/js/util.js b/js/util.js
index cd982bbb17..3e4da433af 100644
--- 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 7f3aea9c1a..4a815c1483 100644
--- 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());
 
-- 
2.39.2