Skip to content

Commit aa695e6

Browse files
teallarsonclaude
andcommitted
Fix WCAG contrast on tool status pills
muted-foreground (~#9a9a9a) on muted background (~#363636 dark / #f5f5f5 light) is ~4.2:1 and ~4.4:1 respectively — both under the 4.5:1 AA threshold for 12px text. Adding text-foreground to each pill overrides the inherited muted-foreground, giving white-on-dark (~14:1) and near-black-on-light (~13:1) in all states. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8eb6564 commit aa695e6

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

templates/_shared/nextjs-ui/components/dashboard/tool-status.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ function StatusDot({ status }: { status: SourceStatus }) {
3939
if (status === "auth_required") {
4040
return <span className="size-2.5 rounded-full bg-amber-500" />;
4141
}
42+
if (status === "skipped") {
43+
return <span className="size-2.5 rounded-full bg-gray-300 opacity-50" />;
44+
}
4245
return <span className="size-2.5 rounded-full bg-gray-300" />;
4346
}
4447

@@ -61,12 +64,13 @@ export function ToolStatusBar({ statuses, authUrls }: ToolStatusBarProps) {
6164
const pill = (
6265
<span
6366
className={cn(
64-
"inline-flex items-center gap-1.5 rounded-full border px-2.5 py-1 transition-colors",
67+
"inline-flex items-center gap-1.5 rounded-full border px-2.5 py-1 text-foreground transition-colors",
6568
status === "connected" &&
6669
"border-green-200 bg-green-50 dark:border-green-900 dark:bg-green-950",
6770
status === "auth_required" &&
6871
"border-amber-200 bg-amber-50 dark:border-amber-900 dark:bg-amber-950",
69-
(status === "checking" || status === "unknown") && "border-border bg-muted"
72+
(status === "checking" || status === "unknown") && "border-border bg-muted",
73+
status === "skipped" && "border-border bg-muted opacity-50"
7074
)}
7175
>
7276
<Icon className="size-3" />

0 commit comments

Comments
 (0)