Skip to content

Conversation

@IEvangelist
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings January 14, 2026 20:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves accessibility compliance and standardizes CSS formatting across the frontend codebase. The primary focus is on ensuring link colors meet WCAG AA contrast requirements while also applying consistent code formatting.

Changes:

  • Added CSS variable overrides for green and orange colors to meet WCAG AA 4.5:1 contrast ratio requirements
  • Hardcoded link color for Lunaria translation dashboard to ensure accessibility compliance
  • Changed git commit retrieval to use merge-base instead of HEAD for build metadata
  • Applied consistent CSS formatting (removed spaces around > combinator, consolidated multi-line declarations)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/frontend/src/styles/site.css Added WCAG-compliant color overrides for light theme; standardized CSS formatting (spacing, multi-line declarations)
src/frontend/lunaria/styles.css Added hardcoded accessible link color (#7084ff) for Lunaria translation dashboard
src/frontend/scripts/write-git-env.mjs Changed git command from rev-parse HEAD to merge-base main HEAD

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@IEvangelist IEvangelist requested a review from eerhardt January 14, 2026 21:02
Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - let's get the comments updated for the latest code before merge.

@IEvangelist IEvangelist enabled auto-merge (squash) January 16, 2026 18:11
@IEvangelist IEvangelist merged commit a3a29f1 into main Jan 16, 2026
6 checks passed
@IEvangelist IEvangelist deleted the more-a11y branch January 16, 2026 18:16
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.

3 participants