#149034 Select all when shift click in outline

src/vs/workbench/contrib/codeEditor/browser/outline/documentSymbolsOutline.ts

@@ -217,16 +217,20 @@ class DocumentSymbolsOutline implements IOutline<DocumentSymbolItem> {
 		return this._outlineModel?.uri;
 	}
 
-	async reveal(entry: DocumentSymbolItem, options: IEditorOptions, sideBySide: boolean): Promise<void> {
+	async reveal(entry: DocumentSymbolItem, options: IEditorOptions, sideBySide: boolean, entireSelection?: boolean): Promise<void> {
 		const model = OutlineModel.get(entry);
 		if (!model || !(entry instanceof OutlineElement)) {
 			return;
 		}
+
+		// #149034 Use entire selection range conditionally (i.e. When user is holding the Shift key)
+		const range = entireSelection ? entry.symbol.selectionRange : Range.collapseToStart(entry.symbol.selectionRange);
+
 		await this._codeEditorService.openCodeEditor({
 			resource: model.uri,
 			options: {
 				...options,
-				selection: Range.collapseToStart(entry.symbol.selectionRange),
+				selection: range,
 				selectionRevealType: TextEditorSelectionRevealType.NearTopIfOutsideViewport,
 			}
 		}, this._editor, sideBySide);

src/vs/workbench/contrib/outline/browser/outlinePane.ts

@@ -300,7 +300,11 @@ export class OutlinePane extends ViewPane {
 
 		// feature: reveal outline selection in editor
 		// on change -> reveal/select defining range
-		this._editorControlDisposables.add(tree.onDidOpen(e => newOutline.reveal(e.element, e.editorOptions, e.sideBySide)));
+		this._editorControlDisposables.add(tree.onDidOpen(e => {
+			// feature #149034:  If shift key, tell reveal to use entire selection
+			const entireSelection = (e.browserEvent instanceof PointerEvent && (e.browserEvent as PointerEvent).shiftKey);
+			newOutline.reveal(e.element, e.editorOptions, e.sideBySide, entireSelection);
+		}));
 		// feature: reveal editor selection in outline
 		const revealActiveElement = () => {
 			if (!this._outlineViewState.followCursor || !newOutline.activeElement) {

src/vs/workbench/services/outline/browser/outline.ts

@@ -83,7 +83,7 @@ export interface IOutline<E> {
 	readonly activeElement: E | undefined;
 	readonly onDidChange: Event<OutlineChangeEvent>;
 
-	reveal(entry: E, options: IEditorOptions, sideBySide: boolean): Promise<void> | void;
+	reveal(entry: E, options: IEditorOptions, sideBySide: boolean, entireSelection?: boolean): Promise<void> | void;
 	preview(entry: E): IDisposable;
 	captureViewState(): IDisposable;
 	dispose(): void;