From cf26d8547afe9cf7f7e52ed1ad1b59f803404820 Mon Sep 17 00:00:00 2001 From: Charles Renwick Date: Tue, 28 Nov 2017 11:29:28 -0500 Subject: [PATCH] Consolidates switch pattern for shortcut keys. Applies @groovecoder suggestions to consolidate a long switch statement. --- webextension/js/popup.js | 32 +++++--------------------------- 1 file changed, 5 insertions(+), 27 deletions(-) diff --git a/webextension/js/popup.js b/webextension/js/popup.js index 1d795bc..faaea97 100644 --- a/webextension/js/popup.js +++ b/webextension/js/popup.js @@ -529,36 +529,14 @@ Logic.registerPanel(P_CONTAINERS_LIST, { case 38: previous(); break; - case 49: - select(1); - break; - case 50: - select(2); - break; - case 51: - select(3); - break; - case 52: - select(4); - break; - case 53: - select(5); - break; - case 54: - select(6); - break; - case 55: - select(7); - break; - case 56: - select(8); - break; - case 57: - select(9); - break; case 48: select(10); break; + default: + if (e.keyCode >= 49 && e.keyCode <= 57) { + select(e.keyCode - 48); + } + break; } });