Skip to content

Conversation

@NidhiKJha
Copy link
Member

@NidhiKJha NidhiKJha commented Jul 17, 2025

This PR ensures that when Solana account is connected, it should render connected state in the dapp view

Related issues

Fixes: #34239

Manual testing steps

  1. Connect extension to Solana Dapp, check the connected popover renders correct network icon and connected state
  2. Connect extension to Uniswap, check the connected popover renders correct network icon and connected state

Screenshots/Recordings

Before

NA

After

Screen.Recording.2025-07-17.at.8.08.58.PM.mov

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.

@NidhiKJha NidhiKJha requested a review from a team as a code owner July 17, 2025 19:09
@metamaskbot metamaskbot added team-core-extension-ux Core Extension UX team INVALID-PR-TEMPLATE PR's body doesn't match template labels Jul 17, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Unintended Debugging Statement

A debugging console.log statement was accidentally left in the code and should be removed before production.

ui/components/multichain/connected-site-popover/connected-site-popover.tsx#L61-L62

const dispatch = useDispatch();
console.log(currentNetwork);

Fix in CursorFix in Web


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

@metamaskbot
Copy link
Collaborator

metamaskbot commented Jul 17, 2025

✨ Files requiring CODEOWNER review ✨

🖥️ @MetaMask/wallet-ux (3 files, +77 -8)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 connected-site-menu/
          • 📄 connected-site-menu.js +8 -2
        • 📁 connected-site-popover/
          • 📄 connected-site-popover.test.tsx +38 -0
          • 📄 connected-site-popover.tsx +31 -6

@metamaskbot
Copy link
Collaborator

Builds ready [575285e]
UI Startup Metrics (1281 ± 64 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1281115214786413241415
load109796512485611311198
domContentLoaded108993612435911261192
domInteractive18145371737
firstPaint67081125544710971167
backgroundConnect21519636817216228
firstReactRender20144652029
getState11484111426
initialActions40559218
loadScripts881730104059921985
setupStore74233713
WebpackHomeuiStartup23892051270812524862560
load18911417235915319952171
domContentLoaded18781408218314419722123
domInteractive1912118181562
firstPaint177671932190180332
backgroundConnect4693927432281
firstReactRender1868838987234371
getState144244241423
initialActions72599627
loadScripts18741403217814319692121
setupStore166198201734
FirefoxBrowserifyHomeuiStartup14871313228914015381728
load1288113617129313321457
domContentLoaded1288113617119313321457
domInteractive1163948382108319
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2816230232751
firstReactRender27205753033
getState10320722724
initialActions30305212
loadScripts1264112014948613121427
setupStore10417121732
WebpackHomeuiStartup18731621243320320182296
load15781369215017216591972
domContentLoaded15771369215017216591971
domInteractive101344507394330
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect332094133861
firstReactRender57467466067
getState143267281631
initialActions61749722
loadScripts15501347212016916321944
setupStore13581131441
Benchmark value 1282 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1098 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1089 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 216 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 882 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1416 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1199 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1192 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 229 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 18 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 985 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2389 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1891 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1879 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 47 exceeds gate value 40 for chrome webpack home mean backgroundConnect
Benchmark value 1874 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2561 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2171 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2124 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 281 exceeds gate value 90 for chrome webpack home p95 backgroundConnect
Benchmark value 371 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 27 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2122 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1487 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1289 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1289 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 117 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 28 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 4 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1265 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 1728 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 319 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 12 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 32 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1874 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1578 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1578 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 101 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 34 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 57 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 7 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1551 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 2296 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1972 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1971 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 330 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 61 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Benchmark value 67 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 22 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1944 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 41 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 2211ms | Sum of p95 exceeds: 2821.8ms
Sum of all benchmark exceeds: 5032.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 38 Bytes (0%)
  • ui: 1.89 KiB (0.03%)
  • common: 911 Bytes (0.01%)

@NidhiKJha NidhiKJha enabled auto-merge July 17, 2025 20:05
@metamaskbot
Copy link
Collaborator

Builds ready [e7787c4]
UI Startup Metrics (1268 ± 61 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1268112715106113031373
load108497112355411201173
domContentLoaded107796612275511131166
domInteractive18144681743
firstPaint70279117643211011150
backgroundConnect2101932377212226
firstReactRender20154242029
getState93417928
initialActions407411216
loadScripts874764101653910960
setupStore73224717
WebpackHomeuiStartup24671945270414125502651
load19651436225816320752232
domContentLoaded19561430224916120472226
domInteractive2113204221655
firstPaint1576340772190299
backgroundConnect3211273383054
firstReactRender1849234959216306
getState1143161324
initialActions10219727734
loadScripts19511426224216220422224
setupStore177211201732
FirefoxBrowserifyHomeuiStartup15261340244016315811834
load13091170219512613401507
domContentLoaded13091170219512613401507
domInteractive122351078122108340
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect271568102953
firstReactRender28218593038
getState14318033862
initialActions5118319318
loadScripts12861146218212513161478
setupStore12318626742
WebpackHomeuiStartup17451495269922018142241
load14681279194914815531824
domContentLoaded14671279194914815531824
domInteractive98333836593305
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect3317315393158
firstReactRender51426455560
getState143257331325
initialActions9122230720
loadScripts14391262192513815251758
setupStore155259311146
Benchmark value 1268 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1085 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1077 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 210 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 874 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1374 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 43 exceeds gate value 41 for chrome browserify home p95 domInteractive
Benchmark value 227 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 16 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 961 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2468 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1966 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1957 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 11 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 1952 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2651 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2232 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2227 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 34 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2224 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1526 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1309 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1309 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 122 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 29 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 15 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 6 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1287 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1834 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 1507 exceeds gate value 1495 for firefox browserify home p95 load
Benchmark value 1507 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded
Benchmark value 340 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 62 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 18 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 1478 exceeds gate value 1475 for firefox browserify home p95 loadScripts
Benchmark value 42 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1746 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1468 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1468 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 33 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 52 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 10 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1439 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 15 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2241 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1824 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1824 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 305 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 58 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Benchmark value 60 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 20 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1758 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 46 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 2115ms | Sum of p95 exceeds: 2538.8ms
Sum of all benchmark exceeds: 4653.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 38 Bytes (0%)
  • ui: 1.87 KiB (0.03%)
  • common: 911 Bytes (0.01%)

@NidhiKJha NidhiKJha added this pull request to the merge queue Jul 17, 2025
Merged via the queue into main with commit 30e2d2b Jul 17, 2025
144 of 145 checks passed
@NidhiKJha NidhiKJha deleted the solana-network branch July 17, 2025 20:43
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2025
@metamaskbot metamaskbot added the release-13.1.0 Issue or pull request that will be included in release 13.1.0 label Jul 17, 2025
@NidhiKJha NidhiKJha changed the title fix: show solana connected state fix: cp-13.0.0 show solana connected state Jul 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-13.1.0 Issue or pull request that will be included in release 13.1.0 team-core-extension-ux Core Extension UX team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]:[Remove GNS] Solana dapps displayed as not connected in MM

5 participants