Skip to content

fix(compose): fix stale lambda captures and effect key issues#836

Merged
bmc08gt merged 1 commit into
code/cashfrom
fix/compose-stale-lambdas-effect-keys
Jun 3, 2026
Merged

fix(compose): fix stale lambda captures and effect key issues#836
bmc08gt merged 1 commit into
code/cashfrom
fix/compose-stale-lambdas-effect-keys

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator
  • Fix stale onStateChanged lambda in TextInput by using rememberUpdatedState, preventing stale closure in LaunchedEffect(Unit)
  • Fix stale onDelete lambda in SwipeToRevealItem by using rememberUpdatedState for the captured callback
  • Replace LaunchedEffect(swipeFraction) in SlideToConfirm with snapshotFlow-based collection to avoid cancelling/restarting the effect on every drag frame
  • Key permission screen LaunchedEffects on permissionState.status instead of Unit so navigation fires when status settles async

- Fix stale onStateChanged lambda in TextInput by using
  rememberUpdatedState, preventing stale closure in LaunchedEffect(Unit)
- Fix stale onDelete lambda in SwipeToRevealItem by using
  rememberUpdatedState for the captured callback
- Replace LaunchedEffect(swipeFraction) in SlideToConfirm with
  snapshotFlow-based collection to avoid cancelling/restarting the
  effect on every drag frame
- Key permission screen LaunchedEffects on permissionState.status
  instead of Unit so navigation fires when status settles async
@bmc08gt bmc08gt self-assigned this Jun 3, 2026
@github-actions github-actions Bot added type: fix Bug fix area: ui Compose UI, theme, components, resources area: onboarding and removed type: fix Bug fix labels Jun 3, 2026
@bmc08gt bmc08gt merged commit e7119fe into code/cash Jun 3, 2026
3 checks passed
@bmc08gt bmc08gt deleted the fix/compose-stale-lambdas-effect-keys branch June 3, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: onboarding area: ui Compose UI, theme, components, resources

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant