[Android] Properly handle requestDisallowInterceptTouchEvent for v3

[j-piasecki]

Description

On Android, when a native view calls requestDisallowInterceptTouchEvent (e.g. a pager starting a swipe), RNGestureHandlerRootView reacted by cancelling all handlers registered on the root. react-native-pager-view calls it eagerly on touch down, so any v3 gesture rendered inside a pager (e.g. inside material top tabs) was cancelled before it could activate — a long press nested in top tabs never activated at all.

This PR makes the cancellation targeted:

• The root view now cancels only legacy handlers (v1/v2 action types) via the new GestureHandlerOrchestrator.cancelAllLegacyHandlers, instead of the old trick of activating the internal RootViewGestureHandler. The root handler is now attached with ACTION_TYPE_NONE so it's excluded from that sweep (and no longer sends dead events to JS).
• v3 handlers are cancelled by new requestDisallowInterceptTouchEvent overrides on RNGestureHandlerDetectorView and ButtonViewGroup. Since the request only bubbles upward from the requesting view, only handlers attached to its ancestors are cancelled — handlers below the requester (like the long press under the pager) keep working.
• Cancellation is skipped while the orchestrator is delivering events (isHandlingTouch), mirroring the existing passingTouch guard, so disallow requests caused by RNGH's own event delivery don't cancel gestures.
• findGestureHandlerRootView now returns the nearest enabled root view so the checks above consult the orchestrator that actually manages the subtree.

Test plan

Tested on reproducer from #2383

Screen.Recording.2026-07-31.at.11.38.38.mov

[Copilot]

Pull request overview

This PR adjusts Android’s requestDisallowInterceptTouchEvent handling to avoid canceling v3 gestures too broadly (notably inside pagers), by making cancellation more targeted across root/orchestrator and v3 detector/button views.

Changes:

• Add GestureHandlerOrchestrator.cancelAllLegacyHandlers() and use it from the root helper instead of triggering cancellation via the internal root handler.
• Override requestDisallowInterceptTouchEvent in v3 host detector and button view to cancel only relevant handlers (and skip cancellation while the orchestrator is handling touch).
• Update root view lookup to prefer the nearest enabled RNGestureHandlerRootView and expose the orchestrator via the root view/root helper.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootView.kt	Exposes orchestrator and returns nearest enabled GH root view when searching ancestors.
packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootHelper.kt	Switches root-level cancellation to cancelAllLegacyHandlers and attaches root handler with ACTION_TYPE_NONE.
packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerDetectorView.kt	Adds requestDisallowInterceptTouchEvent override to cancel v3 handlers attached via the host detector.
packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerButtonViewManager.kt	Adds requestDisallowInterceptTouchEvent override to cancel the button’s managed v3 handler.
packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/core/GestureHandlerOrchestrator.kt	Introduces cancelAllLegacyHandlers() to selectively cancel v1/v2 action types.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: beec65eb-e680-492b-8610-5310e0a4cb44

📥 Commits

Reviewing files that changed from the base of the PR and between dd2e6bc and dc49bcc.

📒 Files selected for processing (5)

• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/core/GestureHandlerOrchestrator.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerButtonViewManager.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerDetectorView.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootHelper.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootView.kt

🚧 Files skipped from review as they are similar to previous changes (5)

• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootHelper.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerButtonViewManager.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootView.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerDetectorView.kt
• packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/core/GestureHandlerOrchestrator.kt

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Improved gesture cancellation when native views request disallowing touch interception.
  • Prevented stale gesture handlers from remaining active during touch transitions.
  • Improved gesture-handler root detection, including handling disabled roots.
  • Updated legacy gesture handling to cancel applicable handlers and clean them up reliably.
  • Improved touch behavior when switching between native views and gesture handlers.

Walkthrough

The Android gesture-handler implementation now exposes orchestrator state, centralizes legacy-handler cancellation, and cancels attached handlers during disallowed touch interception. Root lookup now prefers enabled gesture-handler roots.

Changes

Native interception cancellation

Layer / File(s)	Summary
Orchestrator cancellation API packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/core/GestureHandlerOrchestrator.kt	isHandlingTouch is publicly readable. cancelAllLegacyHandlers() cancels legacy handlers and schedules cleanup.
Root helper integration packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootHelper.kt	The root helper exposes its orchestrator, registers the JS handler with ACTION_TYPE_NONE, uses the new cancellation method, and removes the obsolete local helper.
Interception entry points and root lookup packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerRootView.kt, packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerDetectorView.kt, packages/react-native-gesture-handler/android/src/main/java/com/swmansion/gesturehandler/react/RNGestureHandlerButtonViewManager.kt	Views propagate active disallow-intercept requests and cancel attached handlers when the orchestrator is not handling touch events. Root lookup prefers enabled roots.

Sequence Diagram(s)

sequenceDiagram
  participant ButtonViewGroup
  participant RNGestureHandlerDetectorView
  participant RNGestureHandlerRootView
  participant GestureHandlerOrchestrator
  ButtonViewGroup->>RNGestureHandlerDetectorView: requestDisallowInterceptTouchEvent(true)
  RNGestureHandlerDetectorView->>RNGestureHandlerRootView: find gesture-handler root
  RNGestureHandlerRootView->>GestureHandlerOrchestrator: read isHandlingTouch
  RNGestureHandlerDetectorView->>GestureHandlerOrchestrator: cancel attached handlers

Loading

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the Android v3 changes to requestDisallowInterceptTouchEvent, which is the primary focus of the pull request.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

_{Comment @coderabbitai help to get the list of available commands.}