From 563d18cd2069e1c448137f1fbcf01a0928bc8196 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 17 May 2026 01:30:29 +0000 Subject: [PATCH 1/2] Initial plan From cee7a683f66c064a8299ff6337083d6f9b188987 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 17 May 2026 01:36:38 +0000 Subject: [PATCH 2/2] Fix outline view double-click jump when openMode=doubleClick Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com> --- src/vs/workbench/contrib/outline/browser/outlinePane.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/vs/workbench/contrib/outline/browser/outlinePane.ts b/src/vs/workbench/contrib/outline/browser/outlinePane.ts index e59e5bd2d7ff2..4971de810e1af 100644 --- a/src/vs/workbench/contrib/outline/browser/outlinePane.ts +++ b/src/vs/workbench/contrib/outline/browser/outlinePane.ts @@ -312,6 +312,11 @@ export class OutlinePane extends ViewPane implements IOutlinePane { this._editorControlDisposables.add(tree.onDidOpen(async e => { const myId = ++idPool; const isDoubleClick = e.browserEvent?.type === 'dblclick'; + // when `workbench.list.openMode` is `doubleClick`, a double click is the + // primary way to open an entry and should jump to the symbol (not select + // its full range), matching the single-click behavior in the default mode. + const openOnSingleClick = this.configurationService.getValue('workbench.list.openMode') !== 'doubleClick'; + const selectSymbolRange = isDoubleClick && openOnSingleClick; if (!isDoubleClick) { // workaround for https://github.com/microsoft/vscode/issues/206424 await timeout(150); @@ -319,7 +324,7 @@ export class OutlinePane extends ViewPane implements IOutlinePane { return; } } - await newOutline.reveal(e.element, e.editorOptions, e.sideBySide, isDoubleClick); + await newOutline.reveal(e.element, e.editorOptions, e.sideBySide, selectSymbolRange); })); // feature: reveal editor selection in outline const revealActiveElement = () => {