feat: support disabled prop

Author: zombieJ

README.md

@@ -75,6 +75,7 @@ Then open `http://localhost:8000`.
 | blurDelay                  | Delay before hiding on blur, in seconds.                                  | `number`                                  | -                  |
 | builtinPlacements          | Named placement presets.                                                  | `BuildInPlacements`                       | `{}`               |
 | defaultPopupVisible        | Initial uncontrolled visibility.                                          | `boolean`                                 | `false`            |
+| disabled                   | Temporarily suppress popup visibility without resetting the current open state. | `boolean`                                 | `false`            |
 | focusDelay                 | Delay before showing on focus, in seconds.                                | `number`                                  | -                  |
 | forceRender                | Render popup before it is first shown.                                    | `boolean`                                 | `false`            |
 | fresh                      | Keep popup content updated while closed.                                  | `boolean`                                 | -                  |

README.zh-CN.md

@@ -75,6 +75,7 @@ npm start
 | blurDelay                  | `blur` 后隐藏前的延迟，单位为秒。                                  | `number`                                  | -                  |
 | builtinPlacements          | 命名位置预设。                                                  | `BuildInPlacements`                       | `{}`               |
 | defaultPopupVisible        | 非受控初始显示状态。                                          | `boolean`                                 | `false`            |
+| disabled                   | 临时隐藏弹层，但不主动重置当前打开状态。                       | `boolean`                                 | `false`            |
 | focusDelay                 | `focus` 后显示前的延迟，单位为秒。                                | `number`                                  | -                  |
 | forceRender                | 首次显示前渲染弹层。                                    | `boolean`                                 | `false`            |
 | fresh                      | 关闭时仍保持弹层内容更新。                                  | `boolean`                                 | -                  |

src/index.tsx

@@ -65,6 +65,8 @@ export interface TriggerProps {
   action?: ActionType | ActionType[];
   showAction?: ActionType[];
   hideAction?: ActionType[];
+  /** Temporarily suppress popup visibility without resetting the current open state. */
+  disabled?: boolean;
 
   prefixCls?: string;
 
@@ -159,6 +161,7 @@ export function generateTrigger(
       action = 'hover',
       showAction,
       hideAction,
+      disabled = false,
 
       // Open
       popupVisible,
@@ -319,7 +322,7 @@ export function generateTrigger(
       popupVisible,
     );
 
-    const mergedOpen = internalOpen || false;
+    const mergedOpen = (internalOpen || false) && !disabled;
 
     // ========================== Children ==========================
     const child = React.useMemo(() => {
@@ -374,10 +377,10 @@ export function generateTrigger(
         if (mergedOpen) {
           uniqueContext.show(getUniqueOptions(mouseEnterDelay), isOpen);
         } else {
-          uniqueContext.hide(mouseLeaveDelay);
+          uniqueContext.hide(disabled ? 0 : mouseLeaveDelay);
         }
       }
-    }, [mergedOpen, targetEle]);
+    }, [mergedOpen, targetEle, disabled]);
 
     const openRef = React.useRef(mergedOpen);
     openRef.current = mergedOpen;

tests/basic.test.jsx

@@ -179,6 +179,36 @@ describe('Trigger.Basic', () => {
         trigger(document, '.rc-trigger-popup', 'pointerEnter');
         expect(isPopupHidden()).toBeFalsy();
       });
+
+      it('temporarily hides while disabled and restores without mouse leave', () => {
+        const onOpenChange = jest.fn();
+        const Demo = ({ disabled = false }) => (
+          <Trigger
+            action={['hover']}
+            disabled={disabled}
+            onOpenChange={onOpenChange}
+            popup={<strong>trigger</strong>}
+          >
+            <div className="target">hover</div>
+          </Trigger>
+        );
+
+        const { container, rerender } = render(<Demo />);
+
+        trigger(container, '.target', 'mouseEnter');
+        expect(isPopupHidden()).toBeFalsy();
+        expect(onOpenChange).toHaveBeenCalledWith(true);
+        onOpenChange.mockReset();
+
+        rerender(<Demo disabled />);
+        expect(isPopupHidden()).toBeTruthy();
+        expect(onOpenChange).not.toHaveBeenCalled();
+
+        rerender(<Demo />);
+        expect(isPopupHidden()).toBeFalsy();
+        expect(onOpenChange).not.toHaveBeenCalled();
+      });
+
     });
 
     it('contextMenu works', () => {
@@ -1009,6 +1039,34 @@ describe('Trigger.Basic', () => {
     expect(document.querySelector('.rc-trigger-popup')).toBeTruthy();
   });
 
+  it('temporarily hides a controlled popup without changing its open state', () => {
+    const onOpenChange = jest.fn();
+    const Demo = ({ disabled = false }) => (
+      <Trigger
+        disabled={disabled}
+        onOpenChange={onOpenChange}
+        popup={<strong>trigger</strong>}
+        popupVisible
+      >
+        {({ open }) => <div data-open={open} />}
+      </Trigger>
+    );
+
+    const { container, rerender } = render(<Demo />);
+
+    expect(container.firstChild).toHaveAttribute('data-open', 'true');
+    expect(isPopupHidden()).toBeFalsy();
+
+    rerender(<Demo disabled />);
+    expect(container.firstChild).toHaveAttribute('data-open', 'false');
+    expect(isPopupHidden()).toBeTruthy();
+
+    rerender(<Demo />);
+    expect(container.firstChild).toHaveAttribute('data-open', 'true');
+    expect(isPopupHidden()).toBeFalsy();
+    expect(onOpenChange).not.toHaveBeenCalled();
+  });
+
   describe('click window to hide', () => {
     it('should hide', async () => {
       const onOpenChange = jest.fn();

tests/unique.test.tsx

@@ -69,6 +69,38 @@ describe('Trigger.Unique', () => {
     jest.restoreAllMocks();
   });
 
+  it('ignores leave delay when controlled popup is disabled', async () => {
+    const Demo = ({ disabled = false }) => (
+      <UniqueProvider>
+        <Trigger
+          disabled={disabled}
+          mouseLeaveDelay={10}
+          popup={<strong>trigger</strong>}
+          popupVisible
+          unique
+        >
+          <div className="target" />
+        </Trigger>
+      </UniqueProvider>
+    );
+
+    const { rerender } = render(<Demo />);
+    await awaitFakeTimer();
+    expect(document.querySelector('.rc-trigger-popup')).not.toHaveClass(
+      'rc-trigger-popup-hidden',
+    );
+
+    rerender(<Demo disabled />);
+    expect(document.querySelector('.rc-trigger-popup')).toHaveClass(
+      'rc-trigger-popup-hidden',
+    );
+
+    rerender(<Demo />);
+    expect(document.querySelector('.rc-trigger-popup')).not.toHaveClass(
+      'rc-trigger-popup-hidden',
+    );
+  });
+
   it('moving will not hide the popup', async () => {
     const { container } = render(
       <UniqueProvider>
@@ -376,7 +408,7 @@ describe('Trigger.Unique', () => {
   });
 
   it('esc should close unique popup', async () => {
-    const { container,baseElement } = render(
+    const { container, baseElement } = render(
       <UniqueProvider>
         <Trigger action={['click']} popup={<div>Popup</div>} unique>
           <div className="target" />