Skip to content

fix: improve neutral text and search hover contrast - #88

Open
zeyuyaoy wants to merge 1 commit into
mainfrom
colors
Open

zeyuyaoy wants to merge 1 commit into
mainfrom
colors

Conversation

@zeyuyaoy

@zeyuyaoy zeyuyaoy commented Sep 20, 2026

Copy link
Copy Markdown
Collaborator

max wanted smaller commits :(

@zeyuyaoy zeyuyaoy self-assigned this Sep 20, 2026
Copilot AI lite review requested due to automatic review settings September 20, 2026 02:07
@zeyuyaoy
zeyuyaoy requested a review from TenType as a code owner September 20, 2026 02:07
@zeyuyaoy zeyuyaoy added the enhancement New feature or request label Sep 20, 2026
@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 4fc5e004-1660-4fc1-a384-fa2b85c39e25

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Improves color consistency, contrast, and focus visibility across the interface, including dorm banners and search controls.

Changes:

  • Adds shared UI color tokens.
  • Selects contrasting dorm banner text colors.
  • Improves home, search, and keyboard hint styling.
File Summary
src/​utils/​bannerColors.ts Computes accessible banner colors.
src/​styles/​home.css Uses shared accent and focus colors.
src/​styles/​global.css Defines UI color tokens.
src/​components/​SearchCommand.svelte Improves hover and selection styling.
src/​components/​Kbd.svelte Uses the keyboard-text color token.
src/​components/​DormCard.svelte Applies dynamic banner colors.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@zeyuyaoy zeyuyaoy changed the title feat: colors fix: improve neutral text and search hover contrast Sep 20, 2026
@zeyuyaoy
zeyuyaoy added this pull request to stack #90 September 20, 2026 02:41

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants