Fix(Dialog): fix render multiple binding events (#2749)

* fix(Dialog): fix render multiple binding events * fix(dialog): fix render multiple binding events * fix(dialog): fix ssr render
Tencent · Feb 21, 2024 · a5680f4 · a5680f4
1 parent 1718fbe
commit a5680f4
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/src/dialog/hooks/useDialogPosition.ts b/src/dialog/hooks/useDialogPosition.ts
@@ -1,6 +1,7 @@
-import { useEffect, useRef } from 'react';
+import { MutableRefObject, useEffect, useRef } from 'react';
+import useIsomorphicLayoutEffect from '../../_util/useLayoutEffect';
 
-export default function useDialogPosition(visible, dialogCardRef) {
+export default function useDialogPosition(visible: boolean, dialogCardRef: MutableRefObject<HTMLElement>) {
   const mousePosRef = useRef(null);
 
   const getClickPosition = (e: MouseEvent) => {
@@ -13,13 +14,8 @@ export default function useDialogPosition(visible, dialogCardRef) {
     }, 100);
   };
 
-  if (typeof document !== 'undefined') {
+  useIsomorphicLayoutEffect(() => {
     document.addEventListener('click', getClickPosition, true);
-  }
-
-  useEffect(() => {
-    document.addEventListener('click', getClickPosition, true);
-
     return () => {
       document.removeEventListener('click', getClickPosition, true);
     };