rm focus from reveal

Author: meganrogge

src/vs/workbench/contrib/chat/browser/chat.contribution.ts

@@ -150,7 +150,7 @@ class ChatAccessibleViewContribution extends Disposable {
 				id: 'panelChat',
 				provideContent(): string { return responseContent; },
 				onClose() {
-					widget.reveal(focusedItem, true);
+					widget.focus(focusedItem);
 					provider.dispose();
 				},
 				options: { ariaLabel: nls.localize('chatAccessibleView', "Chat Accessible View"), language: 'typescript', type: AccessibleViewType.View }

src/vs/workbench/contrib/chat/browser/chat.ts

@@ -54,7 +54,8 @@ export interface IChatWidget {
 	readonly inputEditor: ICodeEditor;
 	readonly providerId: string;
 
-	reveal(item: ChatTreeItem, focus?: boolean): void;
+	reveal(item: ChatTreeItem): void;
+	focus(item: ChatTreeItem): void;
 	getFocus(): ChatTreeItem | undefined;
 	acceptInput(query?: string): void;
 	focusLastMessage(): void;

src/vs/workbench/contrib/chat/browser/chatWidget.ts

@@ -376,18 +376,19 @@ export class ChatWidget extends Disposable implements IChatWidget {
 		return withNullAsUndefined(this.tree.getFocus()[0]);
 	}
 
-	reveal(item: ChatTreeItem, focus?: boolean): void {
+	reveal(item: ChatTreeItem): void {
 		this.tree.reveal(item);
-		if (focus) {
-			const items = this.tree.getNode(null).children;
-			const node = items.find(i => i.element?.id === item.id);
-			if (!node) {
-				return;
-			}
+	}
 
-			this.tree.setFocus([node.element]);
-			this.tree.domFocus();
+	focus(item: ChatTreeItem): void {
+		const items = this.tree.getNode(null).children;
+		const node = items.find(i => i.element?.id === item.id);
+		if (!node) {
+			return;
 		}
+
+		this.tree.setFocus([node.element]);
+		this.tree.domFocus();
 	}
 
 	async acceptInput(query?: string | IChatReplyFollowup): Promise<void> {