Results 20 issues of Brian Bergeron

## **Description** Problem: With `Primary currency = Fiat`, the home page showed Ethereum's symbol as `USD` instead of `ETH`. ## **Related issues** ## **Manual testing steps** - Set primary currency...

team-extension-ux
team-assets
needs-assets-ux-review

## **Description** Fixes a race condition during token detection after switching networks. During the network switch, some controllers state are still on the old chain id, and others are on...

team-assets
needs-assets-ux-review

## **Description** Initial implementation of the new token details page. Introduces a chart.js dependency for graphing asset prices. ## **Related issues** Figma: https://www.figma.com/file/TfVzSMJA8KwpWX8TTWQ2iO/MMASSETS-15%3A-Asset-list-and-details?node-id=1719%3A48736&mode=dev ## **Manual testing steps** TODO ## **Screenshots/Recordings**...

needs-ux-ds-review
team-assets
needs-assets-ux-review

## **Description** Fixes an issue where users can't list certain NFTs because MetaMask crashes with: `Invalid data; should be 'approve' method, but instead is 'setApprovalForAll'` The repro we have for...

team-assets

## **Description** Fixes an issue where the NFT detection controller was running too many times. It would send an HTTP request to fetch NFTs when any setting changed. It should...

team-assets

## Explanation Fixes an issue where prices were incorrect for Mantle, the native token of chain 0x1388. This happens because the symbol is `MNT`, but that's also the symbol of...

## Explanation Note: This PR accompanies https://github.com/MetaMask/metamask-extension/pull/23507. Read it for more context. This PR changes the polling strategy for `TokenRatesController`. I want to start/stop polling as metamask is opened/closed, to...

## **Description** When switching to a network for the first time, MM checks whether a token list is available for the network. If not, the 3rd item is shown here:...

team-assets
needs-assets-ux-review

## **Description** Patches https://github.com/MetaMask/core/pull/4045, fixing an issue where Mantle's native currency is mispriced. [![Open in GitHub Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/24337?quickstart=1) ## **Related issues** ## **Manual testing steps** ## **Screenshots/Recordings** ### **Before** ### **After**...

team-assets
needs-assets-ux-review
asset-metadata

## **Description** Adds a setting toggle to show fiat $ conversions on testnets. It is off by default. It also has a bottom sheet friction informing users about the setting....

needs-qa
team-assets
Run Smoke E2E
needs-assets-ux-review