fix: multichain account list item

[montelaidev]

Description

This PR updates the account list item to support displaying multichain balances.

Related issues

Fixes https://github.com/MetaMask/accounts-planning/issues/425

Manual testing steps

This does not affect evm accounts, their balances and behaviour should be the same as before.

1. Add a BTC account
2. Click on the account picker
3. See that there are balances for BTC now.

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• 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.

[github-actions]

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.

[ccharly]

Maybe introduce a new useMultichainAccountBalance that uses the same logic than here: getBtcCachedBalance so we would use it there too

[montelaidev]

As discussed with @ccharly, this would be changed in the overview pr

[ccharly]

I was also able to test it, and this matches our design:

[codecov]

Codecov Report

Attention: Patch coverage is 93.93939% with 4 lines in your changes missing coverage. Please review.

Project coverage is 69.69%. Comparing base (9844d1a) to head (38b767b).
Report is 2 commits behind head on develop.

Files	Patch %	Lines
ui/hooks/useMultichainAccountTotalFiatBalance.ts	90.48%	2 Missing ⚠️
...app/transaction-list/transaction-list.component.js	50.00%	1 Missing ⚠️
ui/selectors/multichain.ts	80.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #25437      +/-   ##
===========================================
+ Coverage    69.65%   69.69%   +0.04%     
===========================================
  Files         1347     1349       +2     
  Lines        47808    47851      +43     
  Branches     13187    13200      +13     
===========================================
+ Hits         33299    33349      +50     
+ Misses       14509    14502       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [38b767b]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (46 ± 4 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	64	97	78	10	5
		domContentLoaded	8	13	10	1	0
		load	38	70	46	7	4
		domInteractive	8	13	10	1	0

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 4.76 KiB (0.07%)
• common: 0 Bytes (0.00%)