-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat(deposit): fix android dob input; improve autofill UX #17001
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
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
…k-mobile into deposit-android-dob-fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: SSN Field Submission Issue
Pressing "Done" on the keyboard for the SSN field (the last form field) no longer submits the form and proceeds to the next screen. This is because the handleOnPressContinue() call was removed from its onSubmitEditing handler. Users must now manually tap the "Continue" button, which is a regression in the expected user experience.
app/components/UI/Ramp/Deposit/Views/BasicInfo/BasicInfo.tsx#L254-L257
metamask-mobile/app/components/UI/Ramp/Deposit/Views/BasicInfo/BasicInfo.tsx
Lines 254 to 257 in fb19722
| maxLength={11} | |
| onSubmitEditing={() => { | |
| Keyboard.dismiss(); | |
| }} |
Was this report helpful? Give feedback by reacting with 👍 or 👎
|
|



Description
screen-20250708-135347.mp4
ScreenRecording_07-08-2025.13-51-58_1.MP4
ScreenRecording_07-08-2025.21-32-41_1.MP4
screen-20250709-064245.mp4
Changelog
CHANGELOG entry:
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist