From db1da678727838cc58c21ac4963ee265743a6f25 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=B0=8F=E8=B1=AA?= <1844749591@qq.com>
Date: Wed, 26 Mar 2025 15:52:43 +0800
Subject: [PATCH 1/3] refacto: rename visibility change props for clarity

---
 .gitignore      |  3 ++-
 src/Tooltip.tsx | 12 ++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/.gitignore b/.gitignore
index 93e1cbd..e466542 100644
--- a/.gitignore
+++ b/.gitignore
@@ -41,4 +41,5 @@ package-lock.json
 .dumi/tmp
 .dumi/tmp-production
 
-bun.lockb
\ No newline at end of file
+bun.lockb
+.vscode/
\ No newline at end of file
diff --git a/src/Tooltip.tsx b/src/Tooltip.tsx
index 5092c6f..f8f3444 100644
--- a/src/Tooltip.tsx
+++ b/src/Tooltip.tsx
@@ -27,8 +27,8 @@ export interface TooltipProps
   placement?: string;
   /** Config popup motion */
   motion?: TriggerProps['popupMotion'];
-  onVisibleChange?: (visible: boolean) => void;
-  afterVisibleChange?: (visible: boolean) => void;
+  onOpenChange?: (visible: boolean) => void;
+  afterOpenChange?: (visible: boolean) => void;
   overlay: (() => React.ReactNode) | React.ReactNode;
   /** @deprecated Please use `styles={{ root: {} }}` */
   overlayStyle?: React.CSSProperties;
@@ -68,8 +68,8 @@ const Tooltip = (props: TooltipProps, ref: React.Ref<TooltipRef>) => {
     overlayStyle,
     prefixCls = 'rc-tooltip',
     children,
-    onVisibleChange,
-    afterVisibleChange,
+    onOpenChange,
+    afterOpenChange,
     motion,
     placement = 'right',
     align = {},
@@ -131,8 +131,8 @@ const Tooltip = (props: TooltipProps, ref: React.Ref<TooltipRef>) => {
       ref={triggerRef}
       popupAlign={align}
       getPopupContainer={getTooltipContainer}
-      onPopupVisibleChange={onVisibleChange}
-      afterPopupVisibleChange={afterVisibleChange}
+      onOpenChange={onOpenChange}
+      afterOpenChange={afterOpenChange}
       popupMotion={motion}
       defaultPopupVisible={defaultVisible}
       autoDestroy={destroyTooltipOnHide}

From b1e392a0679069175cd9561bda5da314b328d7c3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=B0=8F=E8=B1=AA?= <1844749591@qq.com>
Date: Wed, 26 Mar 2025 16:21:46 +0800
Subject: [PATCH 2/3] refactor: update Tooltip component to use 'open' prop for
 visibility control

---
 src/Tooltip.tsx      | 15 ++++++---------
 tests/index.test.tsx |  4 ++--
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/src/Tooltip.tsx b/src/Tooltip.tsx
index f8f3444..061b29b 100644
--- a/src/Tooltip.tsx
+++ b/src/Tooltip.tsx
@@ -23,12 +23,12 @@ export interface TooltipProps
   > {
   trigger?: ActionType | ActionType[];
   defaultVisible?: boolean;
-  visible?: boolean;
+  open?: boolean;
   placement?: string;
   /** Config popup motion */
   motion?: TriggerProps['popupMotion'];
-  onOpenChange?: (visible: boolean) => void;
-  afterOpenChange?: (visible: boolean) => void;
+  onOpenChange?: (open: boolean) => void;
+  afterOpenChange?: (open: boolean) => void;
   overlay: (() => React.ReactNode) | React.ReactNode;
   /** @deprecated Please use `styles={{ root: {} }}` */
   overlayStyle?: React.CSSProperties;
@@ -83,6 +83,7 @@ const Tooltip = (props: TooltipProps, ref: React.Ref<TooltipRef>) => {
     showArrow = true,
     classNames: tooltipClassNames,
     styles: tooltipStyles,
+    open,
     ...restProps
   } = props;
 
@@ -91,11 +92,6 @@ const Tooltip = (props: TooltipProps, ref: React.Ref<TooltipRef>) => {
 
   useImperativeHandle(ref, () => triggerRef.current);
 
-  const extraProps: Partial<TooltipProps & TriggerProps> = { ...restProps };
-  if ('visible' in props) {
-    extraProps.popupVisible = props.visible;
-  }
-
   const getPopupElement = () => (
     <Popup
       key="content"
@@ -140,7 +136,8 @@ const Tooltip = (props: TooltipProps, ref: React.Ref<TooltipRef>) => {
       popupStyle={{ ...overlayStyle, ...tooltipStyles?.root }}
       mouseEnterDelay={mouseEnterDelay}
       arrow={showArrow}
-      {...extraProps}
+      popupVisible={open}
+      {...restProps}
     >
       {getChildren()}
     </Trigger>
diff --git a/tests/index.test.tsx b/tests/index.test.tsx
index 7436ab8..f52e24a 100644
--- a/tests/index.test.tsx
+++ b/tests/index.test.tsx
@@ -222,7 +222,7 @@ describe('rc-tooltip', () => {
     const App = () => {
       const [open, setOpen] = React.useState(false);
       return (
-        <Tooltip overlay={<strong className="x-content">Tooltip content</strong>} visible={open}>
+        <Tooltip overlay={<strong className="x-content">Tooltip content</strong>} open={open}>
           <div
             className="target"
             onClick={() => {
@@ -263,7 +263,7 @@ describe('rc-tooltip', () => {
     };
 
     const { container } = render(
-      <Tooltip classNames={customClassNames} overlay={<div />} styles={customStyles} visible>
+      <Tooltip classNames={customClassNames} overlay={<div />} styles={customStyles} open>
         <button />
       </Tooltip>,
     );

From bc2de933312584230c4dc7f33454aeb5cc1b0eb1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=B0=8F=E8=B1=AA?= <1844749591@qq.com>
Date: Wed, 26 Mar 2025 16:26:16 +0800
Subject: [PATCH 3/3] fix: ts error

---
 docs/examples/formError.tsx       |  6 +++---
 docs/examples/onVisibleChange.tsx | 10 +++++-----
 docs/examples/simple.tsx          |  4 ++--
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/docs/examples/formError.tsx b/docs/examples/formError.tsx
index cec8845..5b317b0 100644
--- a/docs/examples/formError.tsx
+++ b/docs/examples/formError.tsx
@@ -4,13 +4,13 @@ import React, { Component } from 'react';
 import '../../assets/bootstrap.less';
 
 interface TestState {
-  visible: boolean;
+  open: boolean;
   destroy?: boolean;
 }
 
 class Test extends Component {
   state = {
-    visible: false,
+    open: false,
   } as TestState;
 
   handleDestroy = () => {
@@ -33,7 +33,7 @@ class Test extends Component {
       <div>
         <div style={{ marginTop: 100, marginLeft: 100, marginBottom: 100 }}>
           <Tooltip
-            visible={this.state.visible}
+            open={this.state.open}
             motion={{ motionName: 'rc-tooltip-zoom' }}
             trigger={[]}
             overlayStyle={{ zIndex: 1000 }}
diff --git a/docs/examples/onVisibleChange.tsx b/docs/examples/onVisibleChange.tsx
index 0de15c2..20d23b6 100644
--- a/docs/examples/onVisibleChange.tsx
+++ b/docs/examples/onVisibleChange.tsx
@@ -7,16 +7,16 @@ function preventDefault(e) {
 }
 
 interface TestState {
-  visible: boolean;
+  open: boolean;
   destroy?: boolean;
 }
 
 class Test extends Component {
   state = {
-    visible: false,
+    open: false,
   } as TestState;
 
-  onVisibleChange = visible => {
+  onOpenChange = visible => {
     this.setState({
       visible,
     });
@@ -36,9 +36,9 @@ class Test extends Component {
       <div>
         <div style={{ marginTop: 300, marginLeft: 100, marginBottom: 100 }}>
           <Tooltip
-            visible={this.state.visible}
+            open={this.state.open}
             motion={{ motionName: 'rc-tooltip-zoom' }}
-            onVisibleChange={this.onVisibleChange}
+            onOpenChange={this.onOpenChange}
             trigger="click"
             overlay={<span>I am a tooltip</span>}
           >
diff --git a/docs/examples/simple.tsx b/docs/examples/simple.tsx
index 8cb3012..0dd3cf8 100644
--- a/docs/examples/simple.tsx
+++ b/docs/examples/simple.tsx
@@ -88,7 +88,7 @@ class Test extends Component<any, TestState> {
     });
   };
 
-  onVisibleChange = (visible) => {
+  onOpenChange = (visible) => {
     console.log('tooltip', visible); // eslint-disable-line no-console
   };
 
@@ -211,7 +211,7 @@ class Test extends Component<any, TestState> {
             mouseLeaveDelay={0.1}
             destroyTooltipOnHide={this.state.destroyTooltipOnHide}
             trigger={Object.keys(this.state.trigger) as ActionType[]}
-            onVisibleChange={this.onVisibleChange}
+            onOpenChange={this.onOpenChange}
             overlay={<div style={{ height: 50, width: 50 }}>i am a tooltip</div>}
             align={{
               offset: [this.state.offsetX, this.state.offsetY],