Only warn about mouse movement when the mouse is down. (#13101)

packages/cli/src/ui/contexts/MouseContext.tsx

@@ -100,7 +100,8 @@ export function MouseProvider({
         !handled &&
         event.name === 'move' &&
         event.col >= 0 &&
-        event.row >= 0
+        event.row >= 0 &&
+        event.button === 'left'
       ) {
         // Terminal apps only receive mouse move events when the mouse is down
         // so this always indicates a mouse drag that the user was expecting

packages/cli/src/ui/contexts/ScrollProvider.drag.test.tsx

@@ -113,6 +113,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -125,6 +126,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -142,6 +144,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -159,6 +162,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -172,6 +176,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
     }
     expect(scrollBy).not.toHaveBeenCalled();
@@ -209,6 +214,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -231,6 +237,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -268,6 +275,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -305,6 +313,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -342,6 +351,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -407,6 +417,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -424,6 +435,7 @@ describe('ScrollProvider Drag', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
     // Expect scrollTo to be called with target and duration 0

packages/cli/src/ui/contexts/ScrollProvider.test.tsx

@@ -110,6 +110,7 @@ describe('ScrollProvider', () => {
             shift: false,
             ctrl: false,
             meta: false,
+            button: 'none',
           }) === true
         ) {
           handled = true;
@@ -147,6 +148,7 @@ describe('ScrollProvider', () => {
             shift: false,
             ctrl: false,
             meta: false,
+            button: 'none',
           }) === true
         ) {
           handled = true;
@@ -193,6 +195,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -226,6 +229,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -258,6 +262,7 @@ describe('ScrollProvider', () => {
       shift: false,
       ctrl: false,
       meta: false,
+      button: 'none',
     };
     for (const callback of mockUseMouseCallbacks) {
       callback(mouseEvent);
@@ -303,6 +308,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
       callback({
         name: 'scroll-down',
@@ -311,6 +317,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
       callback({
         name: 'scroll-up',
@@ -319,6 +326,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
     }
 
@@ -358,6 +366,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
       callback({
         name: 'scroll-down',
@@ -366,6 +375,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
       callback({
         name: 'scroll-down',
@@ -374,6 +384,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'none',
       });
     }
 
@@ -417,6 +428,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -429,6 +441,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -441,6 +454,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -475,6 +489,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -487,6 +502,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }
 
@@ -498,6 +514,7 @@ describe('ScrollProvider', () => {
         shift: false,
         ctrl: false,
         meta: false,
+        button: 'left',
       });
     }