Skip to content

feat: generic openAsSheet support with auto-adapting dismiss style#750

Merged
bmc08gt merged 1 commit into
code/cashfrom
feat/open-as-sheet
May 21, 2026
Merged

feat: generic openAsSheet support with auto-adapting dismiss style#750
bmc08gt merged 1 commit into
code/cashfrom
feat/open-as-sheet

Conversation

@bmc08gt
Copy link
Copy Markdown
Collaborator

@bmc08gt bmc08gt commented May 21, 2026

  • Add openAsSheet(route) to open any AppRoute as a modal bottom sheet with animated dismiss-replace when a sheet is already open
  • Add FlowDismissStyle composition local so FlowHost auto-swaps back arrow for close (X) when at a sheet root first step
  • Rename navigateTo(list) to navigateAll for clarity; remove single-route navigateTo overload
  • Widen Main.Sheet.initialRoute from Sheets to AppRoute
  • Remove duplicate Sheets.TokenDiscovery route and TokenDiscoverySheet()
  • Remove dead Menu.Deposit route
  • Fix SaveableStateProvider duplicate-key crash in pendingSheetDismiss by removing sheetGeneration from key() block and using it as LaunchedEffect key instead
  • Guard onDismissRequest during pendingSheetDismiss to prevent double onBack()

@bmc08gt bmc08gt self-assigned this May 21, 2026
@github-actions github-actions Bot added type: feature New functionality area: payments Payments, transfers, intents, billing area: auth Login, session, access keys, identity area: ui Compose UI, theme, components, resources area: onramp Deposit, purchase, Coinbase, fiat on-ramp area: scanner QR/Kikcode scanning, camera area: tokens Token accounts, balances, token info area: deeplinks Deep link handling, URL routing, and link parsing and removed type: feature New functionality labels May 21, 2026
@bmc08gt bmc08gt force-pushed the feat/open-as-sheet branch from 5bc2fce to 503df1d Compare May 21, 2026 17:57
@github-actions github-actions Bot added the type: feature New functionality label May 21, 2026
- Add openAsSheet(route) to open any AppRoute as a modal bottom sheet
  with animated dismiss-replace when a sheet is already open
- Add FlowDismissStyle composition local so FlowHost auto-swaps back
  arrow for close (X) when at a sheet root first step
- Rename navigateTo(list) to navigateAll for clarity; remove single-route
  navigateTo overload
- Widen Main.Sheet.initialRoute from Sheets to AppRoute
- Remove duplicate Sheets.TokenDiscovery route and TokenDiscoverySheet()
- Remove dead Menu.Deposit route
- Fix SaveableStateProvider duplicate-key crash in pendingSheetDismiss by
  removing sheetGeneration from key() block and using it as
  LaunchedEffect key instead
- Guard onDismissRequest during pendingSheetDismiss to prevent double
  onBack()
@bmc08gt bmc08gt force-pushed the feat/open-as-sheet branch from 503df1d to fa43e3e Compare May 21, 2026 18:05
@bmc08gt bmc08gt merged commit 58852dd into code/cash May 21, 2026
3 checks passed
@bmc08gt bmc08gt deleted the feat/open-as-sheet branch May 21, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: auth Login, session, access keys, identity area: deeplinks Deep link handling, URL routing, and link parsing area: onramp Deposit, purchase, Coinbase, fiat on-ramp area: payments Payments, transfers, intents, billing area: scanner QR/Kikcode scanning, camera area: tokens Token accounts, balances, token info area: ui Compose UI, theme, components, resources type: feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant