-
Notifications
You must be signed in to change notification settings - Fork 2
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 button disabled when username and password are auto filled #236
Comments
didn't really find a solution for this. |
I found this open issue for React: facebook/react#1159 It sounds browsers won't file the change event because it would be a security vulnerability, so it's difficult to detect. This comment suggests that the best way to handle forms is to never disable the submit button and do all validation on submit: facebook/react#1159 (comment) I also noticed this does not happen when the login page is accessed via client side navigation. |
I did find one interesting solution that uses CSS to detect the autofill event: https://github.com/klarna/ui/blob/v4.10.0/Field/index.jsx#L104-L114 and https://github.com/klarna/ui/blob/v4.10.0/Field/styles.scss#L228-L241 |
i tried this solution but couldn't get it to work. do you want to give it another try? |
according to https://bugs.chromium.org/p/chromium/issues/detail?id=1166619 |
noticed this in Celeste's workshop
If user has chrome to remember the username and password, it looks like now we don't catch the auto fill
so the login button is disabled although user see both inputs are filled.
The text was updated successfully, but these errors were encountered: