Merge pull request jitsi#1159 from jitsi/non-focusable-close-toastr

Makes close button non-focusable.
poshest · Nov 23, 2016 · 56f1535 · 56f1535
2 parents 4900fe0 + b5dfc2a
commit 56f1535
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/modules/UI/UI.js b/modules/UI/UI.js
@@ -474,7 +474,9 @@ UI.start = function () {
             "hideEasing": "linear",
             "showMethod": "fadeIn",
             "hideMethod": "fadeOut",
-            "newestOnTop": false
+            "newestOnTop": false,
+            // this is the default toastr close button html, just adds tabIndex
+            "closeHtml": '<button type="button" tabIndex="-1">&times;</button>'
         };
 
     }

diff --git a/modules/UI/util/MessageHandler.js b/modules/UI/util/MessageHandler.js
@@ -439,7 +439,7 @@ var messageHandler = {
      * @param messageKey the key from the language file for the text of the
      * message.
      * @param messageArguments object with the arguments for the message.
-     * @param options object with language options.
+     * @param options passed to toastr (e.g. timeOut)
      */
     notify: function(displayName, displayNameKey, cls, messageKey,
                      messageArguments, options) {