Skip to content

fix(ui): make the background grid visible in the light theme - #8

Merged
fleizean merged 1 commit into
developfrom
gsd/quick-261019-light-grid
Oct 19, 2026
Merged

fleizean merged 1 commit into
developfrom
gsd/quick-261019-light-grid

Conversation

@fleizean

Copy link
Copy Markdown
Member

Summary

The background grid behind the public and admin pages is visible in the light theme again.

Why

In the light theme the grid lines were stroke-gray-300/40: gray-300 (#d1d5db) at 40% on white comes out at about #eceef1, which is practically invisible. In the dark theme the same pattern reads as a faint grid.

Changes

  • grid-pattern.html: light theme stroke stroke-gray-300/40 → stroke-gray-400/50. The dark theme stroke (dark:stroke-gray-700/30) and the fade masks are unchanged.

Notes for reviewers

Four tones were compared side by side on white (gray-300 at 100%, gray-400 at 50% and 70%, gray-500 at 50%); gray-400 at 50% is the closest match to the dark theme's subtlety. The component is used by the public layout and every admin page.

Checklist

  • The pull request targets develop and is about one thing
  • Commit messages follow Conventional Commits (feat(scope): ..., fix(scope): ...)
  • CI is green (backend tests, frontend build and tests, dependency gate)
  • Schema changes are EF Core migrations generated by the tool, not hand-edited
  • A changed backend DTO has its TypeScript twin updated in src/frontend/src/app/core/models
  • New dependencies are justified and their lock files are updated in the same commit
  • No secrets, tokens or personal data in code, logs or this description
  • Code comments are kept to a minimum; the reasoning lives in this description

The grid lines were gray-300 at 40% on white, about #eceef1, which the eye does not pick out.
gray-400 at 50% keeps them as faint as the dark theme's while they stay visible.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fleizean
fleizean merged commit ab6ddad into develop Oct 19, 2026
5 checks passed
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.

1 participant