Skip to content

Conversation

@lionellbriones
Copy link
Contributor

@lionellbriones lionellbriones commented May 29, 2025

Description

New SRP review UI
https://www.figma.com/design/pViOUcmjwhEzFsdrwknpNc/Onboarding-redesign?node-id=6188-88752&t=KttJt1lvkqwslaMK-4

Screen.Recording.2025-06-11.at.4.23.05.PM.mov

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.

- update social login button radius
- allow closing terms of use modal
- toggle password button stop propagating
@metamaskbot
Copy link
Collaborator

Builds ready [42e483d]
UI Startup Metrics (1203 ± 60 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1203108313556012401307
load104793211805710801144
domContentLoaded104192611705810751138
domInteractive16133541628
firstPaint749130117841310691144
backgroundConnect74264821
firstReactRender20164252034
getState1463882031
initialActions001001
loadScripts79869392757831900
setupStore85183814
WebpackHomeuiStartup21571726255119722612532
load16841342195415817971931
domContentLoaded16781338194515717911926
domInteractive171276121349
firstPaint1586331452174264
backgroundConnect21114062336
firstReactRender14547380110207366
getState144321311321
initialActions315134
loadScripts16751336193415617891921
setupStore4573269019317
FirefoxBrowserifyHomeuiStartup1327118516208213801471
load1178105314547812281320
domContentLoaded1178105314547912281319
domInteractive1013218726111146
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2313139182248
firstReactRender23206042329
getState7436389
initialActions001001
loadScripts1157103613657512121302
setupStore74345611
WebpackHomeuiStartup1495135017519715681696
load13051164156210013661521
domContentLoaded13051164156210013661520
domInteractive79581731686110
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20152822124
firstReactRender40324734245
getState85304810
initialActions002111
loadScripts12861147154210113491502
setupStore7530379
Benchmark value 21 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 46 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2533 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 317 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 40 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 16ms | Sum of p95 exceeds: 334ms
Sum of all benchmark exceeds: 350ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@github-project-automation github-project-automation bot moved this from Needs dev review to Review finalised - Ready to be merged in PR review queue Jun 12, 2025
@nakulmk nakulmk disabled auto-merge June 12, 2025 14:15
@nakulmk nakulmk enabled auto-merge June 12, 2025 14:16
@metamaskbot
Copy link
Collaborator

Builds ready [0fd1c82]
UI Startup Metrics (1236 ± 71 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1236111214797112711363
load107095612526911261187
domContentLoaded106494112476911201181
domInteractive181497121729
firstPaint67284125543610681170
backgroundConnect84276825
firstReactRender20164542128
getState1464282029
initialActions001001
loadScripts81570598267870934
setupStore85303813
WebpackHomeuiStartup22161723269221623572587
load17341346210917718361990
domContentLoaded17281342210417618281981
domInteractive161262101347
firstPaint1596638253182268
backgroundConnect21114572537
firstReactRender15446383111295349
getState1244061426
initialActions315134
loadScripts17251340210217518261977
setupStore4373298720324
FirefoxBrowserifyHomeuiStartup1330119715197213701491
load1185106013687312401347
domContentLoaded1185106013677312391346
domInteractive973617127114138
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2114105122127
firstReactRender23202922328
getState7539389
initialActions001001
loadScripts1165104613486912201324
setupStore74364611
WebpackHomeuiStartup16201389190811817081833
load14091202168812115021623
domContentLoaded14091201168712115001622
domInteractive83571681590107
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect23175362443
firstReactRender44365434749
getState953041018
initialActions002111
loadScripts13891182166612114821601
setupStore86213914
Benchmark value 1236 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1071 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1064 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1181 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 25 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 2216 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1735 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1729 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 1726 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 44 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2588 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 1978 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 324 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 1621 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1410 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1409 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 44 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1389 exceeds gate value 1360 for firefox webpack home mean loadScripts
Sum of mean exceeds: 218ms | Sum of p95 exceeds: 409ms
Sum of all benchmark exceeds: 627ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [9812728]
UI Startup Metrics (1220 ± 69 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1220111114496912601347
load105896012606610941196
domContentLoaded105295212566610881189
domInteractive17143441729
firstPaint76797126542010871196
backgroundConnect74244721
firstReactRender20163122123
getState1454581928
initialActions001001
loadScripts80569799566843940
setupStore85152813
WebpackHomeuiStartup21681731267824423232590
load16851350211020218372012
domContentLoaded16791346209920018332003
domInteractive161256101449
firstPaint1596544459179267
backgroundConnect22135072440
firstReactRender14948375111274355
getState1243961427
initialActions316134
loadScripts16761344208819918311992
setupStore4573339120321
FirefoxBrowserifyHomeuiStartup1321118415226513611455
load1172106113966012211270
domContentLoaded1172106113966012211270
domInteractive983620127107138
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21146482149
firstReactRender24215552428
getState7435389
initialActions001001
loadScripts1153104413705912011257
setupStore64242611
WebpackHomeuiStartup15011356188411215671715
load13061150159710613801526
domContentLoaded13051150159710613801525
domInteractive79521471585111
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21154162139
firstReactRender40294734246
getState84293910
initialActions002111
loadScripts12881131157910613591507
setupStore1052482489
Benchmark value 1196 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1189 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1196 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 21 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 46 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2591 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 1993 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 321 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 40 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 16ms | Sum of p95 exceeds: 450ms
Sum of all benchmark exceeds: 466ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@nakulmk nakulmk added this pull request to the merge queue Jun 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 12, 2025
@lwin-kyaw lwin-kyaw enabled auto-merge June 12, 2025 17:51
@metamaskbot
Copy link
Collaborator

Builds ready [4344f8c]
UI Startup Metrics (1228 ± 72 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1228112115287212531377
load106395413156710891216
domContentLoaded105795013116710821210
domInteractive17143651632
firstPaint77284125641610851166
backgroundConnect74284821
firstReactRender21165862136
getState1463282030
initialActions001001
loadScripts809714105765834957
setupStore85152813
WebpackHomeuiStartup21121639259021822562460
load16301285199416617441885
domContentLoaded16241281198416517401876
domInteractive171172131352
firstPaint1556330549186241
backgroundConnect21104572535
firstReactRender15446372115310363
getState164410401324
initialActions512662634
loadScripts16201280197316317381865
setupStore4873229221315
FirefoxBrowserifyHomeuiStartup1338119315857313761480
load1193103914337612411338
domContentLoaded1193103914327612411338
domInteractive1043624829120145
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2214108112240
firstReactRender23203322427
getState74213810
initialActions001001
loadScripts1173102514197612221313
setupStore64313611
WebpackHomeuiStartup1519137517669715801712
load1319117815649613801509
domContentLoaded1318117815639613801509
domInteractive80611641687112
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21144252228
firstReactRender41284844446
getState95315928
initialActions102112
loadScripts1300115815459713611491
setupStore85243810
Benchmark value 1377 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1216 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1210 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 22 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 958 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 49 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2460 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 315 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 42 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 21ms | Sum of p95 exceeds: 346ms
Sum of all benchmark exceeds: 367ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@chaitanyapotti chaitanyapotti disabled auto-merge June 13, 2025 02:25
@metamaskbot
Copy link
Collaborator

Builds ready [d9044fb]
UI Startup Metrics (1207 ± 61 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1207109514036112501336
load104594411765210771161
domContentLoaded103994111725310721155
domInteractive17144461732
firstPaint775130117039510651162
backgroundConnect74264722
firstReactRender21174452133
getState1354081827
initialActions001001
loadScripts79269992652819904
setupStore85253814
WebpackHomeuiStartup20721617256322422202436
load15911267196117117281858
domContentLoaded15851263195017017201849
domInteractive161167121352
firstPaint1666168176191293
backgroundConnect21114462533
firstReactRender15446366114313354
getState144317311220
initialActions315134
loadScripts15821261194016917171838
setupStore5073209520308
FirefoxBrowserifyHomeuiStartup1288115415677313321412
load1147102714227111931267
domContentLoaded1147102714217111931267
domInteractive983619027113154
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect18134851926
firstReactRender22202822227
getState74365811
initialActions001001
loadScripts1129101213977011771253
setupStore73365621
WebpackHomeuiStartup1505135717588315591676
load1312117715608713641495
domContentLoaded1311117715598713641494
domInteractive81311641688110
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20164752127
firstReactRender40294734245
getState94658930
initialActions002111
loadScripts1294116615438813451477
setupStore85294820
Benchmark value 22 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 50 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 308 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 40 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 20ms | Sum of p95 exceeds: 247ms
Sum of all benchmark exceeds: 267ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [4669a0b]
UI Startup Metrics (1209 ± 69 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1209109014746912401346
load105094612606310811156
domContentLoaded104494112546310741150
domInteractive1713103101630
firstPaint790144126239210651158
backgroundConnect73345725
firstReactRender19164232023
getState1353971927
initialActions001001
loadScripts800698100962834899
setupStore85222812
WebpackHomeuiStartup22001673259619223052508
load17011292208516818091952
domContentLoaded16951288208116718051948
domInteractive16127591435
firstPaint1566158272179262
backgroundConnect2811289352741
firstReactRender14747368106286341
getState145202201525
initialActions612842845
loadScripts16911286207916618031947
setupStore5273219623318
FirefoxBrowserifyHomeuiStartup1291116815677013341435
load1149104014186911821288
domContentLoaded1149104014177011811288
domInteractive963517325111142
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect19134961927
firstReactRender22202922327
getState74364810
initialActions0031301
loadScripts1132102513926811621263
setupStore64112610
WebpackHomeuiStartup15371364177310016101722
load13401179159210314211543
domContentLoaded13401179159210314211543
domInteractive80542061885107
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21157872230
firstReactRender41305244347
getState94315921
initialActions104111
loadScripts13211163157310414051527
setupStore85283810
Benchmark value 25 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 2200 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 53 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2509 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 318 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 41 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 32ms | Sum of p95 exceeds: 315ms
Sum of all benchmark exceeds: 347ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 2.75 KiB (0.04%)
  • common: 358 Bytes (0%)

@lwin-kyaw lwin-kyaw added this pull request to the merge queue Jun 13, 2025
Merged via the queue into main with commit 929f04a Jun 13, 2025
398 of 403 checks passed
@lwin-kyaw lwin-kyaw deleted the feat/srp-flow-review branch June 13, 2025 04:05
@github-actions github-actions bot locked and limited conversation to collaborators Jun 13, 2025
@metamaskbot metamaskbot added the release-12.22.0 Issue or pull request that will be included in release 12.22.0 label Jun 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-onboarding release-12.22.0 Issue or pull request that will be included in release 12.22.0 team-onboarding Onboarding team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants