From 5f7230f18f2c1fed3c331294dd323d1e9201baf0 Mon Sep 17 00:00:00 2001 From: Ammar Date: Wed, 17 Jun 2026 10:41:23 -0500 Subject: [PATCH] fix: distinguish plan Copy file path button icon The plan-file display's 'Copy' (content) and 'Copy file path' buttons both used the Clipboard/ClipboardCheck icons, making them visually indistinguishable. Give 'Copy file path' the Copy/Check icons (the repo's copy-a-value idiom, see RuntimeBadge) so it reads as a distinct action from the content Copy button. --- src/browser/features/Tools/ProposePlanToolCall.tsx | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/browser/features/Tools/ProposePlanToolCall.tsx b/src/browser/features/Tools/ProposePlanToolCall.tsx index 1cf55d47c0..e19938f544 100644 --- a/src/browser/features/Tools/ProposePlanToolCall.tsx +++ b/src/browser/features/Tools/ProposePlanToolCall.tsx @@ -61,9 +61,11 @@ import type { ReviewActionCallbacks } from "../Shared/InlineReviewNote"; import { isPlanFilePath, normalizePlanFilePath } from "@/common/types/review"; import type { ThinkingLevel } from "@/common/types/thinking"; import { + Check, Clipboard, ClipboardCheck, ClipboardList, + Copy, FileText, ListStart, MessageSquareOff, @@ -655,11 +657,13 @@ export const ProposePlanToolCall: React.FC = (props) = const actionButtons: ButtonConfig[] = [copyButton]; // Copy the plan's on-disk file path (replaces the former mux.md "Share" link action). + // Use the Copy/Check icons (the repo's copy-a-value idiom, see RuntimeBadge) so this + // button is visually distinct from the Clipboard "Copy" button that copies plan content. if (planPath) { actionButtons.push({ label: copiedPath ? "Copied" : "Copy file path", onClick: () => void copyPathToClipboard(planPath), - icon: copiedPath ? : , + icon: copiedPath ? : , tooltip: planPath, }); }