fix: handle pr

Tencent · Nov 7, 2022 · 6282def · 6282def
1 parent 69fe878
commit 6282def
Showing 1 changed file with 14 additions and 12 deletions.
diff --git a/src/guide/Guide.tsx b/src/guide/Guide.tsx
@@ -6,7 +6,7 @@ import useConfig from '../hooks/useConfig';
 import Popup from '../popup';
 import { GuideCrossProps, StepPopupPlacement, TdGuideProps } from './type';
 import { addClass, removeClass, isFixed, getWindowScroll } from '../_util/dom';
-import { scrollToParentVisibleArea, getRelativePosition, getTargetElm, scrollToElm, useWatch } from './utils';
+import { scrollToParentVisibleArea, getRelativePosition, getTargetElm, scrollToElm } from './utils';
 import setStyle from '../_common/js/utils/set-style';
 import useControlled from '../hooks/useControlled';
 import { guideDefaultProps } from './defaultProps';
@@ -173,24 +173,26 @@ const Guide = (props: GuideProps) => {
     }
   };
 
-  useWatch(innerCurrent, (newValue) => {
-    if (newValue >= 0 && newValue < steps.length) {
+  useEffect(() => {
+    if (innerCurrent >= 0 && innerCurrent < steps.length) {
       initGuide();
     } else {
       setActive(false);
       destroyGuide();
     }
-  });
-
-  const useMount = (callback) => {
-    useEffect(() => {
-      callback();
-    }, [callback]);
-  };
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [innerCurrent]);
 
-  useMount(() => {
+  useEffect(() => {
     initGuide();
-  });
+    console.log('init');
+
+    return () => {
+      destroyGuide();
+      console.log('destroy');
+    };
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
 
   const renderOverlayLayer = () =>
     createPortal(