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

Prevent exception with browser autocompletion #651

Merged
merged 1 commit into from
Feb 12, 2018

Conversation

vkurlyan
Copy link
Contributor

@vkurlyan vkurlyan commented Feb 12, 2018

I got an exception with "Google smart lock" in chrome. In single page application, if there is hidden login page where chrome automatically fills user's data. Then keyboard is shown because of autocompletion and we get an exception in $keyboard.caret() function "Cannot read length of null".

With this change, the keyboard is still opened for the hidden field but at least there is no exception.
To prevent opening of the keyboard developers should avoid hidden fields which could be autocompleted.

I got an exception with "Google smart lock" in chrome. In single page application, if there is hidden login page where chrome automatically fills user's data. Then keyboard is shown because of autocompletion and we get an exception in $keyboard.caret() function "Cannot read length of null".

With this changes keyboard is still opened but at least there is no exception.
@Mottie Mottie merged commit e4f9dd3 into Mottie:master Feb 12, 2018
@Mottie
Copy link
Owner

Mottie commented Feb 12, 2018

Hi @vkurlyan!

Thanks for the PR! I'm not sure when I'll be ready for the next release. I need some time to work on a few open issues, but if it takes longer than a week, I'll push a new release.

@vkurlyan
Copy link
Contributor Author

@Mottie Thank you.

@Mottie
Copy link
Owner

Mottie commented Feb 24, 2018

v1.28.0 is now available.

@Mottie Mottie mentioned this pull request May 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants