From ccfc12deb15558bccd7d1c5a586fef5b86d386ea Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Sat, 28 Sep 2013 00:36:43 -0400 Subject: [PATCH] controls: fix hotkeys when focusing elements If you're trying to type into a text field, then we don't want to steal the keypresses to switch tabs. --- main.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/main.js b/main.js index 266bb1e..1687598 100644 --- a/main.js +++ b/main.js @@ -61,17 +61,23 @@ window.onload = function() { }; window.onkeypress = function(e) { + if (e.target != document.body) { + /* Only allow the shortcuts when the focus is on the body. + Otherwise you can't type these numbers into text fields. */ + return; + } + switch (e.keyCode) { case 49: // 1 show_page('controls'); break; - case 50: // 1 + case 50: // 2 show_page('metadata'); break; - case 51: // 1 + case 51: // 3 show_page('playlist'); break; - case 52: // 1 + case 52: // 4 show_page('options'); break; } -- 2.39.2