Unable to jump to first element in quick open with Arrow-Down and filtered (fixes #11882)

src/vs/base/parts/quickopen/browser/quickOpenWidget.ts

@@ -335,59 +335,55 @@ export class QuickOpenWidget implements IModelProvider {
 	private navigateInTree(keyCode: KeyCode, isShift?: boolean): void {
 		const model: IModel<any> = this.tree.getInput();
 		const entries = model ? model.entries : [];
-		let focus = this.tree.getFocus();
-		let cycled = false;
-		let revealToTop = false;
+		const oldFocus = this.tree.getFocus();
 
-		// Support cycle-through navigation
-		if (entries.length > 1) {
+		// Normal Navigation
+		switch (keyCode) {
+			case KeyCode.DownArrow:
+				this.tree.focusNext();
+				break;
 
-			// Up from no entry or first entry goes down to last
-			if ((keyCode === KeyCode.UpArrow || (keyCode === KeyCode.Tab && isShift)) && (focus === entries[0] || !focus)) { // TODO@Ben should not make ordering assumptions
-				this.tree.focusLast();
-				cycled = true;
-			}
+			case KeyCode.UpArrow:
+				this.tree.focusPrevious();
+				break;
 
-			// Down from last entry goes to up to first
-			else if ((keyCode === KeyCode.DownArrow || keyCode === KeyCode.Tab && !isShift) && focus === entries[entries.length - 1]) { // TODO@Ben should not make ordering assumptions
-				this.tree.focusFirst();
-				cycled = true;
-			}
-		}
+			case KeyCode.PageDown:
+				this.tree.focusNextPage();
+				break;
 
-		// Normal Navigation
-		if (!cycled) {
-			switch (keyCode) {
-				case KeyCode.DownArrow:
-					this.tree.focusNext();
-					break;
+			case KeyCode.PageUp:
+				this.tree.focusPreviousPage();
+				break;
 
-				case KeyCode.UpArrow:
+			case KeyCode.Tab:
+				if (isShift) {
 					this.tree.focusPrevious();
-					break;
+				} else {
+					this.tree.focusNext();
+				}
+				break;
+		}
 
-				case KeyCode.PageDown:
-					this.tree.focusNextPage();
-					break;
+		let newFocus = this.tree.getFocus();
 
-				case KeyCode.PageUp:
-					this.tree.focusPreviousPage();
-					break;
+		// Support cycle-through navigation if focus did not change
+		if (entries.length > 1 && oldFocus === newFocus) {
 
-				case KeyCode.Tab:
-					if (isShift) {
-						this.tree.focusPrevious();
-					} else {
-						this.tree.focusNext();
-					}
-					break;
+			// Up from no entry or first entry goes down to last
+			if (keyCode === KeyCode.UpArrow || (keyCode === KeyCode.Tab && isShift)) {
+				this.tree.focusLast();
+			}
+
+			// Down from last entry goes to up to first
+			else if (keyCode === KeyCode.DownArrow || keyCode === KeyCode.Tab && !isShift) {
+				this.tree.focusFirst();
 			}
 		}
 
 		// Reveal
-		focus = this.tree.getFocus();
-		if (focus) {
-			revealToTop ? this.tree.reveal(focus, 0).done(null, errors.onUnexpectedError) : this.tree.reveal(focus).done(null, errors.onUnexpectedError);
+		newFocus = this.tree.getFocus();
+		if (newFocus) {
+			this.tree.reveal(newFocus).done(null, errors.onUnexpectedError);
 		}
 	}