Skip to content

Arbitrate pull gestures inside Bottom Sheet - #10

Merged
Cylop merged 1 commit into
mainfrom
fix/sheet-ptr-gesture-arbitration
Sep 5, 2026
Merged

Arbitrate pull gestures inside Bottom Sheet#10
Cylop merged 1 commit into
mainfrom
fix/sheet-ptr-gesture-arbitration

Conversation

@Cylop

@Cylop Cylop commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • stop the Bottom Sheet feed's pointer-down from bubbling into the sheet drag controller
  • preserve native defaults so ordinary list scrolling still works
  • keep the sheet handle as the explicit sheet-drag surface
  • document the integration requirement
  • add a gradual movement regression that reproduces the ownership race

Root cause

Bottom Sheet claims downward content-boundary movement on the first positive pixel, while Pull to Refresh intentionally waits through a 6 px intent slop. On gradual pulls the sheet therefore captured first. This integration explicitly assigns the feed surface to scrolling/PTR and the handle to sheet dragging.

Verification

  • npm run check
  • focused Chromium tests for native scrolling and gradual downward gesture ownership

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
react-pull-to-refresh Ready Ready Preview Sep 5, 2026 1:44pm UTC

@Cylop
Cylop merged commit 5eb3042 into main Sep 5, 2026
4 checks passed
@Cylop
Cylop deleted the fix/sheet-ptr-gesture-arbitration branch September 5, 2026 13:45
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