Skip to content

fix: Remove aggregated balance popover and unused keys #32524

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

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

amandaye0h
Copy link
Contributor

@amandaye0h amandaye0h commented May 5, 2025

Description

This PR removes the aggregated balance popover to reduce the visual noise on the page. It streamlines the IA on the page in the post-Onboarding state, enabling users to focus on their assets and the carousel component.

These changes are part of the overall UI update to the home page.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. After onboarding, popover should not appear
  2. Keys for message.json files should be removed

Screenshots/Recordings

Before

Screenshot 2025-05-06 at 2 33 59 AM

After

Screenshot 2025-05-06 at 2 30 21 AM

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

github-actions bot commented May 5, 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
Copy link
Collaborator

Builds ready [c8b17d5]
UI Startup Metrics (1234 ± 68 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1234112115376812721366
load107197413336211101182
domContentLoaded106496913266211041176
domInteractive18146781731
firstPaint633102122542710601150
backgroundConnect74395722
firstReactRender20164652128
getState1453582129
initialActions001000
loadScripts822720108060855922
setupStore85162813
WebpackHomeuiStartup21681757253916322922403
load16831346198512617791855
domContentLoaded16761342196012617741847
domInteractive161158101343
firstPaint1696336560191326
backgroundConnect281068153659
firstReactRender18555365114313355
getState174271311630
initialActions316135
loadScripts16711341193612617711845
setupStore216250292233
FirefoxBrowserifyHomeuiStartup13601170170210114111543
load12161024156610012771403
domContentLoaded12161024156510012771403
domInteractive1073826537116178
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213230242031
firstReactRender24196782342
getState7430379
initialActions001001
loadScripts11961012154210012581386
setupStore842072069
WebpackHomeuiStartup15371369198612716151810
load13211184175811014031503
domContentLoaded13211184175711014031503
domInteractive79552252184112
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2214102112241
firstReactRender34285353645
getState12528628928
initialActions002111
loadScripts13011166172911013811483
setupStore11530130815
Benchmark value 1071 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1065 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1366 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Sum of mean exceeds: 5ms | Sum of p95 exceeds: 6ms
Sum of all benchmark exceeds: 11ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -1.44 KiB (-0.02%)
  • common: -289 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [0036810]
UI Startup Metrics (1205 ± 59 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1205111915235912341312
load104190812455410721136
domContentLoaded103490112415510661131
domInteractive17135461628
firstPaint723171111940110531099
backgroundConnect84396723
firstReactRender19144242027
getState1554582030
initialActions001000
loadScripts79567395951828888
setupStore85243815
WebpackHomeuiStartup21691680247117223042378
load16921339202113017791853
domContentLoaded16811272201513617731849
domInteractive161169111347
firstPaint1727354278200305
backgroundConnect3010452443353
firstReactRender18953376108299364
getState13457101632
initialActions317145
loadScripts16731269199113517681827
setupStore1764782233
FirefoxBrowserifyHomeuiStartup13821207175811214581603
load12351086159710812961473
domContentLoaded12351086159610812951473
domInteractive1013636639110156
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect211374102138
firstReactRender24197282352
getState74122810
initialActions001001
loadScripts12171069153110512791454
setupStore74364614
WebpackHomeuiStartup15401376201912615911816
load13251178174610913591567
domContentLoaded13251178174610913591567
domInteractive80363032884124
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22144762540
firstReactRender35296563748
getState105337930
initialActions105111
loadScripts13051167172410913411550
setupStore95688822
Benchmark value 24 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Sum of mean exceeds: 0ms | Sum of p95 exceeds: 6ms
Sum of all benchmark exceeds: 6ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -1.44 KiB (-0.02%)
  • common: -499 Bytes (-0.01%)

@metamaskbot
Copy link
Collaborator

Builds ready [bb7ec15]
UI Startup Metrics (1231 ± 71 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1231111013837112881361
load107395612326911191197
domContentLoaded106694012227011151192
domInteractive18136391630
firstPaint678105123143110791184
backgroundConnect83305822
firstReactRender19133642030
getState1454582031
initialActions001001
loadScripts82970897668879948
setupStore75131710
WebpackHomeuiStartup21571735250317723082372
load16691252201814217661857
domContentLoaded16621240201414117591853
domInteractive15115391342
firstPaint1626749463185288
backgroundConnect3110242283555
firstReactRender18855355108301348
getState1346491434
initialActions613473445
loadScripts16571238201114217561848
setupStore1874382431
FirefoxBrowserifyHomeuiStartup13901203167910814551639
load12371050152710512901491
domContentLoaded12371049152710512891491
domInteractive1043736340118173
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21137782334
firstReactRender23205262330
getState84384818
initialActions001001
loadScripts12181035151210512661471
setupStore10425726626
WebpackHomeuiStartup15031357200912015331779
load12901162170010913081514
domContentLoaded12901162170010913081513
domInteractive77451501884125
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22144772341
firstReactRender35275153745
getState104347931
initialActions001011
loadScripts12701143167210912901489
setupStore85405820
Benchmark value 1073 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1066 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1198 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 1184 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 949 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore
Sum of mean exceeds: 9ms | Sum of p95 exceeds: 38ms
Sum of all benchmark exceeds: 47ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: -6.35 KiB (-0.12%)
  • ui: 11.18 KiB (0.16%)
  • common: -12.38 KiB (-0.13%)

@metamaskbot
Copy link
Collaborator

Builds ready [ee58397]
UI Startup Metrics (1198 ± 63 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1198108815076312331290
load104190612756010801141
domContentLoaded103590212685910741136
domInteractive181363101648
firstPaint73878128040610531119
backgroundConnect74353710
firstReactRender241682112751
getState1554192032
initialActions001001
loadScripts79466998857829901
setupStore95354917
WebpackHomeuiStartup21471767242115622852366
load16711250193213817701876
domContentLoaded16651245192513817651873
domInteractive15115891341
firstPaint1786658677226315
backgroundConnect389460633291
firstReactRender17154366115294354
getState204318471440
initialActions316135
loadScripts16601243192214017631871
setupStore1663882033
FirefoxBrowserifyHomeuiStartup13761214183912314441649
load12241073162410912911439
domContentLoaded12241072162411012911439
domInteractive1013726032117157
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213225232134
firstReactRender24205672348
getState84457825
initialActions005101
loadScripts12041055151410312761408
setupStore10423530612
WebpackHomeuiStartup15531383196814216091890
load13321196169712313971612
domContentLoaded13321196169612313971612
domInteractive78341672186122
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22164552335
firstReactRender37294954045
getState1153481032
initialActions102111
loadScripts13131179167912313761600
setupStore85435814
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 49 exceeds gate value 41 for chrome browserify home p95 domInteractive
Benchmark value 51 exceeds gate value 45 for chrome browserify home p95 firstReactRender
Benchmark value 92 exceeds gate value 90 for chrome webpack home p95 backgroundConnect
Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 25 exceeds gate value 24 for firefox browserify home p95 getState
Sum of mean exceeds: 4ms | Sum of p95 exceeds: 17ms
Sum of all benchmark exceeds: 21ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: -6.85 KiB (-0.13%)
  • ui: 55.98 KiB (0.8%)
  • common: -6.43 KiB (-0.07%)

@amandaye0h amandaye0h changed the title fix: Remove aggregated popover and unused keys fix: Remove aggregated balance popover and unused keys May 6, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [acb7f4c]
UI Startup Metrics (1204 ± 56 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1204108613295612521307
load104694911455310831136
domContentLoaded104194411415310781129
domInteractive17136781629
firstPaint78096116039710711138
backgroundConnect7417279
firstReactRender20147981931
getState1463071926
initialActions001000
loadScripts80170190251844887
setupStore85253815
WebpackHomeuiStartup21831637254318023272423
load16831249204014717821872
domContentLoaded16771245202914617781868
domInteractive161168111347
firstPaint1736349372204321
backgroundConnect349333393863
firstReactRender18955368109307339
getState1343671627
initialActions316145
loadScripts16721241200614517751867
setupStore237325382333
FirefoxBrowserifyHomeuiStartup13791184169410814371613
load12351064154811013051477
domContentLoaded12351064154811013051477
domInteractive1073921031122173
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2113123132145
firstReactRender24205672349
getState75353811
initialActions001001
loadScripts12171051151910912891463
setupStore64121610
WebpackHomeuiStartup15141365214913215601811
load1289117516809913281490
domContentLoaded1289117516809913281490
domInteractive79541591884118
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2314102102440
firstReactRender36295253947
getState145297291032
initialActions002111
loadScripts1269115916619913091475
setupStore16530147841
Benchmark value 17 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 41 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 4ms | Sum of p95 exceeds: 13ms
Sum of all benchmark exceeds: 17ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -1.59 KiB (-0.02%)
  • common: -499 Bytes (-0.01%)

@amandaye0h amandaye0h force-pushed the agg-balance-popover-removal branch from 872f80a to 3191b02 Compare May 7, 2025 18:55
@metamaskbot
Copy link
Collaborator

Builds ready [3191b02]
UI Startup Metrics (1227 ± 68 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1227109714226812671356
load106895612305911111180
domContentLoaded106194812246011021174
domInteractive16133231625
firstPaint71785123142810851169
backgroundConnect84476823
firstReactRender22155382145
getState156108131937
initialActions001001
loadScripts82071497556853927
setupStore85172914
WebpackHomeuiStartup22301842250616323502446
load17361388208213618171950
domContentLoaded17301382207713618081944
domInteractive161262101344
firstPaint1626133859182286
backgroundConnect3310283304164
firstReactRender18956364112299342
getState204283391736
initialActions612522546
loadScripts17251380207513718051943
setupStore246264342244
FirefoxBrowserifyHomeuiStartup13581175184312214051628
load12141049170712212671500
domContentLoaded12141049170612212661499
domInteractive1083827537124188
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213210202040
firstReactRender24195772349
getState74273810
initialActions001001
loadScripts11961036167612112481484
setupStore6434569
WebpackHomeuiStartup15281359193512815791819
load13061170171210913491528
domContentLoaded13061170171210913491528
domInteractive76341451683102
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect241478112350
firstReactRender35294743744
getState1053261029
initialActions002111
loadScripts12861151169110813251513
setupStore11529329920
cc: @HowardBraham
Benchmark value 1062 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 37 exceeds gate value 33 for chrome browserify home p95 getState
Benchmark value 2231 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1737 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1730 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 1725 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 1500 exceeds gate value 1495 for firefox browserify home p95 load
Benchmark value 1499 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded
Benchmark value 1484 exceeds gate value 1475 for firefox browserify home p95 loadScripts
Benchmark value 50 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Sum of mean exceeds: 119ms | Sum of p95 exceeds: 28ms
Sum of all benchmark exceeds: 147ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -1.59 KiB (-0.02%)
  • common: -499 Bytes (-0.01%)

@metamaskbot
Copy link
Collaborator

Builds ready [9a19fa7]
UI Startup Metrics (1202 ± 57 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1202110114585712471279
load105194813315510931129
domContentLoaded104594413255510881124
domInteractive16133641624
firstPaint791138113739510681119
backgroundConnect84375722
firstReactRender19153542030
getState16673102132
initialActions003001
loadScripts805715105653844881
setupStore85142812
WebpackHomeuiStartup21651772255414422712367
load16711384201411117401839
domContentLoaded16651381198611017351835
domInteractive161192131350
firstPaint1696740668190299
backgroundConnect271063113447
firstReactRender18454413113298358
getState184308361832
initialActions316135
loadScripts16601379196411117311831
setupStore196258252133
FirefoxBrowserifyHomeuiStartup13991215174711114501661
load12491069161810713041497
domContentLoaded12481069161810713041496
domInteractive1054032334117151
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21136982239
firstReactRender24205962430
getState10417918914
initialActions002001
loadScripts12301055160410612831475
setupStore64363610
WebpackHomeuiStartup15621362218216715991975
load13371173179214213881686
domContentLoaded13371172179114213881686
domInteractive81341441992130
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2514299292341
firstReactRender36306263947
getState14432632932
initialActions102111
loadScripts13161156176213613711652
setupStore95577827
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 22 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
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 1231 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 1661 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 1497 exceeds gate value 1495 for firefox browserify home p95 load
Benchmark value 1496 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded
Benchmark value 1975 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1686 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1686 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 1652 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 16ms | Sum of p95 exceeds: 122ms
Sum of all benchmark exceeds: 138ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -1.59 KiB (-0.02%)
  • common: -499 Bytes (-0.01%)

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

Successfully merging this pull request may close these issues.

2 participants