fix(confirmations): use elevated surface color in AdvancedGasPriceModal#33297
fix(confirmations): use elevated surface color in AdvancedGasPriceModal#33297georgewrmarshall wants to merge 1 commit into
Conversation
Replace background.default with getElevatedSurfaceColor(theme) on the Advanced gas price bottom sheet container so it renders correctly over the elevated confirmation surface in Pure Black mode. Fixes TMCU-1092 Co-authored-by: George Marshall <georgewrmarshall@users.noreply.github.com>
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. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
The changes are limited to:
These changes do not affect:
Since the changes are purely cosmetic (styling/theming adjustments) and only affect the visual appearance of the modal in pure black dark mode, no E2E smoke tests need to be triggered. The unit tests added cover the styling logic adequately. Performance Test Selection: |
|



Description
In Pure Black mode, the Advanced gas price bottom sheet (
AdvancedGasPriceModal) usedbackground.default(#000000) on its container while stacked over an elevated confirmation surface, making the modal appear flat and indistinguishable from the parent.This change replaces
theme.colors.background.defaultwithgetElevatedSurfaceColor(theme)on the modal container, matching the pattern used by other confirmation gas modals (e.g.gas-fee-token-modal). A muted border is also applied in Pure Black dark mode for visual separation.Fixes TMCU-1092.
Changelog
CHANGELOG entry: Fixed Advanced gas price bottom sheet background in Pure Black mode on transaction confirmations.
Related issues
Fixes: TMCU-1092
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
Pre-merge reviewer checklist