remove unnecessary hasMoved state, fix case where tap on same item without moving

Author: computerjazz

src/components/DraggableFlatList.tsx

@@ -78,7 +78,6 @@ function DraggableFlatListInner<T>(props: DraggableFlatListProps<T>) {
     activeIndexAnim,
     containerSize,
     disabled,
-    hasMoved,
     panGestureState,
     resetTouchedCell,
     scrollOffset,
@@ -245,7 +244,6 @@ function DraggableFlatListInner<T>(props: DraggableFlatListProps<T>) {
         set(disabled, 1),
         set(isTouchActiveRef.current.native, 0),
         call([activeIndexAnim], onRelease),
-        cond(not(hasMoved), call([activeIndexAnim], resetHoverState)),
       ],
       [call([activeIndexAnim], resetHoverState), resetTouchedCell]
     )
@@ -323,17 +321,13 @@ function DraggableFlatListInner<T>(props: DraggableFlatListProps<T>) {
                 eq(panGestureState, GestureState.ACTIVE),
                 not(disabled)
               ),
-              [
-                cond(not(hasMoved), set(hasMoved, 1)),
-                set(touchAbsolute, props.horizontal ? x : y),
-              ]
+              [set(touchAbsolute, props.horizontal ? x : y)]
             ),
         },
       ]),
     [
       activeIndexAnim,
       disabled,
-      hasMoved,
       panGestureState,
       props.horizontal,
       touchAbsolute,

src/context/animatedValueContext.tsx

@@ -57,7 +57,6 @@ function useSetupAnimatedValues<T>() {
 
   const isTouchActiveNative = useValue<number>(0);
 
-  const hasMoved = useValue<number>(0);
   const disabled = useValue<number>(0);
 
   const horizontalAnim = useValue(props.horizontal ? 1 : 0);
@@ -102,7 +101,6 @@ function useSetupAnimatedValues<T>() {
       set(touchInit, 0),
       set(activeCellOffset, 0),
       set(activationDistance, 0),
-      set(hasMoved, 0),
     ])
   );
 
@@ -114,7 +112,6 @@ function useSetupAnimatedValues<T>() {
       activeIndexAnim,
       containerSize,
       disabled,
-      hasMoved,
       horizontalAnim,
       hoverAnim,
       hoverAnimConstrained,
@@ -140,7 +137,6 @@ function useSetupAnimatedValues<T>() {
       activeIndexAnim,
       containerSize,
       disabled,
-      hasMoved,
       horizontalAnim,
       hoverAnim,
       hoverAnimConstrained,

src/hooks/useCellTranslate.tsx

@@ -91,7 +91,7 @@ export function useCellTranslate({ cellIndex, cellSize, cellOffset }: Params) {
     )
   );
 
-  // This is required to continually evaluate values
+  // This is a workaround required to continually evaluate values
   useCode(
     () =>
       block([