Skip to content

chore: Remove support link on the home page #32157

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
merged 6 commits into from
Apr 24, 2025
Merged

Conversation

amandaye0h
Copy link
Contributor

@amandaye0h amandaye0h commented Apr 21, 2025

Description

This removes the support link across Tokens, NFTs and the Activity tabs. It improves the IA of the home page, ensuring users can focus on their assets.

Part of MaWaSB updates.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Open MetaMask
  2. Scroll down on the Tokens page
  3. Navigate to the NFTs and Activity tab

Screenshots/Recordings

Before

Screen.Recording.2025-04-23.at.3.14.29.AM.mov

After

Screen.Recording.2025-04-22.at.6.14.07.AM.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.

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
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🖥️ @MetaMask/wallet-ux

  • ui/components/multichain/account-overview/account-overview-tabs.tsx

@metamaskbot
Copy link
Collaborator

Builds ready [110841d]
UI Startup Metrics (1184 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1184107414616712161308
load102293211865810531144
domContentLoaded101792911825710471139
domInteractive16132941626
firstPaint60079118241210151110
backgroundConnect6418278
firstReactRender21154462233
getState145106141834
initialActions001001
loadScripts78267694656807896
setupStore84182813
WebpackHomeuiStartup20991699248418122362352
load16261251191513417301846
domContentLoaded16201247189713317241840
domInteractive161163101341
firstPaint1896263891224355
backgroundConnect269332332655
firstReactRender19155393121320364
getState134269261228
initialActions316135
loadScripts16151246187413217211817
setupStore226311402135
FirefoxBrowserifyHomeuiStartup13451144179913114241593
load12001027163813212751474
domContentLoaded11991026163713212741474
domInteractive1024018128115166
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213209202036
firstReactRender22195552233
getState9416016816
initialActions001001
loadScripts11791010161612912531459
setupStore9418224621
WebpackHomeuiStartup15181342181310916081732
load1299115215579613501496
domContentLoaded1298115015579613491496
domInteractive82571442091130
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect251686132663
firstReactRender36296253847
getState1043861028
initialActions002111
loadScripts1276113515149413251471
setupStore85232811
cc: @HowardBraham
Benchmark value 34 exceeds gate value 33 for chrome browserify home p95 getState
Benchmark value 355 exceeds gate value 334 for chrome webpack home p95 firstPaint
Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 63 exceeds gate value 49 for firefox webpack home p95 backgroundConnect
Sum of mean exceeds: 1ms | Sum of p95 exceeds: 36ms
Sum of all benchmark exceeds: 37ms

Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: -710 Bytes (-0.01%)
  • common: 0 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [c1d8c06]
UI Startup Metrics (1210 ± 65 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1210110414236512571335
load105094212006010931163
domContentLoaded104493811905910841157
domInteractive17136061727
firstPaint72084119941610701145
backgroundConnect7420289
firstReactRender20154852031
getState1353382030
initialActions001001
loadScripts80770094658847919
setupStore84222813
WebpackHomeuiStartup21621769252517222802435
load16641371195213417751861
domContentLoaded16571367194113317691855
domInteractive161168111349
firstPaint1877137867230329
backgroundConnect261095152658
firstReactRender23056410120343368
getState1245881527
initialActions317145
loadScripts16521366192613317651853
setupStore237285362138
FirefoxBrowserifyHomeuiStartup13531198182211314131586
load12081023166511812751451
domContentLoaded12081022166411812751451
domInteractive1034221832116172
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2113100132040
firstReactRender22203222226
getState9416716816
initialActions001001
loadScripts11881005164611912551437
setupStore6434368
WebpackHomeuiStartup14941313183911715691731
load12801137164410313341481
domContentLoaded12801136164410313341481
domInteractive82401612391135
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22147382342
firstReactRender35295443644
getState84576918
initialActions001011
loadScripts12611120163010313111463
setupStore85586817
Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: -731 Bytes (-0.01%)
  • common: 0 Bytes (0%)

@amandaye0h amandaye0h changed the title chore: Remove support link on home page chore: Remove support link on the home page Apr 22, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [4bba4fd]
UI Startup Metrics (1227 ± 60 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1227111413686012621357
load105795911945310831173
domContentLoaded105195511885210781168
domInteractive18136371731
firstPaint716143119941710721126
backgroundConnect74182712
firstReactRender23164362436
getState1454192031
initialActions001001
loadScripts81072293449837915
setupStore84183813
WebpackHomeuiStartup21241743268918122492388
load16291302210413817351852
domContentLoaded16221297209313617261840
domInteractive161157111449
firstPaint200721605161248349
backgroundConnect299236312874
firstReactRender21654392123341363
getState184295421433
initialActions318136
loadScripts16171293206913517231816
setupStore197294282133
FirefoxBrowserifyHomeuiStartup13581162171611114301596
load12071001150111012981405
domContentLoaded12071001150111012981404
domInteractive1094228337121176
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20136582138
firstReactRender22193332227
getState9422422811
initialActions001001
loadScripts1189988147910912801384
setupStore74345621
WebpackHomeuiStartup1467131017749615351674
load1258112915568513121428
domContentLoaded1257112915568513111428
domInteractive86372493596159
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21134562237
firstReactRender35288573743
getState104327929
initialActions001011
loadScripts1238111415398412941407
setupStore85405822
Benchmark value 201 exceeds gate value 199 for chrome webpack home mean firstPaint
Benchmark value 349 exceeds gate value 334 for chrome webpack home p95 firstPaint
Benchmark value 159 exceeds gate value 156 for firefox webpack home p95 domInteractive
Sum of mean exceeds: 2ms | Sum of p95 exceeds: 18ms
Sum of all benchmark exceeds: 20ms

Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: -731 Bytes (-0.01%)
  • common: 0 Bytes (0%)

@amandaye0h amandaye0h added area-design Design bug (previously known as papercuts - ask Hilary for more detail) type-enhancement labels Apr 23, 2025
@amandaye0h amandaye0h marked this pull request as ready for review April 23, 2025 18:56
@amandaye0h amandaye0h requested a review from a team as a code owner April 23, 2025 18:56
@metamaskbot
Copy link
Collaborator

Builds ready [2055d7e]
UI Startup Metrics (1229 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1229113514966712641345
load106096112335810921173
domContentLoaded105495612275710851168
domInteractive18136271729
firstPaint70693118441810771141
backgroundConnect74213811
firstReactRender20165642124
getState1354381930
initialActions001001
loadScripts81372595354842915
setupStore84182812
WebpackHomeuiStartup21371723245716322692342
load16551343195412917641828
domContentLoaded16481339194412817541821
domInteractive161256101346
firstPaint1855937065227308
backgroundConnect2710280293160
firstReactRender22455380119333358
getState1346591534
initialActions317145
loadScripts16431338194212917511820
setupStore227289382134
FirefoxBrowserifyHomeuiStartup13801208173111414401617
load12281043159911612811486
domContentLoaded12281042159911612811486
domInteractive1054123429117149
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21134552331
firstReactRender23195342327
getState84647811
initialActions001001
loadScripts12101030158011612641462
setupStore6424268
WebpackHomeuiStartup15101309190511915851759
load12961139165410313511532
domContentLoaded12961139165410313511532
domInteractive91333444399182
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20145562228
firstReactRender35296653645
getState74303812
initialActions001011
loadScripts12771125163710213311512
setupStore95527821
Benchmark value 182 exceeds gate value 156 for firefox webpack home p95 domInteractive
Sum of mean exceeds: 0ms | Sum of p95 exceeds: 26ms
Sum of all benchmark exceeds: 26ms

Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: -731 Bytes (-0.01%)
  • common: 0 Bytes (0%)

Copy link
Contributor

@vinnyhoward vinnyhoward left a comment

Choose a reason for hiding this comment

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

Less cluttered. I like it LGTM

@amandaye0h
Copy link
Contributor Author

Thanks for the speedy review!!

@amandaye0h amandaye0h added this pull request to the merge queue Apr 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 23, 2025
@georgewrmarshall georgewrmarshall added this pull request to the merge queue Apr 23, 2025
Merged via the queue into main with commit a72d529 Apr 24, 2025
177 checks passed
@georgewrmarshall georgewrmarshall deleted the support-link-removal branch April 24, 2025 00:37
@github-actions github-actions bot locked and limited conversation to collaborators Apr 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-design Design bug (previously known as papercuts - ask Hilary for more detail) team-assets type-enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants