From 6646f2b25433f3ef1fdce57a8bb7038a2859527a Mon Sep 17 00:00:00 2001
From: Roo Code <roomote@roocode.com>
Date: Tue, 14 Oct 2025 07:20:06 +0000
Subject: [PATCH] fix: remove duplicate nested div wrapper in CheckpointMenu
 component

---
 .../chat/checkpoints/CheckpointMenu.tsx       | 74 +++++++++----------
 1 file changed, 36 insertions(+), 38 deletions(-)

diff --git a/webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx b/webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx
index c4e2f63c15..aaeb5a43ff 100644
--- a/webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx
+++ b/webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx
@@ -101,46 +101,44 @@ export const CheckpointMenu = ({
 									</div>
 								</div>
 								<div className="flex flex-col gap-1 group hover:text-foreground">
-									<div className="flex flex-col gap-1 group hover:text-foreground">
-										{!isConfirming ? (
+									{!isConfirming ? (
+										<Button
+											variant="secondary"
+											onClick={() => setIsConfirming(true)}
+											data-testid="restore-files-and-task-btn">
+											{t("chat:checkpoint.menu.restoreFilesAndTask")}
+										</Button>
+									) : (
+										<>
 											<Button
-												variant="secondary"
-												onClick={() => setIsConfirming(true)}
-												data-testid="restore-files-and-task-btn">
-												{t("chat:checkpoint.menu.restoreFilesAndTask")}
+												variant="default"
+												onClick={onRestore}
+												className="grow"
+												data-testid="confirm-restore-btn">
+												<div className="flex flex-row gap-1">
+													<CheckIcon />
+													<div>{t("chat:checkpoint.menu.confirm")}</div>
+												</div>
 											</Button>
-										) : (
-											<>
-												<Button
-													variant="default"
-													onClick={onRestore}
-													className="grow"
-													data-testid="confirm-restore-btn">
-													<div className="flex flex-row gap-1">
-														<CheckIcon />
-														<div>{t("chat:checkpoint.menu.confirm")}</div>
-													</div>
-												</Button>
-												<Button variant="secondary" onClick={() => setIsConfirming(false)}>
-													<div className="flex flex-row gap-1">
-														<Cross2Icon />
-														<div>{t("chat:checkpoint.menu.cancel")}</div>
-													</div>
-												</Button>
-											</>
-										)}
-										{isConfirming ? (
-											<div
-												data-testid="checkpoint-confirm-warning"
-												className="text-destructive font-bold">
-												{t("chat:checkpoint.menu.cannotUndo")}
-											</div>
-										) : (
-											<div className="text-muted transition-colors group-hover:text-foreground">
-												{t("chat:checkpoint.menu.restoreFilesAndTaskDescription")}
-											</div>
-										)}
-									</div>
+											<Button variant="secondary" onClick={() => setIsConfirming(false)}>
+												<div className="flex flex-row gap-1">
+													<Cross2Icon />
+													<div>{t("chat:checkpoint.menu.cancel")}</div>
+												</div>
+											</Button>
+										</>
+									)}
+									{isConfirming ? (
+										<div
+											data-testid="checkpoint-confirm-warning"
+											className="text-destructive font-bold">
+											{t("chat:checkpoint.menu.cannotUndo")}
+										</div>
+									) : (
+										<div className="text-muted transition-colors group-hover:text-foreground">
+											{t("chat:checkpoint.menu.restoreFilesAndTaskDescription")}
+										</div>
+									)}
 								</div>
 							</>
 						)}