fix(popover): fix popover updateVisible method (#1537)

updateVisible method error
growingio · Nov 26, 2021 · b88a80b · b88a80b · vercel · Nov 26, 2021
1 parent efa3788
commit b88a80b
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/popover/Popover.tsx b/src/popover/Popover.tsx
@@ -77,10 +77,11 @@ const Popover = (props: PopoverProps) => {
   const updateVisible = useCallback(
     (resetVisible: boolean) => {
       let realVisible = isUndefined(enterVisible) ? resetVisible : enterVisible;
-      realVisible = disabled === true ? false : realVisible;
+      realVisible = disabled ? false : realVisible;
+
       if (!(overContentRef.current && enterable)) {
         setVisible(realVisible);
-        onVisibleChange?.(resetVisible);
+        onVisibleChange?.(realVisible);
       }
       if (realVisible) {
         update?.();
@@ -103,7 +104,7 @@ const Popover = (props: PopoverProps) => {
     if (!isUndefined(enterVisible)) {
       setVisible(enterVisible);
     }
-    if (disabled === true) {
+    if (disabled) {
       setVisible(false);
     }
   }, [enterVisible, disabled]);
@@ -234,7 +235,11 @@ const Popover = (props: PopoverProps) => {
 
   const renderContent = (
     <>
-      <ResizeObserver onResize={() => { updateVisible(true) }}>
+      <ResizeObserver
+        onResize={() => {
+          updateVisible(true);
+        }}
+      >
         {typeof getContainer === 'function'
           ? ReactDOM.createPortal(contentRender, getContainer(referenceElement as HTMLDivElement))
           : contentRender}