Fix key/mouse handling when settings visible

When the settings menu was being shown, the mouse and keyboard would not
work as expected; for example, neither could be used to modify the value
of an input[type="number"] field. Instead, a mouse click would be
interpreted as Blue getting (un)ready, which is not a desired behavior in
the settings menu.

js/Zatacka.js

@@ -319,17 +319,16 @@ const Zatacka = ((window, document) => {
     }
 
     function keyHandler(event) {
-        if (shouldPreventDefault(event.keyCode)) {
-            event.preventDefault();
-        }
         const callback = game.isStarted() ? gameKeyHandler
                                           : guiController.isShowingSettings() ? settingsKeyHandler
                                                                               : lobbyKeyHandler;
         guiController.keyPressed(event, callback);
     }
 
     function mouseHandler(event) {
-        const callback = game.isStarted() ? gameMouseHandler : lobbyMouseHandler;
+        const callback = game.isStarted() ? gameMouseHandler
+                                          : guiController.isShowingSettings() ? settingsMouseHandler
+                                                                              : lobbyMouseHandler;
         guiController.mouseClicked(event, callback);
     }
 
@@ -340,6 +339,9 @@ const Zatacka = ((window, document) => {
     }
 
     function lobbyKeyHandler(event) {
+        if (shouldPreventDefault(event.keyCode)) {
+            event.preventDefault();
+        }
         const pressedKey = event.keyCode;
         if (pressedKey === KEY_RELOAD) {
             reload();
@@ -366,6 +368,9 @@ const Zatacka = ((window, document) => {
     }
 
     function gameKeyHandler(event) {
+        if (shouldPreventDefault(event.keyCode)) {
+            event.preventDefault();
+        }
         const pressedKey = event.keyCode;
         if (isProceedKey(pressedKey)) {
             if (game.shouldQuitOnProceedKey()) {
@@ -404,6 +409,10 @@ const Zatacka = ((window, document) => {
         }
     }
 
+    function settingsMouseHandler(event) {
+        // Intentionally empty.
+    }
+
     function showSettings() {
         clearTimeout(hintPickTimer);
         clearTimeout(hintProceedTimer);