Skip to content

chore: update delete button from red to white#40261

Merged
vinnyhoward merged 2 commits intomainfrom
chore-mdp-660-change-delete-network-color
Feb 20, 2026
Merged

chore: update delete button from red to white#40261
vinnyhoward merged 2 commits intomainfrom
chore-mdp-660-change-delete-network-color

Conversation

@vinnyhoward
Copy link
Contributor

@vinnyhoward vinnyhoward commented Feb 19, 2026

Description

Update delete button from red to white

Open in GitHub Codespaces

Changelog

CHANGELOG entry:null

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MDP-660

Manual testing steps

  1. Go to this page network picker from the home page
  2. Click on the more icon
  3. Observe change

Screenshots/Recordings

~

Before

image

After

Screenshot 2026-02-19 at 4 37 59 PM

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

Low Risk
Purely a UI styling change to the network list item menu with no logic, data flow, or security impact.

Overview
Updates NetworkListItemMenu so the Delete menu item no longer uses error (red) styling: removes the legacy IconColor.errorDefault icon color and TextColor.errorDefault text color, leaving the default menu styling (white) for the delete action.

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

@vinnyhoward vinnyhoward requested a review from a team as a code owner February 19, 2026 23:45
@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-mobile-ux Mobile UX team label Feb 19, 2026
@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Feb 19, 2026

✨ Files requiring CODEOWNER review ✨

👨‍🔧 @MetaMask/core-extension-ux (1 files, +1 -3)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 network-list-item-menu/
          • 📄 network-list-item-menu.js +1 -3

@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Feb 20, 2026

Builds ready [6d14f22]
UI Startup Metrics (1409 ± 111 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup14091165179011114501626
load118398914539812261373
domContentLoaded117798314079612221349
domInteractive281693162570
firstPaint159701031117201333
backgroundConnect20918727414212234
firstReactRender19123742029
initialActions104113
loadScripts98379212159610281154
setupStore1374961522
numNetworkReqs312290202284
BrowserifyPower User HomeuiStartup2763138910875210327629188
load11851004178514812341533
domContentLoaded1170996170514412171508
domInteractive3219130173074
firstPaint1787432670243284
backgroundConnect1124248912019284914169
firstReactRender24164462635
initialActions201481513
loadScripts966806147714010151302
setupStore1664271933
numNetworkReqs75331352492119
WebpackStandard HomeuiStartup8316821076928831031
load71260788382753863
domContentLoaded70660387781749847
domInteractive2815118222371
firstPaint1096040652143189
backgroundConnect281865103151
firstReactRender18127371928
initialActions106112
loadScripts70460187080747845
setupStore1173851220
numNetworkReqs3122101212587
WebpackPower User HomeuiStartup1290895276528313571760
load77163711921167761060
domContentLoaded76063311751147681047
domInteractive41191813140107
firstPaint1566840079184327
backgroundConnect17513354663168293
firstReactRender23173842431
initialActions101011
loadScripts75763111661127661044
setupStore1243651520
numNetworkReqs963923236118153
FirefoxBrowserifyStandard HomeuiStartup15921381199814516151925
load13501178173311913851585
domContentLoaded13491178173311913831585
domInteractive71322133892140
firstPaint------
backgroundConnect5630181195997
firstReactRender13111811315
initialActions102112
loadScripts13231158164811113611538
setupStore167151201431
numNetworkReqs312098202591
BrowserifyPower User HomeuiStartup27482115473842028923503
load15761315310625416452001
domContentLoaded15751315310625516442001
domInteractive167371703194157392
firstPaint------
backgroundConnect356771130276425951
firstReactRender19156591822
initialActions206122
loadScripts15341294307224216151935
setupStore1347800185116638
numNetworkReqs74291693592145
WebpackStandard HomeuiStartup18421451338435718212187
load15351269296429015281749
domContentLoaded15341269296429015281748
domInteractive117311438147132228
firstPaint------
backgroundConnect66291552977138
firstReactRender17133441828
initialActions103122
loadScripts15071255293328915001726
setupStore288201342575
numNetworkReqs302089162772
WebpackPower User HomeuiStartup27822021597955229533657
load16051282475449316642313
domContentLoaded16041282475449316642312
domInteractive183401684230161604
firstPaint------
backgroundConnect3641051202264489923
firstReactRender22156592329
initialActions213122
loadScripts15621262472548416102077
setupStore17271157234207678
numNetworkReqs73292214084147
📊 Page Load Benchmark Results

Current Commit: 6d14f22 | Date: 2/20/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±43ms) 🟡 | historical mean value: 1.06s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 732ms (±39ms) 🟢 | historical mean value: 742ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 78ms (±12ms) 🟢 | historical mean value: 81ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 43ms 1.03s 1.38s 1.08s 1.38s
domContentLoaded 732ms 39ms 711ms 1.03s 758ms 1.03s
firstPaint 78ms 12ms 60ms 188ms 84ms 188ms
firstContentfulPaint 78ms 12ms 60ms 188ms 84ms 188ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs
  • background: 58 Bytes (0%)
  • ui: -160 Bytes (0%)
  • common: 20 Bytes (0%)

@vinnyhoward vinnyhoward added this pull request to the merge queue Feb 20, 2026
Merged via the queue into main with commit 1b8db1d Feb 20, 2026
177 checks passed
@vinnyhoward vinnyhoward deleted the chore-mdp-660-change-delete-network-color branch February 20, 2026 16:53
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2026
@metamaskbot metamaskbot added the release-13.21.0 Issue or pull request that will be included in release 13.21.0 label Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.21.0 Issue or pull request that will be included in release 13.21.0 size-XS team-mobile-ux Mobile UX team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants