From bcde8d2842cc19b196cbca7b6d2b888c1643b3f2 Mon Sep 17 00:00:00 2001 From: Val Alexander Date: Sat, 4 Apr 2026 13:42:48 -0500 Subject: [PATCH] Use distinct git icons for PR states in sidebar threads MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Thread PR indicators now show GitPullRequestIcon for open, XCircleIcon for closed, and GitMergeIcon for merged — matching the PR Review panel. Co-Authored-By: Claude Opus 4.6 --- apps/web/src/components/Sidebar.tsx | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/apps/web/src/components/Sidebar.tsx b/apps/web/src/components/Sidebar.tsx index 52e9c5048..86dc81827 100644 --- a/apps/web/src/components/Sidebar.tsx +++ b/apps/web/src/components/Sidebar.tsx @@ -8,6 +8,7 @@ import { FolderIcon, GitMergeIcon, GitPullRequestIcon, + XCircleIcon, PlusIcon, RocketIcon, SettingsIcon, @@ -162,6 +163,7 @@ interface PrStatusIndicator { colorClass: string; tooltip: string; url: string; + icon: typeof GitPullRequestIcon; } type ThreadPr = GitStatusResult["pr"]; @@ -188,6 +190,7 @@ function prStatusIndicator(pr: ThreadPr): PrStatusIndicator | null { colorClass: "text-emerald-600 dark:text-emerald-300/90", tooltip: `#${pr.number} PR open: ${pr.title}`, url: pr.url, + icon: GitPullRequestIcon, }; } if (pr.state === "closed") { @@ -196,6 +199,7 @@ function prStatusIndicator(pr: ThreadPr): PrStatusIndicator | null { colorClass: "text-zinc-500 dark:text-zinc-400/80", tooltip: `#${pr.number} PR closed: ${pr.title}`, url: pr.url, + icon: XCircleIcon, }; } if (pr.state === "merged") { @@ -204,6 +208,7 @@ function prStatusIndicator(pr: ThreadPr): PrStatusIndicator | null { colorClass: "text-violet-600 dark:text-violet-300/90", tooltip: `#${pr.number} PR merged: ${pr.title}`, url: pr.url, + icon: GitMergeIcon, }; } return null; @@ -1286,7 +1291,7 @@ export default function Sidebar() { openPrLink(event, prStatus.url); }} > - + } />