Merge pull request #80 from akenneth/file_input_support

Allow hotkeys when focusing on `file` input elements

Author: akenneth

src/utils.ts

@@ -8,7 +8,12 @@ export function isFormField(element: Node): boolean {
   return (
     name === 'select' ||
     name === 'textarea' ||
-    (name === 'input' && type !== 'submit' && type !== 'reset' && type !== 'checkbox' && type !== 'radio') ||
+    (name === 'input' &&
+      type !== 'submit' &&
+      type !== 'reset' &&
+      type !== 'checkbox' &&
+      type !== 'radio' &&
+      type !== 'file') ||
     element.isContentEditable
   )
 }

test/test.js

@@ -72,10 +72,18 @@ describe('hotkey', function () {
       setHTML(`
       <button id="button1" data-hotkey="b">Button 1</button>
       <input id="textfield" />`)
-      document.getElementById('textfield').dispatchEvent(new KeyboardEvent('keydown', {key: 'b'}))
+      document.getElementById('textfield').dispatchEvent(new KeyboardEvent('keydown', {bubbles: true, key: 'b'}))
       assert.deepEqual(elementsActivated, [])
     })
 
+    it('triggers when user is focused on a file input', function () {
+      setHTML(`
+      <button id="button1" data-hotkey="b">Button 1</button>
+      <input id="filefield" type="file" />`)
+      document.getElementById('filefield').dispatchEvent(new KeyboardEvent('keydown', {bubbles: true, key: 'b'}))
+      assert.deepEqual(elementsActivated, ['button1'])
+    })
+
     it('handles multiple keys in a hotkey combination', function () {
       setHTML('<button id="button3" data-hotkey="Control+c">Button 3</button>')
       document.dispatchEvent(new KeyboardEvent('keydown', {key: 'c', ctrlKey: true}))