Skip to content

Commit 0439d8b

Browse files
fix(details): use text-blue-700 in JsonViewer (dark-mode coverage lint, #433)
The dark-mode coverage script in scripts/check-dark-mode-coverage.mjs linted against text-blue-600 (no .dark remap in apps/web/src/index.css); the JSX-side dark:text-blue-400 variant alone is not what the script accepts. Switching to text-blue-700 uses the existing .dark .text-blue-700 remap in index.css. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 2987c39 commit 0439d8b

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

frontend/packages/details/src/JsonViewer.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,11 @@ function Node({ value, depth }: { value: unknown; depth: number }) {
6868
<span className="text-purple-600 dark:text-purple-400">{value ? 'true' : 'false'}</span>
6969
);
7070
if (typeof value === 'number')
71-
return <span className="text-blue-600 dark:text-blue-400">{String(value)}</span>;
71+
// Use text-blue-700 (the variant with a `.dark` remap in
72+
// apps/web/src/index.css) so dark-mode coverage is automatic
73+
// — the JSX-side `dark:` variant alone is not what the
74+
// check-dark-mode-coverage.mjs lint accepts (#433).
75+
return <span className="text-blue-700">{String(value)}</span>;
7276
if (typeof value === 'string')
7377
return (
7478
<span className="text-green-700 dark:text-green-400">

0 commit comments

Comments
 (0)