From 323c48f32399db3999fd163f522ab950b7919d17 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:13:03 +0800
Subject: [PATCH 1/5] refactor: rename onPopupVisibleChange to onOpenChange
 across components

---
 docs/examples/controlled.tsx |  4 ++--
 src/BaseSelect/index.tsx     | 10 +++++-----
 src/SelectTrigger.tsx        | 10 +++++-----
 tests/Combobox.test.tsx      |  6 +++---
 tests/Custom.test.tsx        |  9 +++------
 tests/Popup.test.tsx         |  6 +++---
 tests/Select.test.tsx        |  4 ++--
 7 files changed, 23 insertions(+), 26 deletions(-)

diff --git a/docs/examples/controlled.tsx b/docs/examples/controlled.tsx
index 34bd7a754..7f8c5b1e3 100644
--- a/docs/examples/controlled.tsx
+++ b/docs/examples/controlled.tsx
@@ -43,7 +43,7 @@ class Controlled extends React.Component<{}, ControlledState> {
     console.log('onFocus');
   };
 
-  onPopupVisibleChange = (open) => {
+  onOpenChange = (open) => {
     this.setState({ open });
   };
 
@@ -68,7 +68,7 @@ class Controlled extends React.Component<{}, ControlledState> {
             optionLabelProp="children"
             optionFilterProp="text"
             onChange={this.onChange}
-            onPopupVisibleChange={this.onPopupVisibleChange}
+            onOpenChange={this.onOpenChange}
           >
             <Option value="01" text="jack" title="jack">
               <b
diff --git a/src/BaseSelect/index.tsx b/src/BaseSelect/index.tsx
index d19ee824a..331d883af 100644
--- a/src/BaseSelect/index.tsx
+++ b/src/BaseSelect/index.tsx
@@ -162,7 +162,7 @@ export interface BaseSelectProps extends BaseSelectPrivateProps, React.AriaAttri
   // >>> Open
   open?: boolean;
   defaultOpen?: boolean;
-  onPopupVisibleChange?: (open: boolean) => void;
+  onOpenChange?: (open: boolean) => void;
 
   // >>> Customize Input
   /** @private Internal usage. Do not use in your production. */
@@ -257,7 +257,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
     // Open
     open,
     defaultOpen,
-    onPopupVisibleChange,
+    onOpenChange,
 
     // Active
     activeValue,
@@ -401,11 +401,11 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
         setInnerOpen(nextOpen);
 
         if (mergedOpen !== nextOpen) {
-          onPopupVisibleChange?.(nextOpen);
+          onOpenChange?.(nextOpen);
         }
       }
     },
-    [disabled, mergedOpen, setInnerOpen, onPopupVisibleChange],
+    [disabled, mergedOpen, setInnerOpen, onOpenChange],
   );
 
   // ============================= Search =============================
@@ -814,7 +814,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
         // But we should update `rc-resize-observer` first.
         selectorDomRef.current || node
       }
-      onPopupVisibleChange={onTriggerVisibleChange}
+      onOpenChange={onTriggerVisibleChange}
       onPopupMouseEnter={onPopupMouseEnter}
     >
       {customizeRawInputElement ? (
diff --git a/src/SelectTrigger.tsx b/src/SelectTrigger.tsx
index 140471d39..cedd2a631 100644
--- a/src/SelectTrigger.tsx
+++ b/src/SelectTrigger.tsx
@@ -74,7 +74,7 @@ export interface SelectTriggerProps {
   empty: boolean;
 
   getTriggerDOMNode: (node: HTMLElement) => HTMLElement;
-  onPopupVisibleChange?: (visible: boolean) => void;
+  onOpenChange?: (visible: boolean) => void;
 
   onPopupMouseEnter: () => void;
 }
@@ -102,7 +102,7 @@ const SelectTrigger: React.ForwardRefRenderFunction<RefTriggerProps, SelectTrigg
     getPopupContainer,
     empty,
     getTriggerDOMNode,
-    onPopupVisibleChange,
+    onOpenChange,
     onPopupMouseEnter,
     ...restProps
   } = props;
@@ -158,8 +158,8 @@ const SelectTrigger: React.ForwardRefRenderFunction<RefTriggerProps, SelectTrigg
   return (
     <Trigger
       {...restProps}
-      showAction={onPopupVisibleChange ? ['click'] : []}
-      hideAction={onPopupVisibleChange ? ['click'] : []}
+      showAction={onOpenChange ? ['click'] : []}
+      hideAction={onOpenChange ? ['click'] : []}
       popupPlacement={placement || (direction === 'rtl' ? 'bottomRight' : 'bottomLeft')}
       builtinPlacements={mergedBuiltinPlacements}
       prefixCls={popupPrefixCls}
@@ -175,7 +175,7 @@ const SelectTrigger: React.ForwardRefRenderFunction<RefTriggerProps, SelectTrigg
       })}
       popupStyle={mergedPopupStyle}
       getTriggerDOMNode={getTriggerDOMNode}
-      onPopupVisibleChange={onPopupVisibleChange}
+      onOpenChange={onOpenChange}
     >
       {children}
     </Trigger>
diff --git a/tests/Combobox.test.tsx b/tests/Combobox.test.tsx
index 17b0fc102..8b1eda308 100644
--- a/tests/Combobox.test.tsx
+++ b/tests/Combobox.test.tsx
@@ -455,16 +455,16 @@ describe('Select.Combobox', () => {
   // https://github.com/ant-design/ant-design/issues/16572
   it('close when enter press without active option', () => {
     jest.useFakeTimers();
-    const onPopupVisibleChange = jest.fn();
+    const onOpenChange = jest.fn();
     const { container } = render(
-      <Select mode="combobox" open onPopupVisibleChange={onPopupVisibleChange}>
+      <Select mode="combobox" open onOpenChange={onOpenChange}>
         <Option value="One">One</Option>
         <Option value="Two">Two</Option>
       </Select>,
     );
     keyDown(container.querySelector('input')!, KeyCode.ENTER);
     jest.runAllTimers();
-    expect(onPopupVisibleChange).toHaveBeenCalledWith(false);
+    expect(onOpenChange).toHaveBeenCalledWith(false);
     jest.useRealTimers();
   });
 
diff --git a/tests/Custom.test.tsx b/tests/Custom.test.tsx
index eb781adaf..c33c68c67 100644
--- a/tests/Custom.test.tsx
+++ b/tests/Custom.test.tsx
@@ -15,15 +15,12 @@ describe('Select.Custom', () => {
   });
 
   it('getRawInputElement', () => {
-    const onPopupVisibleChange = jest.fn();
+    const onOpenChange = jest.fn();
     const { container } = render(
-      <Select
-        getRawInputElement={() => <span className="custom" />}
-        onPopupVisibleChange={onPopupVisibleChange}
-      />,
+      <Select getRawInputElement={() => <span className="custom" />} onOpenChange={onOpenChange} />,
     );
     fireEvent.click(container.querySelector('.custom'));
 
-    expect(onPopupVisibleChange).toHaveBeenCalledWith(true);
+    expect(onOpenChange).toHaveBeenCalledWith(true);
   });
 });
diff --git a/tests/Popup.test.tsx b/tests/Popup.test.tsx
index 8b2a371fb..c1a4075f4 100644
--- a/tests/Popup.test.tsx
+++ b/tests/Popup.test.tsx
@@ -13,17 +13,17 @@ describe('Select.Popup', () => {
   injectRunAllTimers(jest);
 
   it('click popup should not trigger close', () => {
-    const onPopupVisibleChange = jest.fn();
+    const onOpenChange = jest.fn();
     render(
       <Select
         open
         options={[{ value: 'bamboo' }]}
-        onPopupVisibleChange={onPopupVisibleChange}
+        onOpenChange={onOpenChange}
         getPopupContainer={() => document.body}
       />,
     );
 
     fireEvent.mouseDown(document.querySelector('.rc-select-dropdown'));
-    expect(onPopupVisibleChange).not.toHaveBeenCalled();
+    expect(onOpenChange).not.toHaveBeenCalled();
   });
 });
diff --git a/tests/Select.test.tsx b/tests/Select.test.tsx
index b79b9a131..911356ad0 100644
--- a/tests/Select.test.tsx
+++ b/tests/Select.test.tsx
@@ -894,13 +894,13 @@ describe('Select.Basic', () => {
         open: true,
       };
 
-      public onPopupVisibleChange = (open) => {
+      public onOpenChange = (open) => {
         this.setState({ open });
       };
 
       public render() {
         return (
-          <Select open={this.state.open} onPopupVisibleChange={this.onPopupVisibleChange}>
+          <Select open={this.state.open} onOpenChange={this.onOpenChange}>
             <Option value="1">1</Option>
           </Select>
         );

From f89406f93b277c40ed9957766a2d6b63f4029a22 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:33:26 +0800
Subject: [PATCH 2/5] refactor: rename visible to open in SelectTrigger and
 BaseSelect components

---
 src/BaseSelect/index.tsx | 2 +-
 src/SelectTrigger.tsx    | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/BaseSelect/index.tsx b/src/BaseSelect/index.tsx
index 331d883af..658ef3eac 100644
--- a/src/BaseSelect/index.tsx
+++ b/src/BaseSelect/index.tsx
@@ -794,7 +794,7 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
       ref={triggerRef}
       disabled={disabled}
       prefixCls={prefixCls}
-      visible={triggerOpen}
+      open={triggerOpen}
       popupElement={optionList}
       animation={animation}
       transitionName={transitionName}
diff --git a/src/SelectTrigger.tsx b/src/SelectTrigger.tsx
index cedd2a631..a93f8975f 100644
--- a/src/SelectTrigger.tsx
+++ b/src/SelectTrigger.tsx
@@ -57,7 +57,7 @@ export interface SelectTriggerProps {
   prefixCls: string;
   children: React.ReactElement;
   disabled: boolean;
-  visible: boolean;
+  open: boolean;
   popupElement: React.ReactElement;
 
   animation?: string;
@@ -74,7 +74,7 @@ export interface SelectTriggerProps {
   empty: boolean;
 
   getTriggerDOMNode: (node: HTMLElement) => HTMLElement;
-  onOpenChange?: (visible: boolean) => void;
+  onOpenChange?: (open: boolean) => void;
 
   onPopupMouseEnter: () => void;
 }
@@ -86,7 +86,7 @@ const SelectTrigger: React.ForwardRefRenderFunction<RefTriggerProps, SelectTrigg
   const {
     prefixCls,
     disabled,
-    visible,
+    open,
     children,
     popupElement,
     animation,
@@ -168,7 +168,7 @@ const SelectTrigger: React.ForwardRefRenderFunction<RefTriggerProps, SelectTrigg
       ref={triggerPopupRef}
       stretch={stretch}
       popupAlign={popupAlign}
-      popupVisible={visible}
+      popupVisible={open}
       getPopupContainer={getPopupContainer}
       popupClassName={classNames(popupClassName, {
         [`${popupPrefixCls}-empty`]: empty,

From 3827425a126c6abfd08af817caf4f7b548172bf5 Mon Sep 17 00:00:00 2001
From: aojunhao123 <1844749591@qq.com>
Date: Sat, 5 Apr 2025 15:48:02 +0800
Subject: [PATCH 3/5] chore: bump version

---
 .gitignore   | 3 ++-
 package.json | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/.gitignore b/.gitignore
index 90ae753cd..761c95236 100644
--- a/.gitignore
+++ b/.gitignore
@@ -40,4 +40,5 @@ tsconfig.test.json
 .dumi/tmp
 .dumi/tmp-test
 .dumi/tmp-production
-.history
\ No newline at end of file
+.history
+pnpm-lock.yaml
\ No newline at end of file
diff --git a/package.json b/package.json
index 47b30e528..349002261 100644
--- a/package.json
+++ b/package.json
@@ -49,8 +49,8 @@
     "react-dom": "*"
   },
   "dependencies": {
-    "@rc-component/trigger": "^3.0.0",
     "@rc-component/motion": "^1.1.4",
+    "@rc-component/trigger": "^3.0.0",
     "@rc-component/util": "^1.2.1",
     "classnames": "2.x",
     "rc-overflow": "^1.4.0",
@@ -62,6 +62,7 @@
     "@testing-library/jest-dom": "^6.4.5",
     "@testing-library/react": "^15.0.6",
     "@types/jest": "^29.5.12",
+    "@types/node": "^22.14.0",
     "@types/react": "^18.3.0",
     "@types/react-dom": "^18.3.0",
     "babel-jest": "^29.6.1",

From fe9f7d781f8dcb323354b379284306835abac0df Mon Sep 17 00:00:00 2001
From: aojunhao123 <1844749591@qq.com>
Date: Mon, 7 Apr 2025 15:47:59 +0800
Subject: [PATCH 4/5] chore: trigger CI build


From 79d23af9c28f09d99db56180d58b3c790848c50b Mon Sep 17 00:00:00 2001
From: aojunhao123 <1844749591@qq.com>
Date: Mon, 7 Apr 2025 15:49:21 +0800
Subject: [PATCH 5/5] chore: trigger CI build