From 6c856a1d7554dc48285b6d792c0f4ae14768ae98 Mon Sep 17 00:00:00 2001
From: Simon <wsj_simon@163.com>
Date: Tue, 12 Jan 2021 16:45:45 +0800
Subject: [PATCH] fix: only allow main button when drag image

---
 src/Preview.tsx        |  2 ++
 tests/preview.test.tsx | 14 ++++++++++++++
 2 files changed, 16 insertions(+)

diff --git a/src/Preview.tsx b/src/Preview.tsx
index 069034e5..07e658c3 100644
--- a/src/Preview.tsx
+++ b/src/Preview.tsx
@@ -171,6 +171,8 @@ const Preview: React.FC<PreviewProps> = props => {
   };
 
   const onMouseDown: React.MouseEventHandler<HTMLDivElement> = event => {
+    // Only allow main button
+    if (event.button !== 0) return;
     event.preventDefault();
     // Without this mask close will abnormal
     event.stopPropagation();
diff --git a/tests/preview.test.tsx b/tests/preview.test.tsx
index da3c7113..4cfac160 100644
--- a/tests/preview.test.tsx
+++ b/tests/preview.test.tsx
@@ -214,6 +214,14 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 2,
+    });
+    expect(wrapper.find('.rc-image-preview-moving').get(0)).toBeUndefined();
+
+    wrapper.find('.rc-image-preview-img').simulate('mousedown', {
+      pageX: 0,
+      pageY: 0,
+      button: 0,
     });
 
     expect(wrapper.find('.rc-image-preview-moving').get(0)).toBeTruthy();
@@ -250,6 +258,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {
@@ -267,6 +276,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {
@@ -286,6 +296,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {
@@ -305,6 +316,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {
@@ -324,6 +336,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {
@@ -343,6 +356,7 @@ describe('Preview', () => {
     wrapper.find('.rc-image-preview-img').simulate('mousedown', {
       pageX: 0,
       pageY: 0,
+      button: 0,
     });
 
     act(() => {