Skip to content

fix: Network Manager style polish #33968

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

gambinish
Copy link
Contributor

@gambinish gambinish commented Jun 27, 2025

Description

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

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.

Copy link
Contributor

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.

@gambinish gambinish changed the title fix: AssetListControlBar padding in activity tab fix: Network Manager style polish Jun 27, 2025
@gambinish gambinish requested a review from a team as a code owner June 27, 2025 18:17
@metamaskbot
Copy link
Collaborator

metamaskbot commented Jun 27, 2025

✨ Files requiring CODEOWNER review ✨

🖥️ @MetaMask/wallet-ux (3 files, +20 -14)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 network-manager/
          • 📁 components/
            • 📁 additional-networks-info/
              • 📄 additional-networks-info.test.tsx +9 -13
              • 📄 additional-networks-info.tsx +2 -1
            • 📄 index.scss +9 -0

@metamaskbot
Copy link
Collaborator

Builds ready [5ae154d]
UI Startup Metrics (1272 ± 81 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1272108814708113151408
load108793412717511381217
domContentLoaded108092812587511311209
domInteractive19146371935
firstPaint71784127143610891212
backgroundConnect84465812
firstReactRender24165472443
getState1564682229
initialActions001001
loadScripts861723101868908986
setupStore96273918
WebpackHomeuiStartup22141661279825023332708
load16891245209720718102050
domContentLoaded16791241208620518032044
domInteractive181294161459
firstPaint1636639165182311
backgroundConnect2710290382537
firstReactRender24891422117379403
getState1354761527
initialActions316134
loadScripts16761235207520418012029
setupStore1774162030
FirefoxBrowserifyHomeuiStartup13661202168310313951629
load1203104415218012491336
domContentLoaded1202104415218012491336
domInteractive1193547768123303
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213110142134
firstReactRender24213522529
getState11414317934
initialActions0024201
loadScripts1185102914977912311312
setupStore74213716
WebpackHomeuiStartup16101380205816717152010
load13921172169314215081656
domContentLoaded13911172169314215081656
domInteractive99343185991282
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22164052335
firstReactRender43327654549
getState1264071232
initialActions002111
loadScripts13731155167314114891633
setupStore85405821
Benchmark value 1273 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1087 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1080 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 24 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 861 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1408 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1217 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1209 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1212 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 986 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2214 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 249 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 2709 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2051 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2045 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 59 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 403 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 2030 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 119 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 303 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 34 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 1392 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1392 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1373 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2010 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 282 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 1633 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 196ms | Sum of p95 exceeds: 911ms
Sum of all benchmark exceeds: 1107ms

@metamaskbot
Copy link
Collaborator

Builds ready [58d62cc]
UI Startup Metrics (1183 ± 62 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1183107014066212251291
load101290911535610531105
domContentLoaded100590411395510461092
domInteractive17145261631
firstPaint690129114640010311093
backgroundConnect84274816
firstReactRender23165262439
getState1665192134
initialActions001001
loadScripts79970492854836886
setupStore95264820
WebpackHomeuiStartup22321776282321523392685
load16971334219418817992029
domContentLoaded16911330218318717892013
domInteractive181299151453
firstPaint1706975497187316
backgroundConnect22115482638
firstReactRender23492441119380409
getState184322431435
initialActions217135
loadScripts16881328217318617862002
setupStore267317521945
FirefoxBrowserifyHomeuiStartup13811186199213714231645
load1203105214759212581390
domContentLoaded1203105114759212571390
domInteractive1203644766133292
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2714287322381
firstReactRender24213532629
getState16518632978
initialActions002001
loadScripts1180103513888612391352
setupStore12421528720
WebpackHomeuiStartup16841416237820317612047
load14451221201816515381710
domContentLoaded14441221201716515381709
domInteractive103345077098284
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect251685112640
firstReactRender43304944648
getState2042503912107
initialActions002111
loadScripts14231203199516415201691
setupStore13625727940
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 34 exceeds gate value 33 for chrome browserify home p95 getState
Benchmark value 20 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2232 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 2686 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2014 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 409 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 2002 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 120 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 28 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 16 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 292 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 81 exceeds gate value 70 for firefox browserify home p95 backgroundConnect
Benchmark value 78 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 1685 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1445 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1445 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 103 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 44 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 21 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 1424 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2047 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1710 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1709 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 284 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 107 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 1691 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 40 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 342ms | Sum of p95 exceeds: 965ms
Sum of all benchmark exceeds: 1307ms

@metamaskbot
Copy link
Collaborator

Builds ready [7eaee3c]
UI Startup Metrics (1182 ± 54 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1182104913185412171281
load100991111325110411098
domContentLoaded100390611225110341093
domInteractive18144661737
firstPaint63880112840110141094
backgroundConnect74294811
firstReactRender25175872640
getState16680122132
initialActions001001
loadScripts79770591651833892
setupStore95375920
WebpackHomeuiStartup22021755268720923072647
load16861302221318417732001
domContentLoaded16801299219818317691987
domInteractive171257111452
firstPaint1716675096178331
backgroundConnect21124582340
firstReactRender24088434121381411
getState164273351231
initialActions218134
loadScripts16771297218818217671975
setupStore177304291625
FirefoxBrowserifyHomeuiStartup14381231202315414941802
load1249108615579813201446
domContentLoaded1248108615579813191445
domInteractive1263755496118343
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect241598112540
firstReactRender26225042631
getState175255371049
initialActions002001
loadScripts1228107015399712961420
setupStore135169231160
WebpackHomeuiStartup16031388272521116911973
load13701188186514914821652
domContentLoaded13691188186414914821652
domInteractive95323295393265
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2516165172545
firstReactRender42356544449
getState165234251335
initialActions102111
loadScripts13481170183814814581633
setupStore13526134828
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 20 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2203 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 240 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 2648 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 411 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 1976 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1438 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1249 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1249 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 127 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 26 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 17 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 13 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1802 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 343 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 49 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 60 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 16 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 1973 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 265 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 35 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 1633 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 101ms | Sum of p95 exceeds: 745ms
Sum of all benchmark exceeds: 846ms

cursor[bot]

This comment was marked as outdated.

@metamaskbot
Copy link
Collaborator

Builds ready [c9a85b4]
UI Startup Metrics (1170 ± 59 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1170104113285912081285
load100290511184910301089
domContentLoaded99589911074810221080
domInteractive17135871641
firstPaint682150109638910171054
backgroundConnect84274815
firstReactRender22165172239
getState1465282128
initialActions001001
loadScripts79470290748821877
setupStore84263816
WebpackHomeuiStartup22101670284422223312640
load17061255217618317932014
domContentLoaded17001250216218217892001
domInteractive1912209231362
firstPaint1736672088183325
backgroundConnect2512280372341
firstReactRender21888472112358397
getState1146481226
initialActions216134
loadScripts16951249215118117851990
setupStore1475571623
FirefoxBrowserifyHomeuiStartup14481245192513415061741
load1256110915749113351418
domContentLoaded1254110915739013261403
domInteractive1263845871135328
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2815180232678
firstReactRender26226552632
getState196181331089
initialActions001001
loadScripts1232108815488913011379
setupStore1049213934
WebpackHomeuiStartup15801378233119516772011
load13521190178514014521639
domContentLoaded13511190178414014521639
domInteractive97324586886280
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2715228272557
firstReactRender41326144347
getState195251341288
initialActions101011
loadScripts13301172176613914321621
setupStore13526033831
Benchmark value 2210 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 2640 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 63 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 397 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 1991 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1449 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1257 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1255 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 127 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 29 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 27 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 19 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 1233 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1741 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 328 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 78 exceeds gate value 70 for firefox browserify home p95 backgroundConnect
Benchmark value 89 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 34 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 27 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 41 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 20 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2011 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 280 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 57 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Benchmark value 88 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 31 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 136ms | Sum of p95 exceeds: 801ms
Sum of all benchmark exceeds: 937ms

cursor[bot]

This comment was marked as outdated.

@metamaskbot
Copy link
Collaborator

❌ test-e2e-chrome-api-specs failed. View the html report here.

@metamaskbot
Copy link
Collaborator

Builds ready [83b79d2]
UI Startup Metrics (1172 ± 71 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1172103615397112181286
load99490311625110321077
domContentLoaded98889811535110261070
domInteractive18135181742
firstPaint6427311263889971065
backgroundConnect74333811
firstReactRender22164252436
getState1663872130
initialActions001001
loadScripts78369795150821862
setupStore95294922
WebpackHomeuiStartup22801820293923123932775
load17161255220019618502069
domContentLoaded17091250217619418432057
domInteractive1912134171462
firstPaint1656743574190330
backgroundConnect3010355433049
firstReactRender23491428113363408
getState194336341754
initialActions318135
loadScripts17061248216519318412046
setupStore267323472250
FirefoxBrowserifyHomeuiStartup14361243226014814921712
load1240108915458813131377
domContentLoaded1240108915458813131377
domInteractive1173643759121275
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect261384152766
firstReactRender26224232632
getState176328431058
initialActions002001
loadScripts1219107015248712901356
setupStore17419134882
WebpackHomeuiStartup16411396218220017272088
load14121197186716015251701
domContentLoaded14121197186716015241700
domInteractive111374948295302
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22155462434
firstReactRender42316954448
getState145197191331
initialActions102111
loadScripts13931178184816015081679
setupStore12526126827
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 42 exceeds gate value 41 for chrome browserify home p95 domInteractive
Benchmark value 22 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2280 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1716 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1709 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 1706 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2775 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2070 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2058 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 62 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 408 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 2046 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1437 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1241 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 118 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 27 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 26 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 17 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 17 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1712 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 275 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 58 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 82 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1641 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1413 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1412 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 111 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 42 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1393 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2088 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1701 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1700 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 302 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 1679 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 304ms | Sum of p95 exceeds: 1189ms
Sum of all benchmark exceeds: 1493ms

@gambinish gambinish merged commit 978e30d into feat/network-manager-UI-2 Jun 30, 2025
263 of 269 checks passed
@gambinish gambinish deleted the fix/network-manager-design-polish branch June 30, 2025 20:14
@github-actions github-actions bot locked and limited conversation to collaborators Jun 30, 2025
@gambinish
Copy link
Contributor Author

Bug: Test Suite Broken by it.only Modifier

The it.only modifier was accidentally committed to a test, causing only that specific test to run and skipping all others in the suite. This breaks full test execution and is typically used for local debugging.

ui/components/multichain/network-manager/components/additional-networks-info/additional-networks-info.test.tsx#L44-L45
Fix in Cursor

Was this report helpful? Give feedback by reacting with 👍 or 👎

👍

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

Successfully merging this pull request may close these issues.

2 participants