Merge pull request #906 from keepassxreboot/fix/check_isvisible_for_dynamic_fields

Check visibility of dynamically added fields

1 files changed, 5 insertions, 5 deletions

diff --git a/keepassxc-browser/content/keepassxc-browser.js b/keepassxc-browser/content/keepassxc-browser.js
index 98a73d9..f9b7513 100755
--- a/keepassxc-browser/content/keepassxc-browser.js
+++ b/keepassxc-browser/content/keepassxc-browser.js
@@ -711,12 +711,12 @@ kpxcObserverHelper.getInputs = function(target) {
 
     // Only include input fields that match with kpxcObserverHelper.inputTypes
     const inputs = [];
-    for (const i of inputFields) {
-        const type = i.getLowerCaseAttribute('type');
+    for (const field of inputFields) {
+        const type = field.getLowerCaseAttribute('type');
 
-        if (kpxcObserverHelper.inputTypes.includes(type)) {
-            kpxcFields.setUniqueId(i);
-            inputs.push(i);
+        if (kpxcObserverHelper.inputTypes.includes(type) && kpxcFields.isVisible(field)) {
+            kpxcFields.setUniqueId(field);
+            inputs.push(field);
         }
     }
     return inputs;