From b065dc2b6079bfc1522727425811e1c8dec87fa6 Mon Sep 17 00:00:00 2001
From: Tyler James Leonhardt <me@tylerleonhardt.com>
Date: Wed, 27 Mar 2024 14:08:23 -0700
Subject: [PATCH] Reveal first child, not separator (#208953)

Fixes https://github.com/microsoft/vscode/issues/208696
---
 src/vs/platform/quickinput/browser/quickInputTree.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/vs/platform/quickinput/browser/quickInputTree.ts b/src/vs/platform/quickinput/browser/quickInputTree.ts
index 672f581e755..8016a9dff34 100644
--- a/src/vs/platform/quickinput/browser/quickInputTree.ts
+++ b/src/vs/platform/quickinput/browser/quickInputTree.ts
@@ -1292,9 +1292,9 @@ export class QuickInputTree extends Disposable {
 
 					if (e.element instanceof QuickPickSeparatorElement) {
 						foundSeparatorAsItem = true;
-						// If the separator is visible, then we should just focus it.
+						// If the separator is visible, then we should just reveal its first child so it's not as jarring.
 						if (this._separatorRenderer.isSeparatorVisible(e.element)) {
-							this._tree.reveal(e.element);
+							this._tree.reveal(e.element.children[0]);
 						} else {
 							// If the separator is not visible, then we should
 							// push it up to the top of the list.