Merge pull request #705 from bgrozev/esc-shortcut

Hide the shortcuts panel when the Escape key is pressed.

modules/UI/UI.js

@@ -1416,10 +1416,21 @@ UI.hideRingOverLay = function () {
 };
 
 /**
- * Shows or hides the keyboard shortcuts panel.'
+ * Shows or hides the keyboard shortcuts panel, depending on the current state.'
  */
 UI.toggleKeyboardShortcutsPanel = function() {
     $('#keyboard-shortcuts').toggle();
 };
 
+/**
+ * Shows or hides the keyboard shortcuts panel.'
+ */
+UI.showKeyboardShortcutsPanel = function(show) {
+    if (show) {
+        $('#keyboard-shortcuts').show();
+    } else {
+        $('#keyboard-shortcuts').hide();
+    }
+};
+
 module.exports = UI;

modules/keyboardshortcut/keyboardshortcut.js

@@ -3,13 +3,10 @@
 var shortcuts = {};
 function initShortcutHandlers() {
     shortcuts = {
-        191: {
+        27: {
-            character: "/",
+            character: "Esc",
-            function: function(e) {
+            function: function() {
-                // Only trigger on "?", not on "/".
+                APP.UI.showKeyboardShortcutsPanel(false);
-                if (e.shiftKey) {
-                    APP.UI.toggleKeyboardShortcutsPanel();
-                }
             }
         },
         67: {
@@ -59,6 +56,15 @@ function initShortcutHandlers() {
             function: function() {
                 APP.conference.toggleVideoMuted();
             }
+        },
+        191: {
+            character: "/",
+            function: function(e) {
+                // Only trigger on "?", not on "/".
+                if (e.shiftKey) {
+                    APP.UI.toggleKeyboardShortcutsPanel();
+                }
+            }
         }
     };
 }