Skip to content

Conversation

@dan437
Copy link
Contributor

@dan437 dan437 commented Dec 4, 2025

Description

Updates row key colors in Confirmations.

Open in GitHub Codespaces

Changelog

CHANGELOG entry: Update row key colors in Confirmations

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/5279

Manual testing steps

  1. Trigger the Confirmations page for different scenarios (Send, dapp, Base network, etc.)
  2. Click on "Show advanced details" button
  3. Check that colors are updated according to the screenshots

Screenshots/Recordings

Before

After

image image

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.

Note

Set default label text color in confirmation info rows to textAlternative and update affected snapshots.

  • UI:
    • Confirmations Info Row: Default label color now falls back to TextColor.textAlternative (ui/components/app/confirm/info/row/row.tsx).
  • Tests/Snapshots:
    • Updated snapshots across confirmation views to reflect the new label color on row keys (multiple files under ui/pages/confirmations/components/confirm/** and ui/components/app/confirm/info/row/**).

Written by Cursor Bugbot for commit d773d38. This will update automatically on new commits. Configure here.

@dan437 dan437 requested a review from a team as a code owner December 4, 2025 13:25
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 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 metamaskbot added the team-confirmations Push issues to confirmations team label Dec 4, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

@MetaMask/confirmations (21 files, +282 -282)
  • 📁 ui/
    • 📁 pages/
      • 📁 confirmations/
        • 📁 components/
          • 📁 confirm/
            • 📁 info/
              • 📁 __snapshots__/
                • 📄 info.test.tsx.snap +27 -27
              • 📁 approve/
                • 📁 __snapshots__/
                  • 📄 approve.test.tsx.snap +8 -8
                • 📁 approve-static-simulation/
                  • 📁 __snapshots__/
                    • 📄 approve-static-simulation.test.tsx.snap +4 -4
              • 📁 native-transfer/
                • 📁 __snapshots__/
                  • 📄 native-transfer.test.tsx.snap +1 -1
              • 📁 nft-token-transfer/
                • 📁 __snapshots__/
                  • 📄 nft-token-transfer.test.tsx.snap +1 -1
              • 📁 personal-sign/
                • 📁 siwe-sign/
                  • 📁 __snapshots__/
                    • 📄 siwe-sign.test.tsx.snap +19 -19
              • 📁 set-approval-for-all-info/
                • 📁 __snapshots__/
                  • 📄 set-approval-for-all-info.test.tsx.snap +2 -2
                • 📁 revoke-set-approval-for-all-static-simulation/
                  • 📁 __snapshots__/
                    • 📄 revoke-set-approval-for-all-static-simulation.test.tsx.snap +3 -3
                • 📁 set-approval-for-all-static-simulation/
                  • 📁 __snapshots__/
                    • 📄 set-approval-for-all-static-simulation.test.tsx.snap +2 -2
              • 📁 shared/
                • 📁 advanced-details/
                  • 📁 __snapshots__/
                    • 📄 advanced-details.test.tsx.snap +4 -4
                • 📁 gas-fees-row/
                  • 📁 __snapshots__/
                    • 📄 gas-fees-row.test.tsx.snap +1 -1
                • 📁 transaction-data/
                  • 📁 __snapshots__/
                    • 📄 transaction-data.test.tsx.snap +55 -55
              • 📁 token-transfer/
                • 📁 __snapshots__/
                  • 📄 token-details-section.test.tsx.snap +1 -1
                  • 📄 token-transfer.test.tsx.snap +1 -1
              • 📁 typed-sign/
                • 📁 __snapshots__/
                  • 📄 typed-sign.test.tsx.snap +47 -47
                • 📁 typed-sign-v4-simulation/
                  • 📁 permit-simulation/
                    • 📁 __snapshots__/
                    • 📄 permit-simulation.test.tsx.snap +4 -4
              • 📁 typed-sign-v1/
                • 📁 __snapshots__/
                  • 📄 typed-sign-v1.test.tsx.snap +3 -3
            • 📁 row/
              • 📁 __snapshots__/
                • 📄 dataTree.test.tsx.snap +28 -28
              • 📁 typed-sign-data/
                • 📁 __snapshots__/
                  • 📄 typedSignData.test.tsx.snap +15 -15
              • 📁 typed-sign-data-v1/
                • 📁 __snapshots__/
                  • 📄 typedSignDataV1.test.tsx.snap +2 -2
        • 📁 confirm/
          • 📁 __snapshots__/
            • 📄 confirm.test.tsx.snap +54 -54

@metamaskbot
Copy link
Collaborator

Builds ready [0da8b0c]
UI Startup Metrics (1235 ± 104 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12351028153110413101427
load103786312949310901223
domContentLoaded103086012779210841218
domInteractive2614160212278
firstPaint5159512934079971195
backgroundConnect21519426113224242
firstReactRender301960103452
getState3516110173982
initialActions104113
loadScripts822658106890878994
setupStore1162231219
numNetworkReqs1056816559
BrowserifyPower User HomeuiStartup19541570262822920962376
load992891141911310011288
domContentLoaded98188214111149871273
domInteractive3016109212894
firstPaint5128112993769381215
backgroundConnect261196820124232538
firstReactRender513910195666
getState207139785117200680
initialActions102112
loadScripts77568012031117851070
setupStore16104481538
numNetworkReqs78651612074139
WebpackStandard HomeuiStartup814695117774839986
load63256294165674768
domContentLoaded62855893464671762
domInteractive2715128222394
firstPaint20272792125202588
backgroundConnect105538926
firstReactRender27204163238
getState261356103442
initialActions103111
loadScripts62555692363669754
setupStore1052841217
numNetworkReqs1057016559
WebpackPower User HomeuiStartup15701202235726317392092
load6835721084107728892
domContentLoaded6755641071107721878
domInteractive38172534030127
firstPaint28888904211299832
backgroundConnect64775615515542
firstReactRender574388106380
getState18114466553184229
initialActions104112
loadScripts6725621069106719876
setupStore17959111550
numNetworkReqs79652422774148
FirefoxBrowserifyStandard HomeuiStartup13141096175114413901582
load1094947154511411431321
domContentLoaded1094946154511411431320
domInteractive67313014288134
firstPaint------
backgroundConnect51213304164116
firstReactRender23186072336
getState1166991030
initialActions103122
loadScripts1065932145910311161278
setupStore1165181034
numNetworkReqs1156616663
BrowserifyPower User HomeuiStartup30891920475180039374543
load1380982274645314482414
domContentLoaded1380982274645314482413
domInteractive973633671101266
firstPaint------
backgroundConnect2972720083844831093
firstReactRender64392723365127
getState285721341252350867
initialActions606313349
loadScripts1247967257532013352229
setupStore19481090269296926
numNetworkReqs77521412471133
WebpackStandard HomeuiStartup14771281216916215041805
load12151065149610212711455
domContentLoaded12151065149610212711455
domInteractive53271933272111
firstPaint------
backgroundConnect49213193843121
firstReactRender282080102941
getState177197291264
initialActions103122
loadScripts1187105014358912451366
setupStore146124181136
numNetworkReqs1256817763
WebpackPower User HomeuiStartup33892121610894242355149
load16651152311454517432832
domContentLoaded16641151310954517432832
domInteractive1023240376100311
firstPaint------
backgroundConnect2272515403372061219
firstReactRender68401952872147
getState353881832317508949
initialActions3162827
loadScripts15281118293642415962624
setupStore13471149194114664
numNetworkReqs77541322273126
📊 Page Load Benchmark Results

Current Commit: 0da8b0c | Date: 12/4/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±38ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 733ms (±36ms) 🟢 | historical mean value: 725ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 78ms (±10ms) 🟢 | historical mean value: 79ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 38ms 1.03s 1.33s 1.08s 1.33s
domContentLoaded 733ms 36ms 714ms 1000ms 756ms 1000ms
firstPaint 78ms 10ms 64ms 164ms 88ms 164ms
firstContentfulPaint 78ms 10ms 64ms 164ms 88ms 164ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 33 Bytes (0%)
  • common: 20 Bytes (0%)

@dan437 dan437 enabled auto-merge December 4, 2025 14:32
@metamaskbot
Copy link
Collaborator

Builds ready [22910d8]
UI Startup Metrics (1263 ± 112 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12631004159611213381480
load105986413919711141239
domContentLoaded105385913839711071234
domInteractive251598182280
firstPaint53385141541910381175
backgroundConnect22119826614230250
firstReactRender322059113855
getState361693134356
initialActions107113
loadScripts8396561175918931002
setupStore1273141421
numNetworkReqs1157117563
BrowserifyPower User HomeuiStartup19991573265623021632396
load1000879138012210251326
domContentLoaded990870137512210051317
domInteractive311798202992
firstPaint4849713593729211261
backgroundConnect255200748110228527
firstReactRender52399285567
getState19714269077199285
initialActions103112
loadScripts78467111691198031098
setupStore1785091644
numNetworkReqs80642452675143
WebpackStandard HomeuiStartup79869499773819963
load62756080668644787
domContentLoaded62255680168639781
domInteractive2614109232196
firstPaint22898786173196659
backgroundConnect1053251224
firstReactRender2719121113137
getState261357103245
initialActions104111
loadScripts62055379266637776
setupStore1062641216
numNetworkReqs1157318568
WebpackPower User HomeuiStartup15981227223327718532136
load665576111894691888
domContentLoaded657568110894680883
domInteractive3318175273098
firstPaint2601021118181259641
backgroundConnect100663218833555
firstReactRender58457996475
getState18014372059188229
initialActions104112
loadScripts654566109692678875
setupStore1794891541
numNetworkReqs81632643673149
FirefoxBrowserifyStandard HomeuiStartup12961069173614213591583
load107792013749211331245
domContentLoaded107792013749211321244
domInteractive59291633183126
firstPaint------
backgroundConnect44212253144122
firstReactRender23184052336
getState147214251028
initialActions102022
loadScripts105090313568411021173
setupStore136104161031
numNetworkReqs1256517663
BrowserifyPower User HomeuiStartup31142120681194840344774
load1422990509861913752637
domContentLoaded1421989509861913752637
domInteractive136331058190109485
firstPaint------
backgroundConnect2522312633352661183
firstReactRender60361522364114
getState303731762286364960
initialActions2020224
loadScripts1311971497752612872361
setupStore15551045229137719
numNetworkReqs76491432173128
WebpackStandard HomeuiStartup15551260189415216471856
load12731088156110813641468
domContentLoaded12731088156110813641468
domInteractive64281333388124
firstPaint------
backgroundConnect54221543069114
firstReactRender27204352937
getState147174171325
initialActions103122
loadScripts12451072154310413341417
setupStore165149211445
numNetworkReqs1256616761
WebpackPower User HomeuiStartup33322031576695342834912
load16541135293155921202856
domContentLoaded16531135293155921202854
domInteractive1892811852981511113
firstPaint------
backgroundConnect2222621093612021068
firstReactRender76383545471209
getState284851518270278848
initialActions401101227
loadScripts15511120289447916182556
setupStore88580614494481
numNetworkReqs77531372273126
📊 Page Load Benchmark Results

Current Commit: 22910d8 | Date: 12/4/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.09s (±50ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 762ms (±45ms) 🟢 | historical mean value: 725ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 83ms (±13ms) 🟢 | historical mean value: 79ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.09s 50ms 1.05s 1.43s 1.17s 1.43s
domContentLoaded 762ms 45ms 729ms 1.10s 820ms 1.10s
firstPaint 83ms 13ms 68ms 200ms 88ms 200ms
firstContentfulPaint 83ms 13ms 68ms 200ms 88ms 200ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 33 Bytes (0%)
  • common: 20 Bytes (0%)

@dan437 dan437 disabled auto-merge December 4, 2025 15:48
@dan437 dan437 enabled auto-merge December 4, 2025 15:48
@metamaskbot
Copy link
Collaborator

Builds ready [d773d38]
UI Startup Metrics (1247 ± 100 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12471013166710013191400
load104186813978210951180
domContentLoaded103586313898010881165
domInteractive2615135202284
firstPaint53597140741410331185
backgroundConnect22019726613227242
firstReactRender312062103455
getState3716111174566
initialActions105113
loadScripts822658115978867957
setupStore1162941318
numNetworkReqs1057317558
BrowserifyPower User HomeuiStartup20281763292219220842410
load1033910168413010531350
domContentLoaded1019903162512910341340
domInteractive3317109203495
firstPaint50610916643769471182
backgroundConnect258201761104231554
firstReactRender5941111166297
getState19113366372196232
initialActions105112
loadScripts81069513721258201133
setupStore19104072336
numNetworkReqs92622072397135
WebpackStandard HomeuiStartup825695110683862990
load64356684775676803
domContentLoaded63856184175672797
domInteractive2715100222294
firstPaint20683787131209604
backgroundConnect1064461324
firstReactRender27204463239
getState261463103444
initialActions104112
loadScripts63555883273669794
setupStore1062731218
numNetworkReqs1156916559
WebpackPower User HomeuiStartup16781330231323518872163
load702612117793711926
domContentLoaded691602116994694913
domInteractive37191382635113
firstPaint30193947199330702
backgroundConnect66763414620544
firstReactRender61499596580
getState18313375261189220
initialActions105112
loadScripts688600115892692904
setupStore211056112252
numNetworkReqs1486834861202284
FirefoxBrowserifyStandard HomeuiStartup12981064166313914081534
load107391713369311281260
domContentLoaded107391213369311271260
domInteractive61292033883130
firstPaint------
backgroundConnect45201513049125
firstReactRender22183952236
getState146176241031
initialActions103122
loadScripts104490013098410941218
setupStore12583131034
numNetworkReqs1156416660
BrowserifyPower User HomeuiStartup25982020453754026304292
load1212966264631312072067
domContentLoaded1212966264631312072067
domInteractive1113054897106372
firstPaint------
backgroundConnect178251189238150763
firstReactRender5737187195793
getState274351088224273827
initialActions217123
loadScripts1134949227819611221515
setupStore1639783195202661
numNetworkReqs82591752679154
WebpackStandard HomeuiStartup14591217199015615541766
load12131041164012012861424
domContentLoaded12121041164012012851423
domInteractive51271403180122
firstPaint------
backgroundConnect47223273844119
firstReactRender27207792834
getState147138201247
initialActions103122
loadScripts11811027161010712581395
setupStore145137171337
numNetworkReqs1256616761
WebpackPower User HomeuiStartup30022137734387528994684
load15801195565865016262838
domContentLoaded15791195565865016262837
domInteractive168293774396107395
firstPaint------
backgroundConnect169271535246154595
firstReactRender60401462261103
getState299711216255403872
initialActions208123
loadScripts14951169563559015732724
setupStore1274724176138591
numNetworkReqs81601722481139
📊 Page Load Benchmark Results

Current Commit: d773d38 | Date: 12/4/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.03s (±43ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 720ms (±62ms) 🟢 | historical mean value: 733ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 88ms (±127ms) 🟢 | historical mean value: 79ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.03s 43ms 1.00s 1.36s 1.05s 1.36s
domContentLoaded 720ms 62ms 693ms 1.29s 740ms 1.29s
firstPaint 88ms 127ms 60ms 1.35s 88ms 1.35s
firstContentfulPaint 88ms 127ms 60ms 1.35s 88ms 1.35s
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 33 Bytes (0%)
  • common: 20 Bytes (0%)

@dan437 dan437 added this pull request to the merge queue Dec 4, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 4, 2025
@dan437 dan437 added this pull request to the merge queue Dec 5, 2025
Merged via the queue into main with commit 0765009 Dec 5, 2025
176 checks passed
@dan437 dan437 deleted the confirmations-ui branch December 5, 2025 09:44
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2025
@metamaskbot metamaskbot added the release-13.14.0 Issue or pull request that will be included in release 13.14.0 label Dec 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.14.0 Issue or pull request that will be included in release 13.14.0 size-XS team-confirmations Push issues to confirmations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants