Skip to content

Conversation

@runway-github
Copy link
Contributor

@runway-github runway-github bot commented Dec 1, 2025

Description

Fix layout inconsistencies in sidepanel and fullscreen modes

Also aligns "Fund your wallet"

Open in GitHub Codespaces

Changelog

CHANGELOG entry: fix: confirmation layout in sidepanel

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/CEUX-740

Manual testing steps

In sidepanel and fullscreen

  • Confirmations
  • Accounts list
  • Import SRP
  • Import PK

Screenshots/Recordings

Before

Confirmations Accounts
<img width="619" height="368" alt="image"
src="https://github.com/user-attachments/assets/840d484c-99ff-4ee0-bf20-21fa10d947a6"
/> <img width="834" height="292" alt="image"
src="https://github.com/user-attachments/assets/b0e3cba9-f849-4134-8235-0c7c9a433a7b"
/>

After

Confirmations Accounts
<img width="578" height="277" alt="image"
src="https://github.com/user-attachments/assets/bda115c2-5d5b-4102-b19b-ded9871c0423"
/> <img width="839" height="222" alt="image"
src="https://github.com/user-attachments/assets/4185ac6e-4426-4132-8a0a-3a11a6e76652"
/>

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.

Note

Unifies layouts across views by switching key routes to RootLayout, removing legacy width constraints, deleting unused styles, and tweaking empty-state styling.

  • Routing/Layout:
    • Switch IMPORT_SRP_ROUTE, CONFIRM_TRANSACTION_ROUTE, and CONFIRMATION_V_NEXT_ROUTE to RootLayout in ui/pages/routes/routes.component.tsx.
    • Remove MultichainMetaFoxLogo and isConfirmTransactionRoute usage from routes.
  • Page Containers:
    • Drop max-w-[600px] on multichain pages (add-wallet-page, multichain-account-address-list-page, multichain-account-private-key-list-page, smart-account-page) to allow full-width within layout.
  • Styles:
    • multi-srp/import-srp/index.scss: remove fixed max-width/height; rely on responsive behavior; keep floating import button responsive.
    • Delete multichain-accounts/account-list/account-list.scss and its import from pages.scss.
  • Wallet Overview:
    • BalanceEmptyState widened and centered (max-w-[460px] self-center) in coin-overview.tsx.

Written by Cursor Bugbot for commit 7a069b0. This will update automatically on new commits. Configure here.

35ba498

…38410)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**

Fix layout inconsistencies in sidepanel and fullscreen modes

Also aligns "Fund your wallet"

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/38410?quickstart=1)

## **Changelog**

<!--
If this PR is not End-User-Facing and should not show up in the
CHANGELOG, you can choose to either:
1. Write `CHANGELOG entry: null`
2. Label with `no-changelog`

If this PR is End-User-Facing, please write a short User-Facing
description in the past tense like:
`CHANGELOG entry: Added a new tab for users to see their NFTs`
`CHANGELOG entry: Fixed a bug that was causing some NFTs to flicker`

(This helps the Release Engineer do their job more quickly and
accurately)
-->

CHANGELOG entry: fix: confirmation layout in sidepanel

## **Related issues**

Fixes: https://consensyssoftware.atlassian.net/browse/CEUX-740

## **Manual testing steps**

In sidepanel and fullscreen

- Confirmations
- Accounts list
- Import SRP
- Import PK

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

| Confirmations | Accounts |
|---|---|
|<img width="619" height="368" alt="image"
src="https://github.com/user-attachments/assets/840d484c-99ff-4ee0-bf20-21fa10d947a6"
/>|<img width="834" height="292" alt="image"
src="https://github.com/user-attachments/assets/b0e3cba9-f849-4134-8235-0c7c9a433a7b"
/>|


### **After**

| Confirmations | Accounts |
|---|---|
|<img width="578" height="277" alt="image"
src="https://github.com/user-attachments/assets/bda115c2-5d5b-4102-b19b-ded9871c0423"
/>|<img width="839" height="222" alt="image"
src="https://github.com/user-attachments/assets/4185ac6e-4426-4132-8a0a-3a11a6e76652"
/>|


## **Pre-merge author checklist**

- [ ] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] 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](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
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.




<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Switches confirmations and Import SRP to RootLayout and removes fixed
widths/padding across multichain pages to align sidepanel/fullscreen
layouts.
> 
> - **Routing/Layout**:
> - Move `CONFIRM_TRANSACTION_ROUTE` and `CONFIRMATION_V_NEXT_ROUTE` to
`RootLayout`.
>   - Move `IMPORT_SRP_ROUTE` to `RootLayout`.
> - Remove `MultichainMetaFoxLogo` rendering on confirm routes and
related `isConfirmTransactionRoute` import.
> - **Multichain Account Pages**:
> - Drop fixed `max-w-[600px]` on `Page` wrappers in `add-wallet-page`,
`multichain-account-address-list-page`,
`multichain-account-private-key-list-page`, and `smart-account-page`.
>   - Remove `account-list` stylesheet and its import.
> - **Styles**:
> - Remove extra padding rule for `.multichain-page` when not using app
header.
> - Import SRP: remove fixed `max-width`/`height`; keep responsive
layout.
> - Wallet overview: widen and center `BalanceEmptyState`
(`max-w[460px]`, `self-center`).
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
1a5118e. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
@runway-github runway-github bot requested a review from a team as a code owner December 1, 2025 20:34
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

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.

@metamaskbot metamaskbot added the team-bots Bot team (for MetaMask Bot, Runway Bot, etc.) label Dec 1, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🔑 @MetaMask/accounts-engineers (5 files, +4 -11)
  • 📁 ui/
    • 📁 pages/
      • 📁 multichain-accounts/
        • 📁 account-list/
          • 📄 account-list.scss +0 -7
        • 📁 add-wallet-page/
          • 📄 add-wallet-page.tsx +1 -1
        • 📁 multichain-account-address-list-page/
          • 📄 multichain-account-address-list-page.tsx +1 -1
        • 📁 multichain-account-private-key-list-page/
          • 📄 multichain-account-private-key-list-page.tsx +1 -1
        • 📁 smart-account-page/
          • 📄 smart-account-page.tsx +1 -1

@metamaskbot
Copy link
Collaborator

Builds ready [6fb88fe]
📊 Page Load Benchmark Results

Current Commit: 6fb88fe | Date: 12/1/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±67ms) 🟡 | historical mean value: 1.05s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 736ms (±65ms) 🟢 | historical mean value: 732ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 76ms (±11ms) 🟢 | historical mean value: 80ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 67ms 1.02s 1.36s 1.25s 1.36s
domContentLoaded 736ms 65ms 703ms 1.01s 936ms 1.01s
firstPaint 76ms 11ms 60ms 168ms 84ms 168ms
firstContentfulPaint 76ms 11ms 60ms 168ms 84ms 168ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms

@metamaskbot
Copy link
Collaborator

Builds ready [7a069b0]
UI Startup Metrics (1248 ± 115 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12481016151411513401427
load1041874131110511171238
domContentLoaded1034869129610311141224
domInteractive2515101182278
firstPaint5259512434049921188
backgroundConnect21719725113225247
firstReactRender321978123662
getState3816180214583
initialActions106112
loadScripts82466211001018881013
setupStore1263251423
numNetworkReqs1257720573
BrowserifyPower User HomeuiStartup21411727288427423572679
load1047922152013310501398
domContentLoaded1031906151113410261391
domInteractive38181753134135
firstPaint52411115283869881161
backgroundConnect24520864869235476
firstReactRender5942112136591
getState217141768123207698
initialActions104112
loadScripts81869213101338151180
setupStore20114462330
numNetworkReqs1426939070195286
WebpackStandard HomeuiStartup8547171166838951019
load66057894977695831
domContentLoaded65557094276691822
domInteractive2816126232393
firstPaint22675835158210658
backgroundConnect13689121330
firstReactRender28204663339
getState271471103543
initialActions106113
loadScripts65256793374688815
setupStore1163841319
numNetworkReqs1257721576
WebpackPower User HomeuiStartup16881275267927319342150
load6835811213115690953
domContentLoaded6735751206116677946
domInteractive38182073636140
firstPaint268981061201246696
backgroundConnect91863818520564
firstReactRender59438396677
getState18312968888178217
initialActions104112
loadScripts6705731197114675936
setupStore21770122347
numNetworkReqs1517038769188306
FirefoxBrowserifyStandard HomeuiStartup13541104242018114461643
load1110929209313811701268
domContentLoaded1110924209213811701268
domInteractive72319629685134
firstPaint------
backgroundConnect50231563162122
firstReactRender24194052536
getState137100141227
initialActions102122
loadScripts1079914198912711281220
setupStore135111131135
numNetworkReqs1256716662
BrowserifyPower User HomeuiStartup25942014462451126583988
load1192974244125011721571
domContentLoaded1192973244025011721570
domInteractive12137579103113385
firstPaint------
backgroundConnect159321391233136731
firstReactRender60391872160100
getState24958848170344628
initialActions208127
loadScripts1128948232017711161499
setupStore1819903220164706
numNetworkReqs102633236280250
WebpackStandard HomeuiStartup15111282213716515911919
load12401088161811612981501
domContentLoaded12401088161711612971501
domInteractive52241653178123
firstPaint------
backgroundConnect54202083661127
firstReactRender27217872837
getState156196231343
initialActions103112
loadScripts12111072159010712731406
setupStore1364591337
numNetworkReqs1256816763
WebpackPower User HomeuiStartup29752125554175931634678
load14361155303937115102591
domContentLoaded14351155303937115102590
domInteractive100314458997351
firstPaint------
backgroundConnect143231069194147360
firstReactRender65402263460143
getState28759906214428762
initialActions3070823
loadScripts13591117287730214211715
setupStore16651058217181657
numNetworkReqs1026325052130242
📊 Page Load Benchmark Results

Current Commit: 7a069b0 | Date: 12/2/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.03s (±38ms) 🟡 | historical mean value: 1.04s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 720ms (±36ms) 🟢 | historical mean value: 729ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 77ms (±10ms) 🟢 | historical mean value: 81ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.03s 38ms 1.01s 1.31s 1.07s 1.31s
domContentLoaded 720ms 36ms 700ms 988ms 749ms 988ms
firstPaint 77ms 10ms 60ms 148ms 88ms 148ms
firstContentfulPaint 77ms 10ms 60ms 148ms 88ms 148ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms

@gauthierpetetin gauthierpetetin merged commit a81a0b1 into release/13.12.0 Dec 2, 2025
171 checks passed
@gauthierpetetin gauthierpetetin deleted the runway-cherry-pick-13.12.0-1764621255 branch December 2, 2025 09:55
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2025
@metamaskbot metamaskbot added the release-13.12.0 Issue or pull request that will be included in release 13.12.0 label Dec 2, 2025
@metamaskbot
Copy link
Collaborator

No release label on PR. Adding release label release-13.12.0 on PR, as PR was cherry-picked in branch 13.12.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.12.0 Issue or pull request that will be included in release 13.12.0 team-bots Bot team (for MetaMask Bot, Runway Bot, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants