Merge branch 'feat/calculator' into feat/repo-homepage

pricebenjamin · Feb 25, 2024 · 5b34c6e · 5b34c6e
2 parents 97776e2 + d913baf
commit 5b34c6e
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/005-calculator/sources.js b/005-calculator/sources.js
@@ -7,7 +7,6 @@ document.addEventListener('DOMContentLoaded', () => {
     appRoot.addEventListener('click', clickHandler);
     document.addEventListener('keydown', keyboardHandler);
     document.addEventListener('keydown', simulateHover);
-    document.addEventListener('keyup', simulateHover);
 
     function clickHandler(event) {
         const target = event.target;
@@ -24,14 +23,20 @@ document.addEventListener('DOMContentLoaded', () => {
         btn.click();
     }
 
+    const buttonTimers = {};
+    const SIMULATED_HOVER_DELAY_MS = 100;
+
     function simulateHover(event) {
         const btn = buttonFilter(event);
         if (!btn) return;
-        if (event.type == 'keydown') {
-            btn.classList.add('hover');
-        } else if (event.type == 'keyup') {
-            btn.classList.remove('hover');
+
+        btn.classList.add('hover');
+        if (buttonTimers[event.key]) {
+            clearTimeout(buttonTimers[event.key]);
         }
+        buttonTimers[event.key] = setTimeout(() => {
+            btn.classList.remove('hover');
+        }, SIMULATED_HOVER_DELAY_MS);
     }
 
     function buttonFilter(event) {