Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login fields not detected on www.thomann.de #2437

Open
z1atk0 opened this issue Jan 14, 2025 · 3 comments
Open

Login fields not detected on www.thomann.de #2437

z1atk0 opened this issue Jan 14, 2025 · 3 comments

Comments

@z1atk0
Copy link

z1atk0 commented Jan 14, 2025

The login fields on https://www.thomann.de/ are not detected automatically, ie. no icon in the username and password fields (when you click the "Account" button (top right, between the language/currency selector and the heart icon), the input mask slides in from the right side). Using the right-click menu to fill in username & password works fine. I have tried username only, custom login fields and improved input field detection to no avail.

  1. Exact login URL and the URL of your entry or entries: https://www.thomann.de/at/index.html
  2. Dump of console output from the page: N/A
  3. Have you tried Username-Only Detection, choosing Custom Fields etc.: ✅ yes
  4. Are you using Autofill instead of the manual one: ❌ no
  5. Does it affect the Username Icon, Autocomplete Menu, Popup Menu? ▶ not sure if I understand the question correctly, but the usual icon does not show up
  6. Version 1.8.0 and later, enable Debug Logging from the settings and inspect the pages's JavaScript console. ▶ see below:
[Debug keepassxc-browser.js:230] KeePassXC-Browser - Input fields found: [global.js:160:13](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
Array [ input#email.fx-input-text__field
 ]0: <input id="email" class="fx-input-text__field" placeholder="E-Mail-Adresse" name="email" aria-label="E-Mail-Adresse" inputmode="email" value="">length: 1<prototype>: Array []
[global.js:163:17](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
[Debug ] KeePassXC-Browser - Login field combinations identified: [global.js:160:13](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
Array [ {} ]
​
0: Object { username: null, password: null, passwordInputs: (1) [],  }
​
length: 1
​
<prototype>: Array []
[global.js:163:17](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
[Debug fields.js:199] KeePassXC-Browser - Input fields found: [global.js:160:13](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
Array [ input#email.fx-input-text__field
 ]
​
0: <input id="email" class="fx-input-text__field" placeholder="E-Mail-Adresse" name="email" aria-label="E-Mail-Adresse" inputmode="email" value="">
​
length: 1
​
<prototype>: Array []
[global.js:163:17](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
[Debug ] KeePassXC-Browser - Login field combinations identified: [global.js:160:13](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)
Array [ {} ]
​
0: Object { username: null, password: null, passwordInputs: (1) [],  }
​
length: 1
​
<prototype>: Array []
[global.js:163:17](moz-extension://aea999d2-9c36-4216-baf0-d6b8d226c957/common/global.js)

Firefox 128.6.0esr (64-bit), debug info from extension's About section:

KeePassXC - 2.7.9
KeePassXC-Browser - 1.9.6
Operating system: Linux x86_64
Browser: Mozilla Firefox 128.0
@varjolintu
Copy link
Member

This happens because of the login form animation. The input fields are detected immediately, but those are not yet visible, so the extension ignores them. The only workaround for now is to open the login form, open the popup from toolbar icon and click Redetect login fields. After that you can fill the credentials normally.

@z1atk0
Copy link
Author

z1atk0 commented Jan 15, 2025

Yup, the redetect login fields trick works for me. 👍 Thanks for the hint, looks like this was the one thing that I didn't think of and try myself. 🙄 😇 😁

@varjolintu
Copy link
Member

Of course we should integrate a proper support for fields behind animations, so let's keep the ticket open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants