refactor: move 'li' variable declaration outside of switch cases

movementlabsxyz · Aug 9, 2024 · 434d098 · 434d098
1 parent a06247e
commit 434d098
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/book/book/book.js b/book/book/book.js
@@ -408,22 +408,20 @@ function playground_text(playground, hidden = true) {
     document.addEventListener('keydown', function (e) {
         if (e.altKey || e.ctrlKey || e.metaKey || e.shiftKey) { return; }
         if (!themePopup.contains(e.target)) { return; }
-
+        var li = document.activeElement.parentElement;
         switch (e.key) {
             case 'Escape':
                 e.preventDefault();
                 hideThemes();
                 break;
             case 'ArrowUp':
                 e.preventDefault();
-                var li = document.activeElement.parentElement;
                 if (li && li.previousElementSibling) {
                     li.previousElementSibling.querySelector('button').focus();
                 }
                 break;
             case 'ArrowDown':
                 e.preventDefault();
-                var li = document.activeElement.parentElement;
                 if (li && li.nextElementSibling) {
                     li.nextElementSibling.querySelector('button').focus();
                 }