Skip to content

fix: polish wallet ui - #1270

Draft
pwltr wants to merge 22 commits into
masterfrom
fix/polish
Draft

pwltr wants to merge 22 commits into
masterfrom
fix/polish

Conversation

@pwltr

@pwltr pwltr commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Description

This PR polishes the current wallet UI stack:

  • Aligns button text styles, small button sizing, scanner/send icon button backgrounds, selected tab indicator color, and money symbol styling.
  • Tightens amount screen, transfer flow, wallet activity, invoice tag, receive confirmation, boost sheet, and activity detail spacing/typography.
  • Updates onboarding/settings intro copy, translations, sheet intro layout behavior, and restore wallet mnemonic emphasis.
  • Keeps a changelog fragment for the wallet polish work and restores the existing incoming payment request fragment from master.

Design

Mapped screen references are listed in docs/screens-map.md for the touched screens, including:

  • HomeScreen.kt → Wallet On-chain › Wallet BTC
  • SavingsWalletScreen.kt → Savings, Spending, Hardware Balances › Savings Populated
  • SpendingWalletScreen.kt → Savings, Spending, Hardware Balances › Spending Ready
  • SpendingAmountScreen.kt / SpendingAdvancedScreen.kt / SpendingConfirmScreen.kt → Transfer to Spending Balance From Savings frames
  • EditInvoiceScreen.kt / ReceiveConfirmScreen.kt / ReceiveQrScreen.kt → receive/send invoice frames
  • ActivityDetailScreen.kt / AllActivityScreen.kt → transaction/activity frames
  • RestoreWalletScreen.kt / OnboardingSlidesScreen.kt / CreateWalletScreen.kt → onboarding frames
  • BackupIntroScreen.kt / QuickPayIntroScreen.kt → settings intro frames

Out-of-map sheets and shared components were cross-checked against the existing iOS implementation and user-provided screenshots during implementation.

Preview

QA Notes

Manual Tests

  • Open the wallet home screen and verify the section spacing between balances, activity, and action areas remains visually consistent.
  • Open Savings and Spending wallet screens and verify the transfer button has the intended space before the first activity row.
  • Walk through the transfer-to-spending amount, advanced, and confirm screens; verify title/amount spacing, number pad spacing, amount symbols, and coin image tilt on slider drag.
  • Open receive invoice editing and verify tags sit in the same row as the add tag button, note/add-tag input styling matches, and selected tag spacing is correct.
  • Open transaction details and the boost sheet; verify the boost icon, fee amount symbol, long duration text, outline suggested-fee button, and single top toast behavior.
  • Walk through onboarding/settings intro screens and restore wallet; verify copy, translations, sheet image placement, and focused mnemonic word weight.

Automated Checks

  • ./gradlew compileDevDebugKotlin

content()
}
if (app != null) {
if (showToastOverlay && app != null) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully this is the right fix for #1267.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this default to false for all sheets?

} else {
Modifier.graphicsLayer {
colorFilter = ColorFilter.tint(Colors.White32)
CompositionLocalProvider(LocalMinimumInteractiveComponentSize provides 0.dp) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixed small button text and disabled styles

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.

2 participants