diff --git a/apps/web/src/components/Sidebar.tsx b/apps/web/src/components/Sidebar.tsx index 5df4a9ea8b8b..5182ee4d783e 100644 --- a/apps/web/src/components/Sidebar.tsx +++ b/apps/web/src/components/Sidebar.tsx @@ -1468,9 +1468,7 @@ const SidebarThreadRow = memo(function SidebarThreadRow(props: { // number from reflowing as PR states stream in. "shrink-0 text-xs tabular-nums hover:underline", variant === "slim" && variantAction === "unsettle" - ? props.isActive - ? "text-secondary-label" - : cn("text-secondary-label transition-colors", settledPrHoverClass) + ? cn("text-secondary-label transition-colors", settledPrHoverClass) : prStatus.colorClass, )} aria-label={prStatus.tooltip} diff --git a/apps/web/src/components/ThreadStatusIndicators.test.ts b/apps/web/src/components/ThreadStatusIndicators.test.ts index 7dd8c79c9b46..0afa45d27d0a 100644 --- a/apps/web/src/components/ThreadStatusIndicators.test.ts +++ b/apps/web/src/components/ThreadStatusIndicators.test.ts @@ -125,10 +125,12 @@ describe("settledPrHoverColorClass", () => { ["merged", "text-violet-600"], ["closed", "text-red-600"], ] as const)("restores the %s pull request color on row hover", (state, colorClass) => { - expect(settledPrHoverColorClass(state)).toContain(`group-hover/v2-row:${colorClass}`); + expect(settledPrHoverColorClass(state)).toContain(`group-hover/sidebar-row:${colorClass}`); }); it("keeps draft pull requests gray on row hover", () => { - expect(settledPrHoverColorClass("open", true)).toContain("group-hover/v2-row:text-zinc-500"); + expect(settledPrHoverColorClass("open", true)).toContain( + "group-hover/sidebar-row:text-zinc-500", + ); }); }); diff --git a/apps/web/src/components/ThreadStatusIndicators.tsx b/apps/web/src/components/ThreadStatusIndicators.tsx index 768154419927..afe56608eda5 100644 --- a/apps/web/src/components/ThreadStatusIndicators.tsx +++ b/apps/web/src/components/ThreadStatusIndicators.tsx @@ -86,13 +86,13 @@ export function settledPrHoverColorClass( switch (state) { case "open": if (isDraft) { - return "group-hover/v2-row:text-zinc-500 dark:group-hover/v2-row:text-zinc-400/80"; + return "group-hover/sidebar-row:text-zinc-500 dark:group-hover/sidebar-row:text-zinc-400/80"; } - return "group-hover/v2-row:text-emerald-600 dark:group-hover/v2-row:text-emerald-300/90"; + return "group-hover/sidebar-row:text-emerald-600 dark:group-hover/sidebar-row:text-emerald-300/90"; case "merged": - return "group-hover/v2-row:text-violet-600 dark:group-hover/v2-row:text-violet-300/90"; + return "group-hover/sidebar-row:text-violet-600 dark:group-hover/sidebar-row:text-violet-300/90"; case "closed": - return "group-hover/v2-row:text-red-600 dark:group-hover/v2-row:text-red-300/90"; + return "group-hover/sidebar-row:text-red-600 dark:group-hover/sidebar-row:text-red-300/90"; } }