From 031bbd06acd2d8e6d9ef68d23cd96ce26764dcb6 Mon Sep 17 00:00:00 2001
From: James Darpinian <jdarpinian@gmail.com>
Date: Fri, 21 Jun 2024 22:52:23 -0700
Subject: [PATCH] Emscripten: Don't swallow browser keyboard shortcuts

---
 code/web/client.html | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/code/web/client.html b/code/web/client.html
index 3a23ab92f3..3afd91b057 100644
--- a/code/web/client.html
+++ b/code/web/client.html
@@ -8,6 +8,18 @@
 <canvas id=canvas></canvas>
 
 <script type=module>
+
+const defaultKeyBindingKeyCodes = ["KeyW", "KeyA", "KeyS", "KeyD", "KeyC", "KeyT", "Digit1", "Digit2", "Digit3", "Digit4", "Digit5", "Digit6", "Digit7", "Digit8", "Digit9", "Tab", "Space", "Enter", "NumpadEnter", "Delete", "Slash", "Backslash", "ArrowUp", "ArrowDown", "ArrowLeft", "ArrowRight", "PageDown", "End", "Escape", "ControlLeft", "ControlRight", "ShiftLeft", "ShiftRight", "AltLeft", "AltRight",];
+const defaultKeyBindingKeyCodesMap = defaultKeyBindingKeyCodes.reduce((acc, code) => { acc[code] = true; return acc; }, {});
+window.addEventListener("keydown", (e) => {
+        // Emscripten SDL2 will preventDefault all keyboard events which prevents browser keyboard shortcuts from working.
+        // This was supposed to be fixed in https://github.com/emscripten-core/emscripten/issues/16462 however the fix regressed.
+        // This hack lets the browser handle everything, except for the default Quake III keybindings. When the above mentioned
+        // regression is fixed, this can be replaced with a call to SDL_SetEventFilter in sdl_input.c respecting the actual
+        // active keybindings instead of hardcoding them here.
+        if (!defaultKeyBindingKeyCodesMap[e.code]) e.preventDefault = () => false;
+    }, { capture: true });
+
 // These strings are set in the generated HTML file in the build directory.
 let CLIENTBIN = '__CLIENTBIN__';
 let BASEGAME = '__BASEGAME__';