MetaMask / MetaMask/metamask-mobile
[UAT Parking lot] - Sep 12
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
### What is this about?
Build: v7.56.0 (2392)
Device: iOS 18.6
UI nits:
1. Duplicate receive functionality on the main wallet screen
2. Missing left/right padding for networks on the Enable smart account screen
3. The spacing for checkmarks on the account selection screen seems a bit awkward with long strings and duplicates functionality of the outline around the avatar. Maybe consider removing it? Also the font weight for the name and balance looks heavier than what we normally use in the app for list items
4. When searching for an account the mobile keyboard blocks the account list, which makes for a poor experience. I have to hit return to dismiss the keyboard in order to be able to select the account I am searching for.
https://github.com/user-attachments/assets/8858f7da-f7a6-4269-9e42-8d7ec127dda2
5. On the import SRP screen, its a bit confusing that we use a down carrot to be able to toggle between 12 and 24 word phrases. Morever, we are displaying this option as a modal vs. a bottom sheet, which is a more common pattern throughout the app. Also the spacing between this toggle and the hide icon (eyeball) seems very tight. Should have at least 44px tap target area for both icons.
### Scenario
_No response_
### Design
_No response_
### Technical Details
_No response_
### Threat Modeling Framework
_No response_
### Acceptance Criteria
_No response_
### Stakeholder review needed before the work gets merged
- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)
### References
_No response_
Contributor guide
Assessment
This issue has not been assessed yet.