Skip to content

refactor: adopt FilledButtonStyle across more screens#119

Open
raulriera wants to merge 1 commit intorefactor-new-serverfrom
refactor/adopt-filled-button-style
Open

refactor: adopt FilledButtonStyle across more screens#119
raulriera wants to merge 1 commit intorefactor-new-serverfrom
refactor/adopt-filled-button-style

Conversation

@raulriera
Copy link
Collaborator

@raulriera raulriera commented Feb 25, 2026

Summary

  • Replaces CodeButton(style: .filled) with Button + .buttonStyle(.filled) in 10 screens
  • Replaces CodeButton(style: .filledCustom) in FundingSelectionSheet by moving the icon into the button label

Screens updated

  • CurrencyInfoScreen ("Buy")
  • CurrencyInfoErrorView ("OK")
  • FundingSelectionSheet ("USD Reserves", "Solana USDC With Phantom")
  • ForceUpgradeScreen ("Update Now")
  • TransactionDetailsModal ("Cancel Send")
  • PermissionScreen (dynamic title)
  • AccessKeyHelpScreen ("Open Photos")
  • OnrampSuccessScreen ("OK")
  • VerifyInfoScreen ("Next")
  • ModalCashReceived (dynamic title)

Test plan

  • Verify all updated screens render buttons correctly
  • Verify press states animate with the 0.7 opacity transition
  • Verify FundingSelectionSheet Phantom button still shows icon inline with text

Replace CodeButton(style: .filled) and CodeButton(style: .filledCustom)
with native Button + .buttonStyle(.filled) in 10 screens.
@raulriera raulriera changed the title Adopt FilledButtonStyle across more screens refactor: adopt FilledButtonStyle across more screens Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant