From 59f110e2cf7ec387faf2dde88a69795fad41fc1a Mon Sep 17 00:00:00 2001 From: Cameron Clough Date: Mon, 10 Aug 2026 12:15:31 +0100 Subject: [PATCH] fix(web): align thread-row draft icon with upstream draft glyph --- apps/web/src/components/Sidebar.tsx | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/apps/web/src/components/Sidebar.tsx b/apps/web/src/components/Sidebar.tsx index a1a2baf857d..ead28d9f5ea 100644 --- a/apps/web/src/components/Sidebar.tsx +++ b/apps/web/src/components/Sidebar.tsx @@ -44,7 +44,6 @@ import { GitBranchIcon, EllipsisIcon, MessageSquareIcon, - PencilIcon, PinIcon, PlusIcon, SearchIcon, @@ -1105,15 +1104,18 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: { aria-label="Unsent draft" data-testid={`sidebar-v2-draft-${thread.id}`} className={cn( - // Matches the PR badge's settled treatment: muted at rest so the - // parked tail stays quiet, brightening with the row on hover. The - // pencil has no state colour of its own, so it lifts to the same - // foreground the title does. + // Same glyph as the draft-session rows so "unsent draft" reads as + // one concept, but muted rather than amber: on a thread row it's a + // quiet status flag, not a block asking for a decision. Matches the + // PR badge's settled treatment: muted at rest so the parked tail + // stays quiet, brightening with the row on hover. The glyph has no + // state colour of its own, so it lifts to the same foreground the + // title does. "inline-flex shrink-0 items-center justify-center text-secondary-label", !props.isActive && "transition-colors group-hover/v2-row:text-foreground", )} > - + ) : null;