Skip to content

Fix grid line color contrast against muted cells - #48

Merged
injoon5 merged 1 commit into
mainfrom
claude/line-color-top-left-cells-s7mda2
Aug 21, 2026
Merged

injoon5 merged 1 commit into
mainfrom
claude/line-color-top-left-cells-s7mda2

Conversation

@injoon5

@injoon5 injoon5 commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

Adjusted the --grid-line CSS custom property to mix against --muted instead of --card for better visual contrast in data grid layouts.

Changes

  • Updated --grid-line color calculation to use var(--muted) as the mix target instead of var(--card)
  • Added explanatory comment clarifying that header and row header cells use bg-muted and sit on the grid's outer edges, where a line tuned only for --card contrast becomes nearly invisible

Details

The grid lines were previously tuned for contrast against the card background color, but header cells (which use the muted background) sit on the grid's top and left edges. This caused the grid lines to be nearly imperceptible in those critical areas. By mixing the grid line color against the muted background instead, the lines now maintain adequate contrast throughout the entire grid layout.

https://claude.ai/code/session_01J7ETLBhYrmFojZ1Mo4g3ns

--grid-line was mixed against --card, giving it almost no contrast
against --muted in dark mode — the exact background used by the
timetable's top header row and left period column, where the line
appeared to vanish.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J7ETLBhYrmFojZ1Mo4g3ns
@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
class-info Error Error Aug 21, 2026 7:03am

@injoon5
injoon5 merged commit 2dcd2e8 into main Aug 21, 2026
2 of 3 checks passed

This branch had an error being deployed

1 failed deployment
Preview — b679689f Deployed Aug 21, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants