MetaMask / MetaMask/metamask-mobile
The balance 'hide' icon can be removed for simplicity
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
What is this about?
Remove eye icon for hiding. Instead, allow for hiding upon tapping balance.
Rationale:
Historically, we’ve prioritized overtly discoverable design affordances. But in cases where interactions are intuitive or follow familiar industry patterns, we can scale back. This is one of those cases—most crypto apps either don’t include a hide feature or simply hide the balance when the value is tapped
Scenario
No response
Design
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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked Figma design for the Mobile Home balance interaction, then locate the existing balance hide icon and its current tap behavior in the mobile home screen. Done means the eye icon is removed and tapping the balance toggles hiding as shown in the design, with relevant tests still passing.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- design, frontend, mobile
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100