Merge branch 'dtw-waleee-fix-help-modal'

closes #3455

lib/std/special/docs/main.js

@@ -1141,10 +1141,8 @@
     }
 
     function onSearchKeyDown(ev) {
-        switch (ev.which) {
+        switch (getKeyString(ev)) {
-            case 13:
+            case "Enter":
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
-
                 // detect if this search changes anything
                 var terms1 = getSearchTerms();
                 startSearch();
@@ -1157,9 +1155,7 @@
                 ev.preventDefault();
                 ev.stopPropagation();
                 return;
-            case 27:
+            case "Esc":
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
-
                 domSearch.value = "";
                 domSearch.blur();
                 curSearchIndex = -1;
@@ -1167,16 +1163,12 @@
                 ev.stopPropagation();
                 startSearch();
                 return;
-            case 38:
+            case "Up":
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
-
                 moveSearchCursor(-1);
                 ev.preventDefault();
                 ev.stopPropagation();
                 return;
-            case 40:
+            case "Down":
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
-
                 moveSearchCursor(1);
                 ev.preventDefault();
                 ev.stopPropagation();
@@ -1210,26 +1202,49 @@
         renderSearchCursor();
     }
 
-    function onWindowKeyDown(ev) {
+    function getKeyString(ev) {
+        var name;
+        var ignoreShift = false;
         switch (ev.which) {
+            case 13:
+                name = "Enter";
+                break;
             case 27:
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
+                name = "Esc";
+                break;
+            case 38:
+                name = "Up";
+                break;
+            case 40:
+                name = "Down";
+                break;
+            default:
+                ignoreShift = true;
+                name = (ev.key != null) ? ev.key : String.fromCharCode(ev.charCode || ev.keyCode);
+        }
+        if (!ignoreShift && ev.shiftKey) name = "Shift+" + name;
+        if (ev.altKey) name = "Alt+" + name;
+        if (ev.ctrlKey) name = "Ctrl+" + name;
+        return name;
+    }
+    
+    function onWindowKeyDown(ev) {
+        switch (getKeyString(ev)) {
+            case "Esc":
                 if (!domHelpModal.classList.contains("hidden")) {
                     domHelpModal.classList.add("hidden");
                     ev.preventDefault();
                     ev.stopPropagation();
                 }
                 break;
-            case 83:
+            case "s":
-                if (ev.shiftKey || ev.ctrlKey || ev.altKey) return;
                 domSearch.focus();
                 domSearch.select();
                 ev.preventDefault();
                 ev.stopPropagation();
                 startAsyncSearch();
                 break;
-            case 191:
+            case "?":
-                if (!ev.shiftKey || ev.ctrlKey || ev.altKey) return;
                 ev.preventDefault();
                 ev.stopPropagation();
                 showHelpModal();