fix(focus): add ignoreElement support and update deps (#557) (#572)

- Update @rc-component/util to 1.9.0
- Add ignoreElement to handle focus elements from Portal
- Add onClose and onFocus type to DrawerPanelEvents
- Add test case for focus handling with Portal

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: zombieJ

package.json

@@ -45,7 +45,7 @@
   "dependencies": {
     "@rc-component/motion": "^1.1.4",
     "@rc-component/portal": "^2.1.3",
-    "@rc-component/util": "^1.7.0",
+    "@rc-component/util": "^1.9.0",
     "clsx": "^2.1.1"
   },
   "devDependencies": {

src/DrawerPanel.tsx

@@ -15,6 +15,7 @@ export interface DrawerPanelEvents {
   onClick?: React.MouseEventHandler<HTMLDivElement>;
   onKeyDown?: React.KeyboardEventHandler<HTMLDivElement>;
   onKeyUp?: React.KeyboardEventHandler<HTMLDivElement>;
+  onFocus?: React.FocusEventHandler<HTMLDivElement>;
 }
 
 export type DrawerPanelAccessibility = Pick<
@@ -23,8 +24,7 @@ export type DrawerPanelAccessibility = Pick<
 >;
 
 export interface DrawerPanelProps
-  extends DrawerPanelEvents,
-    DrawerPanelAccessibility {
+  extends DrawerPanelEvents, DrawerPanelAccessibility {
   prefixCls: string;
   className?: string;
   id?: string;

src/DrawerPopup.tsx

@@ -151,7 +151,13 @@ const DrawerPopup: React.ForwardRefRenderFunction<
   React.useImperativeHandle(ref, () => panelRef.current);
 
   // ========================= Focusable ==========================
-  useFocusable(() => panelRef.current, open, autoFocus, focusTrap, mask);
+  const ignoreElement = useFocusable(
+    () => panelRef.current,
+    open,
+    autoFocus,
+    focusTrap,
+    mask,
+  );
 
   // ============================ Push ============================
   const [pushed, setPushed] = React.useState(false);
@@ -305,6 +311,9 @@ const DrawerPopup: React.ForwardRefRenderFunction<
     onClick,
     onKeyDown,
     onKeyUp,
+    onFocus: (e: React.FocusEvent<HTMLDivElement>) => {
+      ignoreElement(e.target);
+    },
   };
 
   // =========================== Render ==========================

src/hooks/useFocusable.ts

@@ -11,12 +11,14 @@ export default function useFocusable(
   const mergedFocusTrap = focusTrap ?? mask !== false;
 
   // Focus lock
-  useLockFocus(open && mergedFocusTrap, getContainer);
+  const [ignoreElement] = useLockFocus(open && mergedFocusTrap, getContainer);
 
   // Auto Focus
   React.useEffect(() => {
     if (open && autoFocus === true) {
       getContainer()?.focus({ preventScroll: true });
     }
   }, [open]);
+
+  return ignoreElement;
 }

tests/focus.spec.tsx

@@ -0,0 +1,61 @@
+import { cleanup, render, act } from '@testing-library/react';
+import React from 'react';
+import ReactDOM from 'react-dom';
+import Drawer from '../src';
+
+// Mock useLockFocus to track calls
+jest.mock('@rc-component/util/lib/Dom/focus', () => {
+  const actual = jest.requireActual('@rc-component/util/lib/Dom/focus');
+
+  const useLockFocus = (visible: boolean, ...rest: any[]) => {
+    (globalThis as any).__useLockFocusVisible = visible;
+    const hooks = actual.useLockFocus(visible, ...rest);
+    const hooksArray = Array.isArray(hooks) ? hooks : [hooks];
+    const proxyIgnoreElement = (ele: HTMLElement) => {
+      (globalThis as any).__ignoredElement = ele;
+      hooksArray[0](ele);
+    };
+    return [proxyIgnoreElement, ...hooksArray.slice(1)] as ReturnType<
+      typeof actual.useLockFocus
+    >;
+  };
+
+  return {
+    ...actual,
+    useLockFocus,
+  };
+});
+
+describe('Drawer.Focus', () => {
+  beforeEach(() => {
+    jest.useFakeTimers();
+  });
+
+  afterEach(() => {
+    jest.useRealTimers();
+    cleanup();
+  });
+
+  it('should call ignoreElement when input in portal is focused', () => {
+    render(
+      <Drawer open>
+        <input type="text" id="drawer-input" />
+        {ReactDOM.createPortal(
+          <input type="text" id="portal-input" />,
+          document.body,
+        )}
+      </Drawer>,
+    );
+
+    act(() => {
+      jest.runAllTimers();
+    });
+
+    const input = document.getElementById('portal-input') as HTMLElement;
+    act(() => {
+      input.focus();
+    });
+
+    expect((globalThis as any).__ignoredElement).toBe(input);
+  });
+});