fix: onSnapToItem gives floating index

fix #13
dohooo · Oct 27, 2021 · 13d6c64 · 13d6c64
1 parent c9b6eee
commit 13d6c64
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/Carousel.tsx b/src/Carousel.tsx
@@ -133,20 +133,30 @@ function Carousel<T extends unknown = any>(
     const {
         height = '100%',
         data: _data = [],
-        width,
         loop = true,
         mode = 'default',
         renderItem,
         autoPlay,
         autoPlayReverse,
-        autoPlayInterval,
+        autoPlayInterval = 1000,
         parallaxScrollingOffset,
         parallaxScrollingScale,
         onSnapToItem,
         style,
         timingConfig = defaultTimingConfig,
         panGestureHandlerProps = {},
     } = props;
+
+    if (
+        typeof timingConfig.duration === 'number' &&
+        timingConfig.duration > autoPlayInterval
+    ) {
+        throw Error(
+            'The during time of animation  must less than autoplay interval.'
+        );
+    }
+
+    const width = Math.round(props.width);
     const lockController = useLockController();
     const handlerOffsetX = useSharedValue<number>(0);
     const data = React.useMemo<T[]>(() => {

diff --git a/src/useAutoPlay.ts b/src/useAutoPlay.ts
@@ -10,7 +10,7 @@ export function useAutoPlay(opts: {
     const {
         autoPlay = false,
         autoPlayReverse = false,
-        autoPlayInterval = 1000,
+        autoPlayInterval,
         carouselController,
     } = opts;