Skip to content

feat(theme): use a neutral light palette - #939

Merged
xiaocheny214 merged 1 commit into
1024XEngineer:mainfrom
huyanxius:feat/neutral-light-palette
Sep 8, 2026
Merged

feat(theme): use a neutral light palette#939
xiaocheny214 merged 1 commit into
1024XEngineer:mainfrom
huyanxius:feat/neutral-light-palette

Conversation

@huyanxius

Copy link
Copy Markdown
Collaborator

浅色界面的深绿强调改为石墨灰,统一暖白背景、中性文字与选中态。

Why

原来的深绿强调过于抢眼,与素材竞争视觉注意力。

Changes

只调整共享颜色令牌,不改布局和素材。

Verification

vite build、格式检查通过;本地 UI 已验收。

Closes #938

The green accent competed with character artwork.

Use warm white surfaces and graphite emphasis in shared light colors.

Interface hierarchy relies on neutral contrast while media retains its colors.
@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
windup Ready Ready Preview Sep 8, 2026 12:14pm UTC

@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #939   +/-   ##
=======================================
  Coverage   91.27%   91.27%           
=======================================
  Files         193      193           
  Lines       13204    13204           
=======================================
  Hits        12052    12052           
  Misses       1152     1152           
Flag Coverage Δ
backend 91.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fennoai fennoai Bot 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.

Reviewed the complete fixed diff a4f125e6d5d5493c526f168f4e54cf550be8395d...b36c1555efad69e9bafd5ab4761fa575eca51fba. The change is limited to shared light-palette CSS tokens in frontend/src/index.css; consumers retain their semantic token contracts, and the updated text/accent values preserve the intended contrast relationships. GitHub CI checks pass, including frontend checks, lint/test, build, and formatting-related validation. No actionable correctness, compatibility, performance, or security issues found.

Local reproduction was unavailable because this checkout has no installed frontend/node_modules (the package script reported vitest: Permission denied before test execution).

@xiaocheny214
xiaocheny214 merged commit 40af76f into 1024XEngineer:main Sep 8, 2026
10 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.

[Feature]: Use a neutral palette for the light interface

2 participants