fix: prevent animations callback from being run only episodically

This issue was caused by a misuse of the `finished` value provided to `Animated.some_method().start()`'s callback: it simply indicates if an animation was run without interruption. It's it not a value that indicates if the animation is done being run and a callback can now be called.
As explain in the docs, simply providing a callback is actually enough, there was no need to check for `finished` truthiness; and in our case, that was actually causing bugs.

See: https://reactnative.dev/docs/animated#working-with-animations.

Fixes #120

example/src/App.tsx

@@ -6,9 +6,12 @@ import DemoModal from './components/DemoModal'
 import IntroModal from './components/IntroModal'
 import IntroButton from './components/IntroButton'
 
-interface Modal<N, C extends 'lightsalmon' | 'deepskyblue' | 'deeppink'> {
+interface Modal<N, C> {
   origin: 'Hooks' | 'Class' | 'Plain JS'
-  color: C
+  color: C | 'darkgreen'
+  // 👆 Comment this one and uncomment that one 👇 to remove all the TypeScript errors
+  // color: 'lightsalmon' | 'deepskyblue' | 'deeppink' | 'darkgreen'
+  // Note: the TS errors were left voluntarily to showcase the type inference & autocomplete possibilities
   name: N
 }
 

src/lib/ModalStack.tsx

@@ -44,12 +44,10 @@ const ModalStack = <P extends ModalfyParams>(props: Props<P>) => {
         easing: Easing.inOut(Easing.ease),
         duration: backdropAnimationDuration,
         useNativeDriver: true,
-      }).start(({ finished }) => {
-        if (finished) {
-          setStackStatus('hidden')
-          setBackdropClosedItems([])
-          translateY.setValue(sh(100))
-        }
+      }).start(() => {
+        setStackStatus('hidden')
+        setBackdropClosedItems([])
+        translateY.setValue(sh(100))
       })
     }
   }, [backdropAnimationDuration, opacity, translateY, stackStatus])

src/lib/StackItem.tsx

@@ -129,11 +129,9 @@ const StackItem = <P extends ModalfyParams>({
           toValue,
           useNativeDriver: true,
           ...(closeModalCallback ? animateOutConfig : animateInConfig),
-        }).start(({ finished }) => {
-          if (finished) {
-            closeModalCallback?.(stackItem)
-            modalStackItemCallback?.()
-          }
+        }).start(() => {
+          closeModalCallback?.(stackItem)
+          modalStackItemCallback?.()
         })
       }
     },
@@ -203,11 +201,9 @@ const StackItem = <P extends ModalfyParams>({
           toValue,
           useNativeDriver: true,
           ...animateOutConfig,
-        }).start(({ finished }) => {
-          if (finished) {
-            onCloseListener.current({ type: 'closeModal', origin: 'fling' })
-            closeModal(stackItem)
-          }
+        }).start(() => {
+          onCloseListener.current({ type: 'closeModal', origin: 'fling' })
+          closeModal(stackItem)
         })
       }
     },

src/types.ts

@@ -269,9 +269,7 @@ export interface ModalOptions {
    *       easing: Easing.inOut(Easing.exp),
    *       useNativeDriver: true,
    *     }),
-   *   ]).start(({ finished }) => {
-   *     if (finished) callback?.()
-   *   })
+   *   ]).start() => callback?.())
    * }
    * @see [API reference](https://colorfy-software.gitbook.io/react-native-modalfy/api/types/modaloptions#animationin).
    */
@@ -314,9 +312,7 @@ export interface ModalOptions {
    *       easing: Easing.inOut(Easing.exp),
    *       useNativeDriver: true,
    *     }),
-   *   ]).start(({ finished }) => {
-   *     if (finished) callback()
-   *   })
+   *   ]).start(() => callback())
    * }
    * @see [API reference](https://colorfy-software.gitbook.io/react-native-modalfy/api/types/modaloptions#animationout).
    */