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

this line was making the android cursor break. #30

Closed
wants to merge 4 commits into from
Closed

this line was making the android cursor break. #30

wants to merge 4 commits into from

Conversation

minatonda
Copy link

Só removi a linha.
Estava quebrando em alguns dispositivos Android....
Agora não está mais.

@minatonda minatonda mentioned this pull request Sep 19, 2017
@minatonda
Copy link
Author

The second commit followed @pjaer

@luismiguelprs
Copy link

@neves, tem previsão para avaliação desse MR?
Estou com esse problema em alguns dispositivos android também. 😢

@hackuun
Copy link

hackuun commented Sep 26, 2017

This is aweful.
You should type this on starter page - DOESN'T WORK ON MOBILE

@vjoao
Copy link

vjoao commented Sep 27, 2017

@iamdubx this is open source software. People doesn't owe you anything. If you want it to get better, fork it, change it and PR it. You don't need to offend other people`s work just because it doesn't fulfill your lazy needs.

If you don't like it, just find another mask plugin or, even better, build your own. If you are capable of doing the latter, please post the Github link.

@hackuun
Copy link

hackuun commented Sep 27, 2017

@vjoao after investigating, I found out that Android bugs are common on ALL mask libraries! And all of them are using type="tel" to fix them!

So I am sorry for my previous post. I think @neves maybe you can put some info about type="tel" into Docs?

From this point, vue-the-mask is the most elegant solution for most usecases.

@minatonda
Copy link
Author

Personally, for me it works with this pull request.
I tested Android O, Android N and KITKAT devices (S6,S5,MOTOG3,ZENFONE3).

I modified this code directly in .min.js in my project, and it is working.

I'm sorry for the problem of the colleague, I hope he finds a solution for this, and share with us, for when we have the same;)

@neves
Copy link
Member

neves commented Oct 10, 2017

Fixed with version 0.11.0

@neves neves closed this Oct 10, 2017
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.

None yet

5 participants