fix: initial snap points when it is 100% (#105)

gorhom · Dec 13, 2020 · b6b972a · b6b972a
1 parent 6840a47
commit b6b972a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/components/bottomSheet/useTransition.ts b/src/components/bottomSheet/useTransition.ts
@@ -65,7 +65,7 @@ export const useTransition = ({
   const clock = useClock();
   const config = useMemo(
     () => ({
-      toValue: new Animated.Value(0),
+      toValue: new Animated.Value(-1),
       duration: animationDuration,
       easing: animationEasing,
     }),

diff --git a/src/hooks/useNormalizedSnapPoints.ts b/src/hooks/useNormalizedSnapPoints.ts
@@ -8,21 +8,12 @@ export const useNormalizedSnapPoints = (
   handleHeight: number = 0
 ) =>
   useMemo(() => {
-    let normalizedSnapPoints = normalizeSnapPoints(
+    const normalizedSnapPoints = normalizeSnapPoints(
       snapPoints,
       containerHeight,
       topInset
     );
     return normalizedSnapPoints.map(normalizedSnapPoint => {
-      /**
-       * if user sets point to zero and `excludeHandleHeight` true,
-       * we subset handleHeight from the `normalizedSnapPoint` to make
-       * sure that sheets and its handle will be out of the screen.
-       */
-      if (normalizedSnapPoint === 0 && handleHeight !== 0) {
-        normalizedSnapPoint = normalizedSnapPoint - handleHeight;
-      }
-
       return Math.ceil(
         Math.max(containerHeight - normalizedSnapPoint - handleHeight, topInset)
       );