chore: Refine hidden accounts icon
Description
This PR cleans up the icon for hiding accounts for consistency. It polishes the UI.
Changelog
CHANGELOG entry: null
Related issues
Fixes:
- https://consensyssoftware.atlassian.net/browse/MDP-446?atlOrigin=eyJpIjoiZjc5OGFhYmU3ZTc3NDBmYzgyZjNiYWI3ZDVkYzc0YWEiLCJwIjoiaiJ9
- https://consensyssoftware.atlassian.net/browse/MDP-447?atlOrigin=eyJpIjoiZGUxYjg5ODdhYmM5NGUxNGIwMjk0MDA5MjAxYTU1NWYiLCJwIjoiaiJ9
Manual testing steps
Ensure that icon aligns and size matches
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.
[!NOTE] Standardizes hidden-accounts section toggles to medium, alternative-colored arrows and increases right padding on the multichain list header.
- UI/UX
- Hidden accounts toggles:
- In
ui/components/multichain-accounts/multichain-account-list/multichain-account-list.tsxandui/components/multichain/account-list-menu/hidden-account-list.js:
- Update
Iconsize fromIconSize.SmtoIconSize.Mdand color fromIconColor.iconDefaulttoIconColor.iconAlternativeforIconName.ArrowUp/ArrowDown.- In
multichain-account-list.tsx: increasepaddingRighton hidden header from4to6.Written by Cursor Bugbot for commit c0675b76836ab255a5a43da17c7e24b803a2c246. This will update automatically on new commits. Configure here.
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.
✨ Files requiring CODEOWNER review ✨
🔑 @MetaMask/accounts-engineers (2 files, +5 -5)
- 📁 ui/
- 📁 components/
- 📁 multichain/
- 📁 account-list-menu/
- 📄
hidden-account-list.js+2 -2
- 📄
- 📁 account-list-menu/
- 📁 multichain-accounts/
- 📁 multichain-account-list/
- 📄
multichain-account-list.tsx+3 -3
- 📄
- 📁 multichain-account-list/
- 📁 multichain/
- 📁 components/
👨🔧 @MetaMask/core-extension-ux (1 files, +2 -2)
- 📁 ui/
- 📁 components/
- 📁 multichain/
- 📁 account-list-menu/
- 📄
hidden-account-list.js+2 -2
- 📄
- 📁 account-list-menu/
- 📁 multichain/
- 📁 components/
Builds ready [c0675b7]
- builds: chrome, firefox
- builds (beta): chrome, firefox
- builds (flask): chrome, firefox
- builds (test): chrome, firefox
- builds (test-flask): chrome, firefox
- bundle size: Bundle Size Stats
- user-actions-benchmark: User Actions Stats
- storybook: Storybook
- typescript migration: Dashboard
- all artifacts
UI Startup Metrics (1306 ± 100 ms)
| Platform | BuildType | Page | Metric | Mean (ms) | Min (ms) | Max (ms) | Std Dev (ms) | P 75 (ms) | P 95 (ms) |
|---|---|---|---|---|---|---|---|---|---|
| Chrome | Browserify | Standard Home | uiStartup | 1306 | 1077 | 1546 | 100 | 1364 | 1481 |
| load | 1073 | 880 | 1329 | 92 | 1123 | 1250 | |||
| domContentLoaded | 1066 | 874 | 1322 | 92 | 1116 | 1238 | |||
| domInteractive | 24 | 16 | 59 | 9 | 24 | 43 | |||
| firstPaint | 525 | 84 | 1251 | 411 | 1052 | 1141 | |||
| backgroundConnect | 222 | 197 | 262 | 13 | 229 | 249 | |||
| firstReactRender | 49 | 32 | 94 | 14 | 56 | 78 | |||
| getState | 36 | 16 | 89 | 13 | 44 | 61 | |||
| initialActions | 1 | 0 | 5 | 1 | 1 | 3 | |||
| loadScripts | 852 | 678 | 1101 | 88 | 903 | 1021 | |||
| setupStore | 13 | 7 | 29 | 5 | 15 | 24 | |||
| numNetworkReqs | 8 | 6 | 30 | 6 | 6 | 29 | |||
| Browserify | Power User Home | uiStartup | 1882 | 1643 | 2180 | 118 | 1970 | 2090 | |
| load | 1080 | 909 | 1284 | 90 | 1151 | 1220 | |||
| domContentLoaded | 1069 | 903 | 1275 | 90 | 1129 | 1215 | |||
| domInteractive | 26 | 16 | 64 | 9 | 28 | 43 | |||
| firstPaint | 584 | 98 | 1230 | 427 | 1057 | 1177 | |||
| backgroundConnect | 250 | 205 | 663 | 86 | 243 | 537 | |||
| firstReactRender | 49 | 39 | 73 | 7 | 51 | 64 | |||
| getState | 194 | 141 | 289 | 31 | 212 | 258 | |||
| initialActions | 1 | 0 | 4 | 1 | 1 | 2 | |||
| loadScripts | 854 | 684 | 1046 | 89 | 913 | 1002 | |||
| setupStore | 17 | 7 | 59 | 8 | 15 | 38 | |||
| numNetworkReqs | 67 | 58 | 109 | 11 | 67 | 102 | |||
| Webpack | Standard Home | uiStartup | 828 | 699 | 1127 | 64 | 861 | 944 | |
| load | 641 | 565 | 801 | 57 | 677 | 737 | |||
| domContentLoaded | 636 | 561 | 791 | 57 | 672 | 733 | |||
| domInteractive | 21 | 15 | 38 | 6 | 22 | 37 | |||
| firstPaint | 213 | 71 | 801 | 169 | 194 | 676 | |||
| backgroundConnect | 13 | 5 | 70 | 10 | 14 | 32 | |||
| firstReactRender | 52 | 32 | 299 | 39 | 46 | 151 | |||
| getState | 24 | 13 | 46 | 9 | 30 | 43 | |||
| initialActions | 1 | 0 | 3 | 1 | 1 | 1 | |||
| loadScripts | 633 | 558 | 789 | 57 | 670 | 731 | |||
| setupStore | 10 | 6 | 36 | 5 | 11 | 20 | |||
| numNetworkReqs | 8 | 6 | 33 | 6 | 6 | 29 | |||
| Webpack | Power User Home | uiStartup | 1458 | 1161 | 1988 | 162 | 1599 | 1750 | |
| load | 725 | 578 | 1082 | 92 | 790 | 897 | |||
| domContentLoaded | 718 | 572 | 1065 | 91 | 780 | 888 | |||
| domInteractive | 25 | 15 | 81 | 10 | 26 | 47 | |||
| firstPaint | 286 | 90 | 1077 | 220 | 237 | 772 | |||
| backgroundConnect | 60 | 7 | 568 | 127 | 41 | 537 | |||
| firstReactRender | 48 | 40 | 86 | 6 | 49 | 60 | |||
| getState | 191 | 142 | 336 | 40 | 225 | 259 | |||
| initialActions | 1 | 0 | 2 | 0 | 1 | 1 | |||
| loadScripts | 715 | 570 | 1062 | 90 | 778 | 886 | |||
| setupStore | 15 | 9 | 68 | 9 | 14 | 34 | |||
| numNetworkReqs | 66 | 58 | 110 | 10 | 66 | 95 | |||
| Firefox | Browserify | Standard Home | uiStartup | 1254 | 1101 | 1674 | 121 | 1326 | 1470 |
| load | 1029 | 922 | 1359 | 73 | 1072 | 1157 | |||
| domContentLoaded | 1028 | 917 | 1359 | 73 | 1072 | 1156 | |||
| domInteractive | 55 | 34 | 212 | 30 | 73 | 102 | |||
| firstPaint | - | - | - | - | - | - | |||
| backgroundConnect | 35 | 19 | 182 | 21 | 35 | 69 | |||
| firstReactRender | 38 | 31 | 144 | 13 | 38 | 54 | |||
| getState | 10 | 6 | 25 | 4 | 10 | 22 | |||
| initialActions | 1 | 0 | 5 | 1 | 2 | 2 | |||
| loadScripts | 1006 | 906 | 1300 | 65 | 1044 | 1122 | |||
| setupStore | 13 | 6 | 282 | 27 | 10 | 22 | |||
| numNetworkReqs | 8 | 6 | 26 | 5 | 7 | 24 | |||
| Browserify | Power User Home | uiStartup | 2757 | 1598 | 4143 | 619 | 3232 | 3586 | |
| load | 1735 | 998 | 2707 | 573 | 2260 | 2498 | |||
| domContentLoaded | 1734 | 997 | 2707 | 574 | 2260 | 2498 | |||
| domInteractive | 316 | 34 | 1433 | 437 | 950 | 1112 | |||
| firstPaint | - | - | - | - | - | - | |||
| backgroundConnect | 142 | 22 | 1201 | 165 | 161 | 345 | |||
| firstReactRender | 56 | 34 | 120 | 14 | 59 | 94 | |||
| getState | 144 | 72 | 339 | 53 | 167 | 259 | |||
| initialActions | 2 | 1 | 7 | 1 | 2 | 3 | |||
| loadScripts | 1684 | 972 | 2569 | 557 | 2176 | 2447 | |||
| setupStore | 69 | 5 | 1109 | 160 | 58 | 304 | |||
| numNetworkReqs | 70 | 43 | 151 | 18 | 67 | 118 | |||
| Webpack | Standard Home | uiStartup | 1556 | 1389 | 1861 | 100 | 1619 | 1744 | |
| load | 1271 | 1133 | 1553 | 65 | 1308 | 1388 | |||
| domContentLoaded | 1271 | 1132 | 1553 | 65 | 1308 | 1388 | |||
| domInteractive | 70 | 30 | 302 | 41 | 85 | 136 | |||
| firstPaint | - | - | - | - | - | - | |||
| backgroundConnect | 48 | 17 | 129 | 22 | 52 | 107 | |||
| firstReactRender | 45 | 36 | 116 | 12 | 47 | 59 | |||
| getState | 12 | 7 | 48 | 6 | 14 | 19 | |||
| initialActions | 1 | 1 | 3 | 1 | 2 | 2 | |||
| loadScripts | 1245 | 1117 | 1528 | 64 | 1279 | 1362 | |||
| setupStore | 13 | 7 | 52 | 6 | 14 | 20 | |||
| numNetworkReqs | 8 | 6 | 28 | 6 | 7 | 25 | |||
| Webpack | Power User Home | uiStartup | 2766 | 1769 | 3681 | 573 | 3222 | 3471 | |
| load | 1867 | 1141 | 2946 | 551 | 2373 | 2561 | |||
| domContentLoaded | 1866 | 1141 | 2946 | 551 | 2373 | 2561 | |||
| domInteractive | 283 | 29 | 1590 | 418 | 234 | 1061 | |||
| firstPaint | - | - | - | - | - | - | |||
| backgroundConnect | 132 | 25 | 1230 | 204 | 144 | 407 | |||
| firstReactRender | 57 | 39 | 122 | 16 | 59 | 101 | |||
| getState | 128 | 80 | 356 | 50 | 151 | 236 | |||
| initialActions | 2 | 0 | 3 | 1 | 2 | 3 | |||
| loadScripts | 1799 | 1120 | 2916 | 536 | 2294 | 2525 | |||
| setupStore | 22 | 4 | 481 | 50 | 20 | 62 | |||
| numNetworkReqs | 70 | 55 | 134 | 16 | 68 | 117 |
📊 Page Load Benchmark Results
Current Commit: c0675b7 | Date: 12/9/2025
📄 Localhost MetaMask Test Dapp
Samples: 100
Summary
- pageLoadTime-> current mean value: 1.05s (±45ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
- domContentLoaded-> current mean value: 732ms (±39ms) 🟢 | historical mean value: 734ms ⬇️ (historical data)
- firstContentfulPaint-> current mean value: 75ms (±13ms) 🟢 | historical mean value: 79ms ⬇️ (historical data)
📈 Detailed Results
| Metric | Mean | Std Dev | Min | Max | P95 | P99 |
|---|---|---|---|---|---|---|
| pageLoadTime | 1.05s | 45ms | 1.02s | 1.37s | 1.09s | 1.37s |
| domContentLoaded | 732ms | 39ms | 705ms | 1.05s | 753ms | 1.05s |
| firstPaint | 75ms | 13ms | 56ms | 188ms | 88ms | 188ms |
| firstContentfulPaint | 75ms | 13ms | 56ms | 188ms | 88ms | 188ms |
| largestContentfulPaint | 0ms | 0ms | 0ms | 0ms | 0ms | 0ms |
Bundle size diffs [🚨 Warning! Bundle size has increased!]
- background: 58 Bytes (0%)
- ui: 8 Bytes (0%)
- common: 20 Bytes (0%)