metamask-extension
metamask-extension copied to clipboard
fix: asset picker bug
Description
the goal of this PR is to fix some v12.1 bugs related to balance display and asset picker modal:
- The symbols no longer show for each list item
- Blocked token symbols (e.g., BNB on Ethereum Mainnet) are no longer greyed out in the UI despite still being inert
- Token names are not truncated
- Fiat and crypto balance are inverted on the wallet view
- Token symbol is USD instead of ETH on the asset picker modal
Related issues
Fixes: #25575
Manual testing steps
- Go to the wallet view and check crypto and fiat value for native token
- Select Ethereum Mainnet
- Go to Send page
- Click on destination asset picker
- Observe that symbols are not present and names are not truncated
- Search for "CHZ"
- Observe that it is not styled as a disabled button
Screenshots/Recordings
Before
After
Pre-merge author checklist
- [x] I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using JSDoc format if applicable
- [x] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
Pre-merge reviewer checklist
- [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.