-
Notifications
You must be signed in to change notification settings - Fork 155
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
unhandled error with browser autofill #565
Comments
Glad to hear that! Could you provide more details so I can fix it in a reasonable time, please? Ideally:
Thank you! |
Sry for the late reponse.
Here's the example
|
Thanks! I can reproduce it with your example (on Windows 10, Chrome). The browser auto-fills the input and then fires the event with
-> it fails because it can't convert How we should treat that "malformed" |
I've done another research round. Demonstration & explanationTry https://jsfiddle.net/0mhwj6v5/1/ with opened console in dev tools on your browsers. You should see something like or The first image is a Firefox version. Autofill fires only The second image is a Chrome version. Autofill fires both Bug reports / related issues
Expected behaviorI wasn't able to find official specs for autofill behavior. I think the Firefox's behavior is the correct one - it doesn't make sense to fire General solutionI wasn't able to find a good solution that works on all browsers unfortunately. There are MANY tips, workaround and hacks, but most of them are either too old or too ugly. You can find workarounds like this one - https://github.com/paperjs/paper.js/pull/1358/files - in multiple projects. Solution for SeedThe first idea:
Next steps@Notmeor I would be glad if you try to create a PR, implementing the idea above or your idea (if you have any). Once we know it works on your Mac (+ Safari, Chrome, Firefox), I'll test it on Windows and merge it. |
I built an admin spa with seed, and everything worked out great!
only that every time i log in, an error would pop up in browser console saying
it only happens when using password autofill, and except for the error msg, all functionalities seem intact.
The text was updated successfully, but these errors were encountered: