Merge pull request #2351 from keepassxreboot/fix/predefined_sites

Fix Predefined Sites for some pages
keepassxreboot · Oct 1, 2024 · 69e2986 · 69e2986
2 parents 8b31246 + a30cb27
commit 69e2986
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/keepassxc-browser/common/sites.js b/keepassxc-browser/common/sites.js
@@ -35,7 +35,8 @@ const PREDEFINED_SITELIST = [
 ];
 
 const IMPROVED_DETECTION_PREDEFINED_SITELIST = [
-    'https://secure.chase.com/*'
+    'https://secure.chase.com/*',
+    'https://www.icloud.com/'
 ];
 
 const googleUrl = 'https://accounts.google.com';
@@ -87,7 +88,8 @@ kpxcSites.exceptionFound = function(identifier, field) {
               || document.location.origin.startsWith('https://mail.protonmail.com')
               && identifier === 'mailboxPassword') {
         return true;
-    } else if (document.location.origin === 'https://www.patreon.com' && field?.name === 'current-password') {
+    } else if (document.location.origin === 'https://www.patreon.com' &&
+               (field?.name === 'current-password' || field?.innerHTML?.includes('current-password'))) {
         return true;
     } else if (document.location.origin === 'https://wordpress.com' && identifier?.value === 'login__form-password') {
         return true;

diff --git a/keepassxc-browser/content/fields.js b/keepassxc-browser/content/fields.js
@@ -105,7 +105,7 @@ kpxcFields.getSegmentedTOTPFields = function(inputs, combinations) {
                 totpInputs.length === DEFAULT_SEGMENTED_TOTP_FIELDS) &&
             totpInputs.every(
                 input =>
-                    (input.inputMode === 'numeric' && input.pattern.includes('0-9')) ||
+                    ((input.inputMode === 'numeric' || input.inputMode === '') && input.pattern.includes('0-9')) ||
                     ((input.type === 'text' || input.type === 'number') && input.maxLength === 1) ||
                     input.type === 'tel'
             )