From 4d3adb09cd99ce43f93703dee358f29637dddd74 Mon Sep 17 00:00:00 2001 From: Henrik Ingo Date: Tue, 1 Jan 2019 21:40:46 +0200 Subject: [PATCH] Fix form plugin for input fields. It turns out input[type=text] will only find input fields where the type attribute is explicitly set to text, but would skip fields that left it out and defaulted to type text. This changes to catch all types of input elements. --- js/impress.js | 5 ++--- src/plugins/form/form.js | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/js/impress.js b/js/impress.js index 60fba7c..cf0d8ff 100644 --- a/js/impress.js +++ b/js/impress.js @@ -1628,7 +1628,7 @@ api = event.detail.api; var gc = api.lib.gc; - var selectors = [ "input[type=text]", "textarea", "select", "[contenteditable=true]" ]; + var selectors = [ "input", "textarea", "select", "[contenteditable=true]" ]; for ( var selector of selectors ) { var elements = document.querySelectorAll( selector ); if ( !elements ) { @@ -1893,7 +1893,7 @@ document.addEventListener( "keyup", function( event ) { - if ( event.keyCode === 72 ) { // "h" + if ( event.keyCode === 72 || event.keyCode === 191 ) { // "h" || "?" event.preventDefault(); toggleHelp(); } @@ -3203,7 +3203,6 @@ if ( event.shiftKey ) { switch ( event.keyCode ) { case 9: // Shift+tab - case 32: // Shift+space api.prev(); break; } diff --git a/src/plugins/form/form.js b/src/plugins/form/form.js index 54172bb..bbd9f47 100644 --- a/src/plugins/form/form.js +++ b/src/plugins/form/form.js @@ -32,7 +32,7 @@ api = event.detail.api; var gc = api.lib.gc; - var selectors = [ "input[type=text]", "textarea", "select", "[contenteditable=true]" ]; + var selectors = [ "input", "textarea", "select", "[contenteditable=true]" ]; for ( var selector of selectors ) { var elements = document.querySelectorAll( selector ); if ( !elements ) {