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

no ability to select the numbers #12

Open
anselmh opened this issue Jan 30, 2013 · 5 comments
Open

no ability to select the numbers #12

anselmh opened this issue Jan 30, 2013 · 5 comments

Comments

@anselmh
Copy link

anselmh commented Jan 30, 2013

There is no ability to select the numbers after entering some. It just deselects immediately.

@rakeev
Copy link

rakeev commented Jan 30, 2013

+1 and no way to correct one digit in the beginning of number

@zdfs
Copy link
Owner

zdfs commented Jan 30, 2013

This is an issue with the input masking plugin, not the experiment itself.

@anselmh
Copy link
Author

anselmh commented Jan 30, 2013

The input masking plugin has an option to disable 'disabling native focus'. This should fix the issue. From accessbility view the input masking plugin is not good as it is implemented here. I don't really know if that is a general problem with this plugin.
Can you investigate here and if it's not fixable on your own, file an issue over at the plugin's repo? Would love to see this code more accessible and usable because it is a great idea. :)

@zdfs
Copy link
Owner

zdfs commented Jan 30, 2013

Considering that this is labeled as an "experiment", I'm not sure when I'm going to have time to investigate. If I remember correctly, I did turn off "disabling native focus" and it only presented even more problems. In my opinion, input masking should be provided natively, but as far as I'm aware, no discussions are being had at the WHATWG or any other body around this topic.

The experiment itself was never developed for production, just an attempt to see how far I could take it. I have more ideas, especially for mobile, but my time to dedicate to this is very limited.

@anselmh
Copy link
Author

anselmh commented Jan 30, 2013

Okay, thanks for explaining. Maybe you should add this to the README to avoid people use this in production.

This was referenced Apr 23, 2014
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

No branches or pull requests

3 participants