wip2

Author: MrRefactor

android/src/main/java/com/reactnativepagerview/PagerViewViewManager.kt

@@ -124,78 +124,25 @@ class PagerViewViewManager :
     }
 
     override fun onDropViewInstance(view: NestedScrollableHost) {
-        try {
-            val viewPager = PagerViewViewManagerImpl.getViewPager(view)
-            // Access private mRecyclerView field using reflection
-            val recyclerViewField = ViewPager2::class.java.getDeclaredField("mRecyclerView")
-            recyclerViewField.isAccessible = true
-            val recyclerView = recyclerViewField.get(viewPager) as RecyclerView
-
-            // Stop any scroll/drag in progress first
-            recyclerView.stopScroll()
-
-            // Suppress layout to prevent any pending layout operations from executing
-            recyclerView.suppressLayout(true)
-
-            // Access and stop the ViewFlinger directly via reflection - do this early
-            try {
-                val viewFlingerField = RecyclerView::class.java.getDeclaredField("mViewFlinger")
-                viewFlingerField.isAccessible = true
-                val viewFlinger = viewFlingerField.get(recyclerView) as? Runnable
-                viewFlinger?.let {
-                    recyclerView.removeCallbacks(it)
-                    // Also try to stop it via reflection if it has a stop method
-                    try {
-                        val stopMethod = viewFlinger.javaClass.getDeclaredMethod("stop")
-                        stopMethod.isAccessible = true
-                        stopMethod.invoke(viewFlinger)
-                    } catch (ignored: Exception) {}
-                }
-            } catch (ignored: Exception) {
-                // ViewFlinger reflection may fail on some versions
-            }
+        val vp = view.getChildAt(0) as? ViewPager2
+        val recyclerView = vp?.getChildAt(0) as? RecyclerView
 
-            // Clear the recycler's scrap views before clearing adapter
-            try {
-                val recyclerField = RecyclerView::class.java.getDeclaredField("mRecycler")
-                recyclerField.isAccessible = true
-                val recycler = recyclerField.get(recyclerView)
-
-                // Clear scrap heaps
-                val clearScrapMethod = recycler.javaClass.getDeclaredMethod("clear")
-                clearScrapMethod.isAccessible = true
-                clearScrapMethod.invoke(recycler)
-            } catch (ignored: Exception) {
-                // Recycler reflection may fail on some versions
-            }
+        // Stop any in-progress scroll/fling
+        recyclerView?.stopScroll()
 
-            // Clear any pending animations and layout transitions
-            recyclerView.clearAnimation()
-            recyclerView.itemAnimator = null
-            recyclerView.layoutTransition = null
-            recyclerView.recycledViewPool.clear()
-
-            // Remove all pending operations and choreographer callbacks
-            recyclerView.removeCallbacks(null)
-
-            // Try to clear any pending adapter updates
-            try {
-                val adapterHelperField = RecyclerView::class.java.getDeclaredField("mAdapterHelper")
-                adapterHelperField.isAccessible = true
-                val adapterHelper = adapterHelperField.get(recyclerView)
-                val resetMethod = adapterHelper.javaClass.getDeclaredMethod("reset")
-                resetMethod.isAccessible = true
-                resetMethod.invoke(adapterHelper)
-            } catch (ignored: Exception) {}
-
-            // Clear the adapter to prevent recycling during teardown
-            viewPager.adapter = null
-
-            // Unsuppress layout after clearing adapter (won't do anything as view is being
-            // destroyed)
-            recyclerView.suppressLayout(false)
-        } catch (e: Exception) {
-            // View might already be in an invalid state
+        // Disable item animations to prevent animator callbacks during teardown
+        recyclerView?.itemAnimator = null
+
+        // Clear cached views in the recycled pool
+        recyclerView?.recycledViewPool?.clear()
+
+        try {
+            // Clear adapter to prevent post-teardown fling callbacks.
+            // setAdapter(null) internally calls removeAndRecycleAllViews which
+            // can throw if views are still attached during mid-scroll teardown.
+            recyclerView?.adapter = null
+        } catch (_: IllegalArgumentException) {
+            // Safe to ignore during teardown — view is being destroyed
         }
         super.onDropViewInstance(view)
     }

example/src/MaterialTopTabExample.tsx

@@ -2,22 +2,43 @@ import React, { useState } from 'react';
 import { createNativeStackNavigator } from '@react-navigation/native-stack';
 import { createMaterialTopTabNavigator } from '@react-navigation/material-top-tabs';
 
-import { View, Text, Button } from 'react-native';
+import { View, Text, Button, FlatList, StyleSheet } from 'react-native';
 
-function Tab1() {
+const DATA = Array.from({ length: 50 }, (_, i) => ({
+  id: String(i),
+  title: `Item ${i + 1}`,
+}));
+
+function ScrollableTab({ name, color }: { name: string; color: string }) {
   return (
-    <View style={{ flex: 1, justifyContent: 'center', alignItems: 'center' }}>
-      <Text>Tab 1</Text>
-    </View>
+    <FlatList
+      data={DATA}
+      keyExtractor={(item) => item.id}
+      renderItem={({ item }) => (
+        <View style={[styles.item, { backgroundColor: color }]}>
+          <Text style={styles.itemText}>
+            {name} - {item.title}
+          </Text>
+        </View>
+      )}
+    />
   );
 }
 
-function Tab2(props: any) {
-  return (
-    <View style={{ flex: 1, justifyContent: 'center', alignItems: 'center' }}>
-      <Button title="Logout" onPress={props.onLogout} />
-    </View>
-  );
+function Tab1() {
+  return <ScrollableTab name="Tab 1" color="#ff6b6b" />;
+}
+
+function Tab2() {
+  return <ScrollableTab name="Tab 2" color="#4ecdc4" />;
+}
+
+function Tab3() {
+  return <ScrollableTab name="Tab 3" color="#45b7d1" />;
+}
+
+function Tab4() {
+  return <ScrollableTab name="Tab 4" color="#96ceb4" />;
 }
 
 const PreAuthScreen = (props: any) => {
@@ -30,19 +51,14 @@ const PreAuthScreen = (props: any) => {
 
 const PostAuthScreen = (props: any) => {
   const { Navigator, Screen } = createMaterialTopTabNavigator();
-  const onLogout = () => {
-    setTimeout(() => {
-      props.setIsSignedIn(false);
-    }, 0);
-  };
 
   return (
     <View style={{ flex: 1 }}>
       <Navigator>
         <Screen name="Tab1" component={Tab1} />
-        <Screen name="Tab2">
-          {(props: any) => <Tab2 {...props} onLogout={onLogout} />}
-        </Screen>
+        <Screen name="Tab2" component={Tab2} />
+        <Screen name="Tab3" component={Tab3} />
+        <Screen name="Tab4" component={Tab4} />
       </Navigator>
     </View>
   );
@@ -70,3 +86,17 @@ export function MaterialTopBarExample() {
     </Navigator>
   );
 }
+
+const styles = StyleSheet.create({
+  item: {
+    padding: 20,
+    marginVertical: 2,
+    marginHorizontal: 8,
+    borderRadius: 8,
+  },
+  itemText: {
+    fontSize: 16,
+    color: '#fff',
+    fontWeight: '600',
+  },
+});