Skip to content

Conversation

@ghgoodreau
Copy link
Contributor

@ghgoodreau ghgoodreau commented Jul 21, 2025

Description

This PR replaces the 'Bridge' copy in some cases to 'Swaps' where we missed it in the Unified Swaps experience.

Open in GitHub Codespaces

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

  1. Go to the unified swaps page.
  2. The 'bridge' label at the top should no longer be visible, instead, it should say 'swap'

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.

@github-actions
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.

@metamaskbot metamaskbot added the team-swaps-and-bridge Swaps and Bridge team label Jul 21, 2025
@metamaskbot
Copy link
Collaborator

metamaskbot commented Jul 21, 2025

✨ Files requiring CODEOWNER review ✨

🔄 @MetaMask/swaps-engineers (1 files, +11 -2)
  • 📁 ui/
    • 📁 pages/
      • 📁 bridge/
        • 📄 index.tsx +11 -2

@ghgoodreau ghgoodreau marked this pull request as ready for review July 21, 2025 16:10
@ghgoodreau ghgoodreau requested a review from a team as a code owner July 21, 2025 16:10
@metamaskbot
Copy link
Collaborator

Builds ready [d0b1652]
UI Startup Metrics (1262 ± 59 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1262112914855912981368
load108198212295311131182
domContentLoaded107397812225411101178
domInteractive18145081739
firstPaint63969120943110651142
backgroundConnect2121982397214230
firstReactRender21135062235
getState1036281127
initialActions50758617
loadScripts868777100852903977
setupStore74254817
WebpackHomeuiStartup23401815271013224282531
load18351358225314619262114
domContentLoaded18271352224114619172107
domInteractive2112203251562
firstPaint1596252374183307
backgroundConnect3310302463253
firstReactRender1798837279220344
getState133255251323
initialActions122178251035
loadScripts18221348222914619122096
setupStore216237371742
FirefoxBrowserifyHomeuiStartup14911300216613815431754
load1283113915058413371467
domContentLoaded1283113915058313361466
domInteractive1143942174112336
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect261678102848
firstReactRender27206782854
getState13316828639
initialActions508211220
loadScripts1261111914848213071438
setupStore13325233663
WebpackHomeuiStartup17701501247420218792172
load14891286188414915731845
domContentLoaded14881286188314915721844
domInteractive102373926495292
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect3118100143665
firstReactRender53427665762
getState923271327
initialActions619411621
loadScripts14641267184714615441814
setupStore2342555017211
Benchmark value 1263 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1081 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1074 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 212 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 869 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1368 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 231 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 17 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 977 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2340 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1836 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1827 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 22 exceeds gate value 21 for chrome webpack home mean domInteractive
Benchmark value 13 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 1823 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2532 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2114 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2108 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 63 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 35 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2097 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1491 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1284 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1283 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 115 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 27 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 13 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 5 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1262 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 13 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1754 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 336 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 39 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 20 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 63 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1770 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1489 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1489 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 31 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 54 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 1465 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 23 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2172 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1845 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1844 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 292 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 65 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Benchmark value 62 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 21 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1814 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 211 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 1563ms | Sum of p95 exceeds: 2154.8ms
Sum of all benchmark exceeds: 3717.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 2.18 KiB (0.04%)
  • ui: 3.13 KiB (0.04%)
  • common: 549 Bytes (0.01%)

SteP-n-s
SteP-n-s previously approved these changes Jul 22, 2025
@ghgoodreau ghgoodreau enabled auto-merge July 22, 2025 16:47
};

const isSwap = useIsMultichainSwap();
const chainId = useSelector(getCurrentChainId);
Copy link
Member

Choose a reason for hiding this comment

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

Does this return non-EVM chainIds as well?

@metamaskbot
Copy link
Collaborator

Builds ready [74a9050]
UI Startup Metrics (1281 ± 61 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1281115515186113191384
load109997612936311431215
domContentLoaded109197112866211341206
domInteractive18135281741
firstPaint61679119443710871153
backgroundConnect2131992427216224
firstReactRender20155152029
getState1043061325
initialActions30768211
loadScripts8847611077639281002
setupStore74243715
WebpackHomeuiStartup24271881603140524502840
load19301437550240619882267
domContentLoaded19201430548640219842241
domInteractive2212229341556
firstPaint1576545270174317
backgroundConnect59103669232297
firstReactRender1578836959173310
getState2032744912184
initialActions62233710
loadScripts19131426548140219822228
setupStore146234221419
FirefoxBrowserifyHomeuiStartup14731272218515115041715
load1268111814948113161433
domContentLoaded1268111714948113161433
domInteractive1033639358103266
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2617107132752
firstReactRender27206192959
getState10323526723
initialActions7019022344
loadScripts1246109914647912971413
setupStore13327533839
WebpackHomeuiStartup17821515242917018742107
load15171284193214216081813
domContentLoaded15171284193214216071813
domInteractive100393575799244
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2819120162659
firstReactRender52446755562
getState933671324
initialActions616710823
loadScripts14931264191113815791780
setupStore13525426834
Benchmark value 1282 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1099 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1092 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 214 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 885 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1384 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1215 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1207 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 225 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 11 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 1003 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2427 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1931 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1921 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 23 exceeds gate value 21 for chrome webpack home mean domInteractive
Benchmark value 60 exceeds gate value 40 for chrome webpack home mean backgroundConnect
Benchmark value 1913 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2840 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2267 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2241 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 297 exceeds gate value 90 for chrome webpack home p95 backgroundConnect
Benchmark value 10 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2228 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1473 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1269 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1268 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 27 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 8 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1246 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 14 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1715 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 266 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 59 exceeds gate value 55 for firefox browserify home p95 firstReactRender
Benchmark value 44 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 39 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1783 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1518 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1518 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 29 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 53 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 1494 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2107 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1813 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1813 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 244 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 59 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Benchmark value 62 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 23 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1780 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 34 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 2035ms | Sum of p95 exceeds: 2626.8ms
Sum of all benchmark exceeds: 4661.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 3.42 KiB (0.06%)
  • ui: 5.31 KiB (0.07%)
  • common: 6.28 KiB (0.07%)

@ghgoodreau ghgoodreau added this pull request to the merge queue Jul 22, 2025
Merged via the queue into main with commit 3ecc4d1 Jul 22, 2025
145 checks passed
@ghgoodreau ghgoodreau deleted the fix-unified-swap-header-label branch July 22, 2025 21:45
@github-actions github-actions bot locked and limited conversation to collaborators Jul 22, 2025
@metamaskbot metamaskbot added the release-13.1.0 Issue or pull request that will be included in release 13.1.0 label Jul 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.1.0 Issue or pull request that will be included in release 13.1.0 team-swaps-and-bridge Swaps and Bridge team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants