From 7af6f94cf40a6a02cefc682c1e531fcc5bf79599 Mon Sep 17 00:00:00 2001 From: Tyler James Leonhardt <2644648+TylerLeonhardt@users.noreply.github.com> Date: Mon, 29 Dec 2025 00:48:22 -0600 Subject: [PATCH] Have ENTER fire accept in QuickTree (#285289) Fixes https://github.com/microsoft/vscode/issues/278700 --- src/vs/platform/quickinput/browser/tree/quickTree.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/vs/platform/quickinput/browser/tree/quickTree.ts b/src/vs/platform/quickinput/browser/tree/quickTree.ts index 090a711127d..9a8f9389e55 100644 --- a/src/vs/platform/quickinput/browser/tree/quickTree.ts +++ b/src/vs/platform/quickinput/browser/tree/quickTree.ts @@ -36,11 +36,12 @@ export class QuickTree extends QuickInput implements I private readonly _onDidChangeCheckboxState = this._register(new Emitter()); readonly onDidChangeCheckboxState: Event = this._onDidChangeCheckboxState.event; + private readonly _onDidAcceptEmitter = this._register(new Emitter()); readonly onDidAccept: Event; constructor(ui: QuickInputUI) { super(ui); - this.onDidAccept = ui.onDidAccept; + this.onDidAccept = Event.any(ui.onDidAccept, this._onDidAcceptEmitter.event); this._registerAutoruns(); this._register(ui.tree.onDidChangeCheckedLeafItems(e => this._onDidChangeCheckedLeafItems.fire(e as T[]))); this._register(ui.tree.onDidChangeCheckboxState(e => this._onDidChangeCheckboxState.fire(e.item as T))); @@ -243,7 +244,6 @@ export class QuickTree extends QuickInput implements I * @param inBackground Whether you are accepting an item in the background and keeping the picker open. */ accept(_inBackground?: boolean): void { - // No-op for now since we expect only multi-select quick trees which don't need - // the speed of accept. + this._onDidAcceptEmitter.fire(); } }