Skip to content

In the trigger or action list, scroll attempts move items too easily#2149

Merged
sds100 merged 2 commits into
developfrom
claude/issue-2074-fix-scroll-drag
Jun 1, 2026
Merged

In the trigger or action list, scroll attempts move items too easily#2149
sds100 merged 2 commits into
developfrom
claude/issue-2074-fix-scroll-drag

Conversation

@keymapper-ai
Copy link
Copy Markdown
Collaborator

Closes #2074

Summary

  • Root cause: The .draggable() modifier was attached to the entire ElevatedCard in both ActionListItem and TriggerKeyListItem. Because startDragImmediately = false only adds a small pointer-slop threshold, any ordinary swipe gesture on the card body was intercepted as a reorder drag instead of being propagated to the parent LazyColumn as a scroll.
  • Fix: Removed .draggable() from the card. Added it exclusively to the DragHandle icon with startDragImmediately = true, so:
    • Swiping anywhere on the item body → list scrolls normally.
    • Touching the drag handle and dragging → item reorders immediately (no long-press required from the handle).
    • Long-pressing anywhere on the list → item reorders via the existing dragContainer / detectDragGesturesAfterLongPress mechanism (unchanged).

Known limitations / areas for human review

  • The drag handle hit target remains 24 dp × 24 dp. If users find this too small, consider wrapping the icon in a Box with a larger Modifier.minimumInteractiveComponentSize() or explicit padding. This can be a follow-up.
  • Only ActionListItem and TriggerKeyListItem were changed. If other list items in the app use the same .draggable()-on-card pattern they may have the same issue and should be reviewed.

Generated by Claude Code

claude and others added 2 commits May 31, 2026 00:18
Previously the .draggable() modifier was applied to the entire ElevatedCard,
causing any vertical swipe on an item to start a reorder instead of scrolling
the parent LazyColumn. Moving the modifier to the DragHandle icon means:
- touch anywhere on the card body → list scrolls normally
- touch the drag handle and swipe → item reorders immediately
- long-press anywhere on the list → item reorders (unchanged via dragContainer)
@sds100 sds100 merged commit 8b536ca into develop Jun 1, 2026
4 checks passed
@sds100 sds100 deleted the claude/issue-2074-fix-scroll-drag branch June 1, 2026 14:28
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.

In the trigger or action list, scroll attempts move items too easily

3 participants