Skip to content

Commit aec652d

Browse files
samejrclaude
andcommitted
fix(webapp): swap the last hard-coded greys for theme tokens
Four spots still carried fixed neutral colors, so they kept their dark-theme appearance on Light and White: - the agent composer's stop button, a charcoal surface with a white glyph; both now follow the theme, since the surface goes light with it - the API keys "no scope" dot, which tracks the dimmed text color rather than a surface color that would all but vanish on a light card - chart reference-line labels and the dashed line itself Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent e9080b2 commit aec652d

3 files changed

Lines changed: 11 additions & 6 deletions

File tree

apps/webapp/app/components/dashboard-agent/DashboardAgentComposer.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,11 +54,13 @@ export function DashboardAgentComposer({
5454
const sendButton = isStreaming ? (
5555
<Button
5656
variant="minimal/small"
57-
className="aspect-square h-7 min-w-0 bg-charcoal-600 p-1 hover:bg-charcoal-550"
57+
className="aspect-square h-7 min-w-0 bg-surface-control p-1 hover:bg-surface-control-hover"
5858
aria-label="Stop generating"
5959
tooltip="Stop generating"
6060
onClick={onStop}
61-
LeadingIcon={<StopIcon className="size-4 text-white" />}
61+
// Not text-white: this button's surface goes light with the theme, unlike
62+
// the indigo Send button below it.
63+
LeadingIcon={<StopIcon className="size-4 text-text-bright" />}
6264
/>
6365
) : (
6466
<Button

apps/webapp/app/components/primitives/charts/ChartLine.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ function ReferenceLineLabel({
168168
y={viewBox.y}
169169
dominantBaseline="middle"
170170
textAnchor="start"
171-
fill="#878C99"
171+
className="fill-text-dimmed"
172172
fontSize={REFERENCE_LABEL_FONT_SIZE}
173173
>
174174
{value}
@@ -180,7 +180,7 @@ function ReferenceLineLabel({
180180
x={viewBox.x + viewBox.width - 4}
181181
y={viewBox.y + 12}
182182
textAnchor="end"
183-
fill="#878C99"
183+
className="fill-text-dimmed"
184184
fontSize={REFERENCE_LABEL_FONT_SIZE}
185185
>
186186
{value}
@@ -458,7 +458,7 @@ export function ChartLineRenderer({
458458
<ReferenceLine
459459
key={`ref-${line.y}-${line.label ?? ""}`}
460460
y={line.y}
461-
stroke={line.color ?? "#4D525B"}
461+
stroke={line.color ?? "var(--color-border-brighter)"}
462462
strokeDasharray="4 4"
463463
strokeWidth={1}
464464
// extendDomain (not "hidden") means recharts does NOT wrap this layer in the plot

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.apikeys/route.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1097,7 +1097,10 @@ function ApiKeyScopePanel({
10971097
? "bg-blue-500"
10981098
: tone === "write"
10991099
? "bg-amber-500"
1100-
: "bg-charcoal-600"
1100+
: // A dot has to stay visible, so this tracks the
1101+
// dimmed text color rather than a surface color
1102+
// that would all but vanish on a light card.
1103+
"bg-text-dimmed"
11011104
)}
11021105
/>
11031106
<span

0 commit comments

Comments
 (0)