fix: reset when prepare not exist (#44)

* chore: skip prepare if no need

* chore: fix logic

* chore: comment it

Author: zombieJ

src/hooks/useStatus.ts

@@ -101,12 +101,8 @@ export default function useStatus(
   const [patchMotionEvents] = useDomMotionEvents(onInternalMotionEnd);
 
   // ============================= Step =============================
-  const eventHandlers = React.useMemo<{
-    [STEP_PREPARE]?: MotionPrepareEventHandler;
-    [STEP_START]?: MotionEventHandler;
-    [STEP_ACTIVE]?: MotionEventHandler;
-  }>(() => {
-    switch (status) {
+  const getEventHandlers = (targetStatus: MotionStatus) => {
+    switch (targetStatus) {
       case STATUS_APPEAR:
         return {
           [STEP_PREPARE]: onAppearPrepare,
@@ -131,7 +127,13 @@ export default function useStatus(
       default:
         return {};
     }
-  }, [status]);
+  };
+
+  const eventHandlers = React.useMemo<{
+    [STEP_PREPARE]?: MotionPrepareEventHandler;
+    [STEP_START]?: MotionEventHandler;
+    [STEP_ACTIVE]?: MotionEventHandler;
+  }>(() => getEventHandlers(status), [status]);
 
   const [startStep, step] = useStepQueue(status, !supportMotion, newStep => {
     // Only prepare step can be skip
@@ -205,10 +207,15 @@ export default function useStatus(
       nextStatus = STATUS_LEAVE;
     }
 
+    const nextEventHandlers = getEventHandlers(nextStatus);
+
     // Update to next status
-    if (nextStatus) {
+    if (nextStatus && (supportMotion || nextEventHandlers[STEP_PREPARE])) {
       setStatus(nextStatus);
       startStep();
+    } else {
+      // Set back in case no motion but prev status has prepare step
+      setStatus(STATUS_NONE);
     }
   }, [visible]);
 

tests/CSSMotion.spec.tsx

@@ -547,6 +547,7 @@ describe('CSSMotion', () => {
 
     // hide immediately since motion is disabled
     rerender(<Demo motion={false} visible={false} />);
+
     expect(container.querySelector('.hidden')).toBeTruthy();
   });