From c1de12f7c1d3f70f8960f6c37e77a898910694a0 Mon Sep 17 00:00:00 2001
From: Alex Ross <alros@microsoft.com>
Date: Thu, 21 Mar 2019 17:48:06 +0100
Subject: [PATCH] Fix jumping cursor on up arrow in quick pick (#70761)

Fixes #69378
---
 src/vs/workbench/browser/parts/quickinput/quickInput.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/vs/workbench/browser/parts/quickinput/quickInput.ts b/src/vs/workbench/browser/parts/quickinput/quickInput.ts
index e2018e0f8a6..67f3da5e769 100644
--- a/src/vs/workbench/browser/parts/quickinput/quickInput.ts
+++ b/src/vs/workbench/browser/parts/quickinput/quickInput.ts
@@ -499,6 +499,7 @@ class QuickPick<T extends IQuickPickItem> extends QuickInput implements IQuickPi
 							if (this.canSelectMany) {
 								this.ui.list.domFocus();
 							}
+							event.preventDefault();
 							break;
 						case KeyCode.UpArrow:
 							if (this.ui.list.getFocusedElements().length) {
@@ -509,6 +510,7 @@ class QuickPick<T extends IQuickPickItem> extends QuickInput implements IQuickPi
 							if (this.canSelectMany) {
 								this.ui.list.domFocus();
 							}
+							event.preventDefault();
 							break;
 						case KeyCode.PageDown:
 							if (this.ui.list.getFocusedElements().length) {
@@ -519,6 +521,7 @@ class QuickPick<T extends IQuickPickItem> extends QuickInput implements IQuickPi
 							if (this.canSelectMany) {
 								this.ui.list.domFocus();
 							}
+							event.preventDefault();
 							break;
 						case KeyCode.PageUp:
 							if (this.ui.list.getFocusedElements().length) {
@@ -529,6 +532,7 @@ class QuickPick<T extends IQuickPickItem> extends QuickInput implements IQuickPi
 							if (this.canSelectMany) {
 								this.ui.list.domFocus();
 							}
+							event.preventDefault();
 							break;
 					}
 				}),
-- 
GitLab