fix: right click should not trigger drag (#307)

Author: zombieJ

src/hooks/useScrollDrag.ts

@@ -40,7 +40,7 @@ export default function useScrollDrag(
 
       const onMouseDown = (e: MouseEvent) => {
         // Skip if element set draggable
-        if ((e.target as HTMLElement).draggable) {
+        if ((e.target as HTMLElement).draggable || e.button !== 0) {
           return;
         }
         // Skip if nest List has handled this event

tests/scroll.test.js

@@ -577,8 +577,10 @@ describe('List.Scroll', () => {
   });
 
   describe('mouse down drag', () => {
-    function dragDown(container, mouseY) {
-      fireEvent.mouseDown(container.querySelector('li'));
+    function dragDown(container, mouseY, button = 0) {
+      fireEvent.mouseDown(container.querySelector('li'), {
+        button,
+      });
 
       let moveEvent = createEvent.mouseMove(container.querySelector('li'));
       moveEvent.pageY = mouseY;
@@ -621,6 +623,26 @@ describe('List.Scroll', () => {
       expect(getScrollTop(container)).toBe(0);
     });
 
+    it('right click should not move', () => {
+      const onScroll = jest.fn();
+      const { container } = render(
+        <List
+          component="ul"
+          itemKey="id"
+          itemHeight={20}
+          height={100}
+          data={genData(100)}
+          onScroll={onScroll}
+        >
+          {({ id }) => <li>{id}</li>}
+        </List>,
+      );
+
+      // Drag down
+      dragDown(container, 100, 2);
+      expect(getScrollTop(container)).toBe(0);
+    });
+
     it('can not move when item add draggable', () => {
       const onScroll = jest.fn();
       const { container } = render(