-
Notifications
You must be signed in to change notification settings - Fork 49
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
refactor(wallet-mobile): display native kb for device pin #3653
base: develop
Are you sure you want to change the base?
Conversation
@jorbuedo Thanks for checking it. I tried an experiment based on your comments for using the hidden text inputs. The native kb does show up but looks like it is not consistent. I will try further to improve it. Here is a recording from iOS how it works. Uploading Simulator Screen Recording - iPhone11 - 2024-09-24 at 15.54.37.mp4… |
apps/wallet-mobile/src/features/Auth/CreatePinInput/CreatePinInput.tsx
Outdated
Show resolved
Hide resolved
apps/wallet-mobile/src/features/Auth/CreatePinInput/CreatePinInput.tsx
Outdated
Show resolved
Hide resolved
I don't like that in Android you can close the keyboard and then not have how to open it again. |
@jorbuedo Yeah that seems to be an issue that I might need time to research. As this is earmarked for 5.1, will try again after closing the pending bugs. Thanks. |
display native kb for device pin
Ticket
YOMO-1893
Note
Please create the ticket if missing it.