fix(android): touch events on android new arch

Author: lodev09

android/src/main/java/com/lodev09/truesheet/TrueSheetView.kt

@@ -19,14 +19,19 @@ import com.lodev09.truesheet.core.Utils
 class TrueSheetView(context: Context) :
   ViewGroup(context),
   LifecycleEventListener {
-  private var eventDispatcher: EventDispatcher? = null
 
   private val reactContext: ThemedReactContext
     get() = context as ThemedReactContext
 
   private val surfaceId: Int
     get() = UIManagerHelper.getSurfaceId(this)
 
+  var eventDispatcher: EventDispatcher?
+    get() = rootSheetView.eventDispatcher
+    set(eventDispatcher) {
+      rootSheetView.eventDispatcher = eventDispatcher
+    }
+
   var initialIndex: Int = -1
   var initialIndexAnimated: Boolean = true
 
@@ -62,11 +67,8 @@ class TrueSheetView(context: Context) :
 
   init {
     reactContext.addLifecycleEventListener(this)
-    eventDispatcher = UIManagerHelper.getEventDispatcherForReactTag(reactContext, id)
 
     rootSheetView = RootSheetView(context)
-    rootSheetView.eventDispatcher = eventDispatcher
-
     sheetDialog = TrueSheetDialog(reactContext, rootSheetView)
 
     // Configure Sheet Dialog
@@ -173,6 +175,13 @@ class TrueSheetView(context: Context) :
     // Do nothing as we are laid out by UIManager
   }
 
+  override fun setId(id: Int) {
+    super.setId(id)
+
+    // Forward the ID to our content view, so event dispatching behaves correctly
+    rootSheetView.id = id
+  }
+
   override fun onAttachedToWindow() {
     super.onAttachedToWindow()
 
@@ -193,7 +202,7 @@ class TrueSheetView(context: Context) :
 
   override fun onDetachedFromWindow() {
     super.onDetachedFromWindow()
-    sheetDialog.dismiss()
+    onDropInstance()
   }
 
   override fun addView(child: View, index: Int) {
@@ -219,7 +228,6 @@ class TrueSheetView(context: Context) :
 
   override fun removeViewAt(index: Int) {
     UiThreadUtil.assertOnUiThread()
-
     val child = getChildAt(index)
     rootSheetView.removeView(child)
   }
@@ -229,14 +237,12 @@ class TrueSheetView(context: Context) :
     // Those will be handled by the rootView which lives in the dialog
   }
 
-  override fun dispatchPopulateAccessibilityEvent(event: AccessibilityEvent): Boolean {
-    // Explicitly override this to prevent accessibility events being passed down to children
-    // Those will be handled by the rootView which lives in the dialog
-    return false
-  }
+  // Explicitly override this to prevent accessibility events being passed down to children
+  // Those will be handled by the mHostView which lives in the dialog
+  public override fun dispatchPopulateAccessibilityEvent(event: AccessibilityEvent): Boolean = false
 
   override fun onHostResume() {
-    // do nothing
+    configureIfShowing()
   }
 
   override fun onHostPause() {
@@ -245,6 +251,10 @@ class TrueSheetView(context: Context) :
 
   override fun onHostDestroy() {
     // Drop the instance if the host is destroyed which will dismiss the dialog
+    onDropInstance()
+  }
+
+  fun onDropInstance() {
     reactContext.removeLifecycleEventListener(this)
     sheetDialog.dismiss()
   }
@@ -311,7 +321,7 @@ class TrueSheetView(context: Context) :
     eventDispatcher?.dispatchEvent(TrueSheetEvent(surfaceId, id, name, data))
   }
 
-  private fun configureIfShowing() {
+  fun configureIfShowing() {
     if (sheetDialog.isShowing) {
       sheetDialog.configure()
       sheetDialog.positionFooter()

android/src/main/java/com/lodev09/truesheet/TrueSheetViewManager.kt

@@ -8,6 +8,7 @@ import com.facebook.react.bridge.ReadableArray
 import com.facebook.react.bridge.ReadableType
 import com.facebook.react.common.MapBuilder
 import com.facebook.react.uimanager.ThemedReactContext
+import com.facebook.react.uimanager.UIManagerHelper
 import com.facebook.react.uimanager.ViewGroupManager
 import com.facebook.react.uimanager.annotations.ReactProp
 import com.lodev09.truesheet.core.Utils
@@ -19,7 +20,19 @@ class TrueSheetViewManager : ViewGroupManager<TrueSheetView>() {
 
   override fun onDropViewInstance(view: TrueSheetView) {
     super.onDropViewInstance(view)
-    view.onHostDestroy()
+    view.onDropInstance()
+  }
+
+  override fun addEventEmitters(reactContext: ThemedReactContext, view: TrueSheetView) {
+    val dispatcher = UIManagerHelper.getEventDispatcherForReactTag(reactContext, view.id)
+    dispatcher?.let {
+      view.eventDispatcher = it
+    }
+  }
+
+  override fun onAfterUpdateTransaction(view: TrueSheetView) {
+    super.onAfterUpdateTransaction(view)
+    view.configureIfShowing()
   }
 
   override fun getExportedCustomDirectEventTypeConstants(): MutableMap<String, Any> =

android/src/main/java/com/lodev09/truesheet/core/RootSheetView.kt

@@ -32,8 +32,8 @@ class RootSheetView(private val context: Context?) :
 
   private val jSTouchDispatcher = JSTouchDispatcher(this)
   private var jSPointerDispatcher: JSPointerDispatcher? = null
-  var sizeChangeListener: ((w: Int, h: Int) -> Unit)? = null
 
+  var sizeChangeListener: ((w: Int, h: Int) -> Unit)? = null
   var eventDispatcher: EventDispatcher? = null
 
   private val reactContext: ThemedReactContext
@@ -59,46 +59,42 @@ class RootSheetView(private val context: Context?) :
   }
 
   override fun onInterceptTouchEvent(event: MotionEvent): Boolean {
-    eventDispatcher?.let { jSTouchDispatcher.handleTouchEvent(event, it) }
-    jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, true)
+    eventDispatcher?.let { eventDispatcher ->
+      jSTouchDispatcher.handleTouchEvent(event, eventDispatcher, reactContext)
+      jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, true)
+    }
     return super.onInterceptTouchEvent(event)
   }
 
   @SuppressLint("ClickableViewAccessibility")
   override fun onTouchEvent(event: MotionEvent): Boolean {
-    eventDispatcher?.let { jSTouchDispatcher.handleTouchEvent(event, it) }
-    jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, false)
+    eventDispatcher?.let { eventDispatcher ->
+      jSTouchDispatcher.handleTouchEvent(event, eventDispatcher, reactContext)
+      jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, false)
+    }
     super.onTouchEvent(event)
-
     // In case when there is no children interested in handling touch event, we return true from
     // the root view in order to receive subsequent events related to that gesture
     return true
   }
 
   override fun onInterceptHoverEvent(event: MotionEvent): Boolean {
-    jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, true)
+    eventDispatcher?.let { jSPointerDispatcher?.handleMotionEvent(event, it, true) }
     return super.onHoverEvent(event)
   }
 
   override fun onHoverEvent(event: MotionEvent): Boolean {
-    jSPointerDispatcher?.handleMotionEvent(event, eventDispatcher, false)
+    eventDispatcher?.let { jSPointerDispatcher?.handleMotionEvent(event, it, false) }
     return super.onHoverEvent(event)
   }
 
-  @Deprecated("Deprecated in Java")
-  override fun onChildStartedNativeGesture(ev: MotionEvent) {
-    eventDispatcher?.let {
-      if (ev != null) {
-        jSTouchDispatcher.onChildStartedNativeGesture(ev, it)
-      }
+  override fun onChildStartedNativeGesture(childView: View, ev: MotionEvent) {
+    eventDispatcher?.let { eventDispatcher ->
+      jSTouchDispatcher.onChildStartedNativeGesture(ev, eventDispatcher)
+      jSPointerDispatcher?.onChildStartedNativeGesture(childView, ev, eventDispatcher)
     }
   }
 
-  override fun onChildStartedNativeGesture(childView: View?, ev: MotionEvent) {
-    eventDispatcher?.let { jSTouchDispatcher.onChildStartedNativeGesture(ev, it) }
-    jSPointerDispatcher?.onChildStartedNativeGesture(childView, ev, eventDispatcher)
-  }
-
   override fun onChildEndedNativeGesture(childView: View, ev: MotionEvent) {
     eventDispatcher?.let { jSTouchDispatcher.onChildEndedNativeGesture(ev, it) }
     jSPointerDispatcher?.onChildEndedNativeGesture()

example/ios/Podfile.lock

@@ -1214,7 +1214,7 @@ PODS:
     - React-Core
   - react-native-safe-area-context (5.2.0):
     - React-Core
-  - react-native-true-sheet (2.0.2):
+  - react-native-true-sheet (2.0.3):
     - DoubleConversion
     - glog
     - hermes-engine
@@ -1923,7 +1923,7 @@ SPEC CHECKSUMS:
   React-microtasksnativemodule: 72564d5469003687d39bfc4efad281df8efc0684
   react-native-maps: ee1e65647460c3d41e778071be5eda10e3da6225
   react-native-safe-area-context: 849d7df29ecb2a7155c769c0b76849ba952c2aa3
-  react-native-true-sheet: 7fb131224336979607e4d81c523e07f618ae3a71
+  react-native-true-sheet: b5cc3652405dd472035b5af191857179c47d56d8
   React-nativeconfig: cb207ebba7cafce30657c7ad9f1587a8f32e4564
   React-NativeModulesApple: 82a8bee52df9f5b378195a500f22be3a6ef0f890
   React-perflogger: 8152bab3f0eb4b8751f282f9af7caed2c823a9ea