From a5892723a4ed6dd78d668176c518e7f979fe7d0f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=99=B3=E5=BB=B7=E5=AE=89?=
 <73953029+nrps9909@users.noreply.github.com>
Date: Thu, 27 Aug 2026 05:00:40 +0800
Subject: [PATCH] fix: prevent preview controls from submitting forms

---
 src/Preview/CloseBtn.tsx    |  1 +
 src/Preview/PrevNext.tsx    |  1 +
 tests/previewGroup.test.tsx | 22 ++++++++++++++++++++++
 3 files changed, 24 insertions(+)

diff --git a/src/Preview/CloseBtn.tsx b/src/Preview/CloseBtn.tsx
index 00bacda8..85599a45 100644
--- a/src/Preview/CloseBtn.tsx
+++ b/src/Preview/CloseBtn.tsx
@@ -14,6 +14,7 @@ export default function CloseBtn(props: CloseBtnProps) {
 
   return (
     <button
+      type="button"
       className={clsx(`${prefixCls}-close`, className)}
       style={style}
       onClick={onClick}
diff --git a/src/Preview/PrevNext.tsx b/src/Preview/PrevNext.tsx
index 338f782f..43b6d837 100644
--- a/src/Preview/PrevNext.tsx
+++ b/src/Preview/PrevNext.tsx
@@ -27,6 +27,7 @@ export default function PrevNext(props: PrevNextProps) {
   return (
     <>
       <button
+        type="button"
         className={clsx(switchCls, `${switchCls}-prev`, {
           [`${switchCls}-disabled`]: prevDisabled,
         })}
diff --git a/tests/previewGroup.test.tsx b/tests/previewGroup.test.tsx
index c13b6337..c126d1d3 100644
--- a/tests/previewGroup.test.tsx
+++ b/tests/previewGroup.test.tsx
@@ -151,6 +151,28 @@ describe('PreviewGroup', () => {
     expect(previewProgressElement.textContent).toEqual('current:1 / total:3');
   });
 
+  it('uses non-submitting button types for preview controls', () => {
+    const { container } = render(
+      <Image.PreviewGroup>
+        <Image src="src1" />
+        <Image src="src2" />
+      </Image.PreviewGroup>,
+    );
+
+    fireEvent.click(container.querySelector('.rc-image'));
+    act(() => {
+      jest.runAllTimers();
+    });
+
+    [
+      '.rc-image-preview-close',
+      '.rc-image-preview-switch-prev',
+      '.rc-image-preview-switch-next',
+    ].forEach(selector => {
+      expect(document.querySelector(selector)).toHaveAttribute('type', 'button');
+    });
+  });
+
   it('Switch', () => {
     const previewProgressElementPath = '.rc-image-preview-progress';
     const { container } = render(