perf tweaks

Author: computerjazz

src/components/CellRendererComponent.tsx

@@ -142,16 +142,11 @@ function CellRendererComponent<T>(props: Props<T>) {
         isAndroid && { elevation: isActive ? 1 : 0 },
         { flexDirection: horizontal ? "row" : "column" },
         (isWeb || isIOS) && { zIndex: isActive ? 999 : 0 },
+        style,
       ]}
       pointerEvents={activeKey ? "none" : "auto"}
     >
-      <Animated.View
-        // Including both animated styles and non-animated styles causes react-native-web
-        // to ignore updates in non-animated styles. Solution is to separate anima
-        style={style}
-      >
-        <CellProvider isActive={isActive}>{children}</CellProvider>
-      </Animated.View>
+      <CellProvider isActive={isActive}>{children}</CellProvider>
     </Animated.View>
   );
 }

src/components/DraggableFlatList.tsx

@@ -402,7 +402,7 @@ function DraggableFlatListInner<T>(props: DraggableFlatListProps<T>) {
             extraData={extraData}
             keyExtractor={keyExtractor}
             onScroll={scrollHandler}
-            scrollEventThrottle={1}
+            scrollEventThrottle={16}
             simultaneousHandlers={props.simultaneousHandlers}
             removeClippedSubviews={false}
           />

src/procs.ts

@@ -203,25 +203,26 @@ export const setupCell = proc(
             set(finished, 0),
             startClock(clock),
           ]),
+
+          cond(neq(prevSpacerIndex, spacerIndex), [
+            cond(eq(spacerIndex, -1), [
+              // Hard reset to prevent stale state bugs
+              cond(clockRunning(clock), stopClock(clock)),
+              hardReset(position, finished, time, toValue),
+            ]),
+          ]),
+          cond(finished, [onFinished, set(time, 0), set(finished, 0)]),
+          set(prevSpacerIndex, spacerIndex),
+          set(prevToValue, toValue),
+          set(prevIsDraggingCell, isDraggingCell),
+          cond(clockRunning(clock), runSpring),
         ],
         [
-          // Reset the spacer index when drag ends
-          set(spacerIndex, -1),
-          set(position, 0),
+          // // Reset the spacer index when drag ends
+          cond(neq(spacerIndex, -1), set(spacerIndex, -1)),
+          cond(neq(position, 0), set(position, 0)),
         ]
       ),
-      cond(neq(prevSpacerIndex, spacerIndex), [
-        cond(eq(spacerIndex, -1), [
-          // Hard reset to prevent stale state bugs
-          cond(clockRunning(clock), stopClock(clock)),
-          hardReset(position, finished, time, toValue),
-        ]),
-      ]),
-      cond(finished, [onFinished, set(time, 0), set(finished, 0)]),
-      set(prevSpacerIndex, spacerIndex),
-      set(prevToValue, toValue),
-      set(prevIsDraggingCell, isDraggingCell),
-      cond(clockRunning(clock), runSpring),
       position,
     ])
 );