fix(confirmations): align Edit network fee sheet with elevated surfaces#33294
fix(confirmations): align Edit network fee sheet with elevated surfaces#33294georgewrmarshall wants to merge 2 commits into
Conversation
Remove the primary.muted fill on selected gas options in pure black dark mode and use getElevatedSurfaceColor for the EstimatesModal container so the sheet matches other confirmation bottom sheets. Fixes TMCU-1016 Co-authored-by: Cursor <cursoragent@cursor.com>
|
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. |
PR template — items to address before "Ready for review"Warnings — informational, address before merging:
See docs/readme/ready-for-review.md for the full Definition of Ready for Review. |
Leave primary.muted selection fill as-is; pure black fix is limited to EstimatesModal elevated surface styling. Co-authored-by: Cursor <cursoragent@cursor.com>
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
These changes are cosmetic in nature (background color and border styling only) and fall under the "COSMETIC CHANGES — IGNORE FOR TEST SELECTION" guidance. The changes do not affect any user-facing functional flows, state management, business logic, or E2E-testable behaviors. No E2E smoke tests are warranted. Performance Test Selection: |
|



Description
In Pure Black dark mode, the Edit network fee bottom sheet (
EstimatesModal) showed two visual regressions:background.default(#000000), so it did not match other elevated confirmation bottom sheets.primary.mutedfill, creating a visible dark blue/grey band inside the sheet.This PR applies the same style-only stopgap used elsewhere in confirmations (#33152,
gas-fee-token-modal):estimates-modal.styles.ts—getElevatedSurfaceColor()+ muted border in pure black dark modegas-option.styles.ts— omitprimary.mutedrow fill in pure black dark mode; selection remains on the left-edge indicatorChangelog
CHANGELOG entry: Fixed Edit network fee bottom sheet styling in Pure Black theme so the sheet surface and selected gas option match other elevated confirmation sheets.
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/TMCU-1016
Refs: https://consensyssoftware.atlassian.net/browse/TMCU-622
Manual testing steps
Unit tests:
gas-option.styles.test.ts,estimates-modal.styles.test.ts, and existing component tests pass.Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an examplePre-merge reviewer checklist
Made with Cursor