fix: refine esc propagation and split key/keyCode tests

Author: yoyo837

src/Preview/index.tsx

@@ -328,20 +328,30 @@ const Preview: React.FC<PreviewProps> = props => {
   };
 
   const escClosingRef = useRef(false);
+  const openRef = useRef(open);
+  openRef.current = open;
 
   // >>>>> Effect: Keyboard
   const onKeyDown = useEvent((event: KeyboardEvent) => {
-    if (open) {
+    if (openRef.current) {
       const { keyCode, key } = event;
 
       if (keyCode === KeyCode.ESC || key === 'Escape') {
+        if (escClosingRef.current) {
+          return;
+        }
+
         escClosingRef.current = true;
+        openRef.current = false;
         event.preventDefault();
+        if (keyCode === KeyCode.ESC) {
+          event.stopPropagation();
+        }
         onClose?.();
 
-        Promise.resolve().then(() => {
+        setTimeout(() => {
           escClosingRef.current = false;
-        });
+        }, 0);
         return;
       }
 

tests/preview.portal.test.tsx

@@ -57,14 +57,30 @@ describe('Preview portal esc fallback', () => {
     expect(onClose).toHaveBeenCalledTimes(1);
   });
 
-  it('avoids duplicate close when keydown esc already handled', () => {
+  it('avoids duplicate close when keydown esc already handled (key only)', () => {
     const onClose = jest.fn();
 
     render(
       <Preview prefixCls="rc-image-preview" open src="x" mousePosition={null} onClose={onClose} />,
     );
 
-    fireEvent.keyDown(window, { key: 'Escape', keyCode: 27 });
+    fireEvent.keyDown(window, { key: 'Escape' });
+
+    act(() => {
+      (global as any).__portalProps.onEsc({ top: true });
+    });
+
+    expect(onClose).toHaveBeenCalledTimes(1);
+  });
+
+  it('avoids duplicate close when keydown esc already handled (keyCode only)', () => {
+    const onClose = jest.fn();
+
+    render(
+      <Preview prefixCls="rc-image-preview" open src="x" mousePosition={null} onClose={onClose} />,
+    );
+
+    fireEvent.keyDown(window, { keyCode: 27 });
 
     act(() => {
       (global as any).__portalProps.onEsc({ top: true });