Disable followup questions in the inputOnTop mode (microsoft#187884)

For now, until we iron out what they should look like.

Author: TylerLeonhardt

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

@@ -77,7 +77,7 @@ export class ChatInputPart extends Disposable implements IHistoryNavigationWidge
 
 	constructor(
 		// private readonly editorOptions: ChatEditorOptions, // TODO this should be used
-		private readonly options: { renderFollowupsBelow: boolean },
+		private readonly options: { renderFollowups: boolean },
 		@IChatWidgetHistoryService private readonly historyService: IChatWidgetHistoryService,
 		@IModelService private readonly modelService: IModelService,
 		@IInstantiationService private readonly instantiationService: IInstantiationService,
@@ -178,14 +178,8 @@ export class ChatInputPart extends Disposable implements IHistoryNavigationWidge
 	render(container: HTMLElement, initialValue: string, widget: IChatWidget) {
 		this.container = dom.append(container, $('.interactive-input-part'));
 
-		let inputContainer: HTMLElement;
-		if (this.options.renderFollowupsBelow) {
-			inputContainer = dom.append(this.container, $('.interactive-input-and-toolbar'));
-			this.followupsContainer = dom.append(this.container, $('.interactive-input-followups'));
-		} else {
-			this.followupsContainer = dom.append(this.container, $('.interactive-input-followups'));
-			inputContainer = dom.append(this.container, $('.interactive-input-and-toolbar'));
-		}
+		this.followupsContainer = dom.append(this.container, $('.interactive-input-followups'));
+		const inputContainer = dom.append(this.container, $('.interactive-input-and-toolbar'));
 
 		const inputScopedContextKeyService = this._register(this.contextKeyService.createScoped(inputContainer));
 		CONTEXT_IN_CHAT_INPUT.bindTo(inputScopedContextKeyService).set(true);
@@ -256,6 +250,9 @@ export class ChatInputPart extends Disposable implements IHistoryNavigationWidge
 	}
 
 	async renderFollowups(items?: IChatReplyFollowup[]): Promise<void> {
+		if (!this.options.renderFollowups) {
+			return;
+		}
 		this.followupsDisposables.clear();
 		dom.clearNode(this.followupsContainer);
 

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

@@ -142,7 +142,7 @@ export class ChatWidget extends Disposable implements IChatWidget {
 
 		this.container = dom.append(parent, $('.interactive-session'));
 		if (renderInputOnTop) {
-			this.createInput(this.container, { renderFollowupsBelow: true });
+			this.createInput(this.container, { renderFollowups: false });
 			this.listContainer = dom.append(this.container, $(`.interactive-list`));
 		} else {
 			this.listContainer = dom.append(this.container, $(`.interactive-list`));
@@ -334,8 +334,8 @@ export class ChatWidget extends Disposable implements IChatWidget {
 		this.previousTreeScrollHeight = this.tree.scrollHeight;
 	}
 
-	private createInput(container: HTMLElement, options?: { renderFollowupsBelow: boolean }): void {
-		this.inputPart = this.instantiationService.createInstance(ChatInputPart, { renderFollowupsBelow: options?.renderFollowupsBelow ?? false });
+	private createInput(container: HTMLElement, options?: { renderFollowups: boolean }): void {
+		this.inputPart = this.instantiationService.createInstance(ChatInputPart, { renderFollowups: options?.renderFollowups ?? true });
 		this.inputPart.render(container, '', this);
 
 		this._register(this.inputPart.onDidFocus(() => this._onDidFocus.fire()));