-
Notifications
You must be signed in to change notification settings - Fork 8
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
TOTP Codes should auto-select any contents upon tab #155
Comments
I looked into this, and it's incredibly simple to do.. but.. I ran into an edge-case that I wasn't able to easily workaround. The auto-tabbing is based on an onChange event handler, but if you type the same digit that's already displayed, it doesn't trigger the change event, which causes it not to advance, which instantly breaks your flow. The alternative is to simulate the change event, tab on keyup, etc, which quickly went down the path of "This is getting messy" |
@dd32 I've raised the above PR and I think it covers the issues you've described. Try it out :) |
Ideally at step 4, upon tabbing to an input that has contents, it should select the existing content to allow typing over it, but not clear it.
This could also be achieved as a general "Upon field being focused, select the character" style to allow easy overwrite.
The text was updated successfully, but these errors were encountered: