From c3f6052435249033a36b807bc95e03942aba664f Mon Sep 17 00:00:00 2001 From: Jesper Dam Date: Wed, 12 Mar 2014 10:08:14 +0100 Subject: [PATCH] Better key identifiers Previously we identified keys in keyboard events by the 'key' property if it was set, and 'keyCode' otherwise. This turns out to be problematic as Firefox no longer leaves 'key' undefined (so we fall back to using 'keyCode'), but instead sets 'key' to 'MozPrintableKey' for all printable keys. This meant that when (printable) keys are released, we can't match it against the corresponding keydown event, and instead just send a keyup event for the last keydown received. Now, if both 'key' and 'keyCode' are set, use the concatenation of both. Otherwise prefer 'keyCode', as that is at least unique for every key. This should let us release the right keys on keyup events. --- include/keyboard.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/include/keyboard.js b/include/keyboard.js index f21f9767..3fde55bb 100644 --- a/include/keyboard.js +++ b/include/keyboard.js @@ -152,11 +152,14 @@ var kbdUtil = (function() { // Get a key ID from a keyboard event // May be a string or an integer depending on the available properties function getKey(evt){ - if (evt.key) { - return evt.key; + if ('keyCode' in evt && 'key' in evt) { + return evt.key + ':' + evt.keyCode; + } + else if ('keyCode' in evt) { + return evt.keyCode; } else { - return evt.keyCode; + return evt.key; } }