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

refactor(wallet-mobile): display native kb for device pin #3653

Draft
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

rahulnr7
Copy link
Contributor

@rahulnr7 rahulnr7 commented Sep 24, 2024

display native kb for device pin

Ticket

YOMO-1893

Note

Please create the ticket if missing it.

@rahulnr7 rahulnr7 added the wip Shows that a PR shouldn't be merge label Sep 24, 2024
@jorbuedo jorbuedo added the proposal Feature proposal for triage. Required for non-trivial features. label Sep 24, 2024
@rahulnr7
Copy link
Contributor Author

@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…

@rahulnr7
Copy link
Contributor Author

Simulator Screenshot - iPhone11 - 2024-09-24 at 16 21 23

@stackchain stackchain added this to the 5.1.0 "Notify me" milestone Sep 24, 2024
@jorbuedo
Copy link
Contributor

I don't like that in Android you can close the keyboard and then not have how to open it again.
Maybe force a focus on the input when clicking anywhere on that screen?

@rahulnr7
Copy link
Contributor Author

@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.

@stackchain stackchain changed the title fix(wallet): display native kb for device pin refactor(wallet-mobile): display native kb for device pin Oct 1, 2024
@stackchain stackchain marked this pull request as draft October 10, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal Feature proposal for triage. Required for non-trivial features. wip Shows that a PR shouldn't be merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants