Skip to content

fix(accounts): vertically align row arrows on account details page - #46822

Merged
yanrong-chen merged 1 commit into
mainfrom
cursor/align-account-details-arrow-0247
Sep 30, 2026
Merged

yanrong-chen merged 1 commit into
mainfrom
cursor/align-account-details-arrow-0247

Conversation

@yanrong-chen

@yanrong-chen yanrong-chen commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Description

On the account details page, the > arrow at the end of each row sat slightly higher than the text next to it. This PR centres the arrows so they line up with the text in every row, including Account name, Networks, Private keys, Smart Account, Wallet, Secret Recovery Phrase and Remove account.

Changelog

CHANGELOG entry: Fixed the vertical alignment of the arrow icons on the account details page.

Manual testing steps

  1. Run the extension and unlock a wallet.
  2. Open the account menu and go to an account's details page.
  3. Verify the > arrow in each row is vertically centred with the row's text.

Screenshots/Recordings

Before

Screenshot 2026-09-30 at 1 42 40 PM Screenshot 2026-09-30 at 1 56 34 PM

After

Screenshot 2026-09-30 at 2 30 13 PM Screenshot 2026-09-30 at 2 30 21 PM

Pre-merge author checklist

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.
Open in Web Open in Cursor 

The arrow ButtonIcon in each row was wrapped in a block-level Box. Since
ButtonIcon is inline-flex, it sat on the wrapper's text baseline, adding
descender space below it and pushing the arrow above the row's vertical
centre relative to the value text.

Render the ButtonIcon directly inside the already flex/items-center value
container so it is centred with the text.

Co-authored-by: Yanrong Chen <yanrong-chen@users.noreply.github.com>
@metamask-ci

metamask-ci Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

✨ Files requiring CODEOWNER review ✨

🔑 @MetaMask/accounts-engineers (2 files, +55 -62)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain-accounts/
        • 📁 multichain-srp-backup/
          • 📄 multichain-srp-backup.tsx +7 -8
    • 📁 pages/
      • 📁 multichain-accounts/
        • 📁 multichain-account-details-page/
          • 📄 multichain-account-details-page.tsx +48 -54

@metamask-ci

metamask-ci Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor
Builds ready [581a3a5]
⚡ Performance Benchmarks (Total: 🟢 8 pass · 🟡 3 warn · 🔴 2 fail)

Baseline (latest main): 171ed20 | Date: 7/28/2026 | Pipeline: 36775702107 | Baseline logs

Metricschrome-webpackfirefox-webpack
loadNewAccount
[Sentry log · main/release]
🔴 load_new_account(p95) [CI log]🔴 load_new_account(p95) [CI log]

Regressions (🔴 2 failures)

Interaction Benchmarks · Samples: 5 🔴 2
Benchmarkchrome-webpackfirefox-webpack
loadNewAccount
[Sentry log · main/release]
🔴 [CI log]
🔴 load_new_account
🔴 [CI log]
🔴 load_new_account
confirmTx
[Sentry log · main/release]
🟡 [CI log]🟢 [CI log]
bridgeUserActions
[Sentry log · main/release]
🟡 [CI log]🟢 [CI log]

📈 Results compared to the previous 5 runs on main

  • ↑ loadNewAccount/load_new_account: +418%
  • ↑ loadNewAccount/total: +418%
  • ↑ confirmTx/longTaskCount: +33%
  • ↑ confirmTx/longTaskTotalDuration: +67%
  • ↑ confirmTx/longTaskMaxDuration: +78%
  • ↑ confirmTx/tbt: +92%
  • ↓ confirmTx/inp: -17%
  • ↑ bridgeUserActions/bridge_load_page: +160%
  • ↑ bridgeUserActions/bridge_load_asset_picker: +38%
  • ↑ bridgeUserActions/longTaskCount: +67%
  • ↑ bridgeUserActions/longTaskTotalDuration: +121%
  • ↑ bridgeUserActions/longTaskMaxDuration: +34%
  • ↑ bridgeUserActions/tbt: +239%
  • ↑ bridgeUserActions/total: +12%
  • ↑ loadNewAccount/load_new_account: +422%
  • ↑ loadNewAccount/total: +422%
  • ↓ loadNewAccount/inp: -24%
  • ↓ loadNewAccount/fcp: -45%
  • ↑ loadNewAccount/lcp: +1130%
  • ↓ confirmTx/longTaskCount: -100%
  • ↓ confirmTx/longTaskTotalDuration: -100%
  • ↓ confirmTx/longTaskMaxDuration: -100%
  • ↓ confirmTx/tbt: -100%
  • ↓ confirmTx/inp: -24%
  • ↓ confirmTx/fcp: -45%
  • ↑ confirmTx/lcp: +1164%
  • ↑ bridgeUserActions/bridge_load_page: +319%
  • ↑ bridgeUserActions/bridge_load_asset_picker: +81%
  • ↓ bridgeUserActions/longTaskCount: -100%
  • ↓ bridgeUserActions/longTaskTotalDuration: -100%
  • ↓ bridgeUserActions/longTaskMaxDuration: -100%
  • ↓ bridgeUserActions/tbt: -100%
  • ↑ bridgeUserActions/total: +45%
  • ↑ bridgeUserActions/inp: +38%
  • ↑ bridgeUserActions/lcp: +1082%

🌐 Core Web Vitals — 🟢 good · 🟡 needs improvement · 🔴 poor (web.dev thresholds)

  • 🟡 loadNewAccount/FCP: p75 1.9s
  • 🟡 confirmTx/FCP: p75 1.9s
  • 🟡 bridgeUserActions/FCP: p75 1.9s
Startup Benchmarks · Samples: 100
Benchmarkchrome-webpackfirefox-webpack
startupStandardHome
[Sentry log · main/release]
🟢 [CI log]🟢 [CI log]

📈 Results compared to the previous 5 runs on main

  • ↑ startupStandardHome/uiStartup: +14%
  • ↑ startupStandardHome/load: +15%
  • ↑ startupStandardHome/domContentLoaded: +14%
  • ↑ startupStandardHome/domInteractive: +18%
  • ↑ startupStandardHome/firstPaint: +16%
  • ↓ startupStandardHome/firstReactRender: -99%
  • ↓ startupStandardHome/initialActions: -33%
  • ↑ startupStandardHome/loadScripts: +14%
  • ↑ startupStandardHome/setupStore: +52%
  • ↑ startupStandardHome/numNetworkReqs: +10%
  • ↑ startupStandardHome/longTaskMaxDuration: +13%
  • ↑ startupStandardHome/tbt: +15%
  • ↑ startupStandardHome/inp: +14%
  • ↑ startupStandardHome/fcp: +15%
  • ↑ startupStandardHome/lcp: +18%
  • ↓ startupStandardHome/domInteractive: -34%
  • ↓ startupStandardHome/firstReactRender: -98%
  • ↓ startupStandardHome/initialActions: -50%
  • ↑ startupStandardHome/setupStore: +169%
  • ↓ startupStandardHome/fcp: -30%
User Journey Benchmarks · Samples: 5 · mock API
Benchmarkchrome-webpackfirefox-webpack
onboardingImportWallet
[Sentry log · main/release]
🟢 [CI log]🟢 [CI log]
onboardingNewWallet
[Sentry log · main/release]
🟢 [CI log]🟡 [CI log]
🟡 total

📈 Results compared to the previous 5 runs on main

  • ↑ onboardingImportWallet/confirmSrpToPwForm: +13%
  • ↓ onboardingImportWallet/doneButtonToHomeScreen: -94%
  • ↓ onboardingImportWallet/openAccountMenuToAccountListLoaded: -75%
  • ↓ onboardingImportWallet/longTaskCount: -86%
  • ↓ onboardingImportWallet/longTaskTotalDuration: -96%
  • ↓ onboardingImportWallet/longTaskMaxDuration: -93%
  • ↓ onboardingImportWallet/tbt: -100%
  • ↓ onboardingImportWallet/total: -89%
  • ↑ onboardingNewWallet/srpButtonToPwForm: +11%
  • ↓ onboardingNewWallet/doneButtonToAssetList: -83%
  • ↓ onboardingNewWallet/longTaskCount: -100%
  • ↓ onboardingNewWallet/longTaskTotalDuration: -100%
  • ↓ onboardingNewWallet/longTaskMaxDuration: -100%
  • ↓ onboardingNewWallet/tbt: -100%
  • ↓ onboardingNewWallet/total: -79%
Dapp Page Load Benchmarks · Samples: 100
Benchmarkchrome-webpack
dappPageLoad
[Sentry log · main/release]
🟢 [CI log]

📈 Results compared to the previous 5 runs on main

  • ↓ dappPageLoad/pageLoadTime: -66%
  • ↓ dappPageLoad/domContentLoaded: -22%
  • ↓ dappPageLoad/firstPaint: -50%
  • ↓ dappPageLoad/firstContentfulPaint: -50%
Bundle Size Diffs
Status Bundle Total Diff Change
✅ background 15.56 MiB +120 Bytes 0.00%
✅ ui 18.98 MiB -101 Bytes 0.00%
✅ common 0 Bytes 0 Bytes 0.00%
✅ other 1.69 MiB 0 Bytes 0.00%
✅ content scripts 1.97 MiB 0 Bytes 0.00%
✅ zip 22.93 MiB +46 Bytes 0.00%

@sonarqubecloud

Copy link
Copy Markdown

@yanrong-chen
yanrong-chen marked this pull request as ready for review September 30, 2026 21:31
@yanrong-chen
yanrong-chen requested a review from a team as a code owner September 30, 2026 21:31
@yanrong-chen
yanrong-chen added this pull request to the merge queue Sep 30, 2026
Merged via the queue into main with commit 3c658fb Sep 30, 2026
158 of 159 checks passed
@yanrong-chen
yanrong-chen deleted the cursor/align-account-details-arrow-0247 branch September 30, 2026 23:47
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 30, 2026
@metamaskbot metamaskbot added the release-13.52.0 Issue or pull request that will be included in release 13.52.0 label Sep 30, 2026

This branch was successfully deployed

1 active deployment
pr-comment — 581a3a59 Deployed Sep 30, 2026 by yanrong-chen via identify-codeowners #84334
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.52.0 Issue or pull request that will be included in release 13.52.0 size-M team-accounts-framework Accounts team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants