Move confirmation actions to bottom section

Part of microsoft#257468

Author: Tyriar

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

@@ -351,11 +351,13 @@ abstract class BaseChatConfirmationWidget extends Disposable {
 		const elements = dom.h('.chat-confirmation-widget2@root', [
 			dom.h('.chat-confirmation-widget-title', [
 				dom.h('.chat-title@title'),
-				dom.h('.chat-buttons@buttons'),
+				dom.h('.chat-toolbar-container@buttonsContainer', [
+					dom.h('.chat-toolbar@toolbar'),
+				]),
 			]),
 			dom.h('.chat-confirmation-widget-message@message'),
-			dom.h('.chat-buttons-container@buttonsContainer', [
-				dom.h('.chat-toolbar@toolbar'),
+			dom.h('.chat-confirmation-widget-buttons', [
+				dom.h('.chat-buttons@buttons'),
 			]),
 		]);
 		this._domNode = elements.root;

src/vs/workbench/contrib/chat/browser/chatContentParts/media/chatConfirmationWidget.css

@@ -188,17 +188,6 @@
 	justify-content: space-between;
 	column-gap: 10px;
 
-	.chat-buttons {
-		display: flex;
-		column-gap: 10px;
-		align-items: center;
-	}
-
-	.monaco-button {
-		overflow-wrap: break-word;
-		padding: 2px 4px;
-	}
-
 	.rendered-markdown {
 		line-height: 24px !important;
 		p {
@@ -226,20 +215,43 @@
 	background: var(--vscode-chat-requestBackground);
 	border: 1px solid var(--vscode-chat-requestBorder);
 	padding: 6px 9px 0 9px;
-	border-bottom-left-radius: 3px;
-	border-bottom-right-radius: 3px;
 
 	p {
 		margin-top: 0;
 		margin-bottom: 9px;
 	}
 }
 
+
 .interactive-session .interactive-response .chat-confirmation-widget2 .chat-confirmation-message-terminal .interactive-result-code-block {
-	border-top-left-radius: 0px;
-	border-top-right-radius: 0px;
+	border-radius: 0px;
+}
+.interactive-session .interactive-response .chat-confirmation-widget2 .chat-confirmation-widget-buttons:not(:has(.chat-confirmation-widget-buttons)) .chat-confirmation-message-terminal .interactive-result-code-block {
+	border-bottom-left-radius: 4px;
+	border-bottom-right-radius: 4px;
 }
 
-.chat-confirmation-widget2.hideButtons .chat-buttons {
+.chat-confirmation-widget2.hideButtons .chat-confirmation-widget-buttons,
+.chat-confirmation-widget2.hideButtons .chat-toolbar-container {
 	display: none;
 }
+
+.chat-confirmation-widget2 .chat-confirmation-widget-buttons {
+	display: flex;
+	border: 1px solid var(--vscode-chat-requestBorder);
+	border-top: none;
+	border-bottom-left-radius: 4px;
+	border-bottom-right-radius: 4px;
+	padding: 5px 9px;
+
+	.chat-buttons {
+		display: flex;
+		column-gap: 10px;
+		align-items: center;
+
+		.monaco-button {
+			overflow-wrap: break-word;
+			padding: 2px 4px;
+		}
+	}
+}

src/vs/workbench/contrib/chat/browser/chatContentParts/toolInvocationParts/chatTerminalToolConfirmationSubPart.ts

@@ -87,18 +87,18 @@ export class ChatTerminalToolConfirmationSubPart extends BaseChatToolInvocationS
 		terminalData = migrateLegacyTerminalToolSpecificData(terminalData);
 
 		const { title, message, disclaimer, terminalCustomActions } = toolInvocation.confirmationMessages;
-		const runLabel = localize('run', "Run");
-		const runKeybinding = keybindingService.lookupKeybinding(AcceptToolConfirmationActionId)?.getLabel();
-		const runTooltip = runKeybinding ? `${runLabel} (${runKeybinding})` : runLabel;
+		const continueLabel = localize('continue', "Continue");
+		const continueKeybinding = keybindingService.lookupKeybinding(AcceptToolConfirmationActionId)?.getLabel();
+		const continueTooltip = continueKeybinding ? `${continueLabel} (${continueKeybinding})` : continueLabel;
 		const cancelLabel = localize('cancel', "Cancel");
 		const cancelKeybinding = keybindingService.lookupKeybinding(CancelChatActionId)?.getLabel();
 		const cancelTooltip = cancelKeybinding ? `${cancelLabel} (${cancelKeybinding})` : cancelLabel;
 
 		const buttons: IChatConfirmationButton[] = [
 			{
-				label: runLabel,
+				label: continueLabel,
 				data: true,
-				tooltip: runTooltip,
+				tooltip: continueTooltip,
 				moreActions: terminalCustomActions,
 			},
 			{

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts

@@ -337,8 +337,8 @@ export class RunInTerminalTool extends Disposable implements IToolImpl {
 			}
 			confirmationMessages = isAutoApproved ? undefined : {
 				title: args.isBackground
-					? localize('runInTerminal.background', "{0} (background terminal)", shellType)
-					: shellType,
+					? localize('runInTerminal.background', "Run `{0}` command? (background terminal)", shellType)
+					: localize('runInTerminal', "Run `{0}` command?", shellType),
 				message: new MarkdownString(args.explanation),
 				disclaimer,
 				terminalCustomActions: customActions,