Skip to content

fix: preserve working state color fallback - #83

Open
yltx wants to merge 1 commit into
NanmiCoder:mainfrom
yltx:fix/working-state-color
Open

fix: preserve working state color fallback#83
yltx wants to merge 1 commit into
NanmiCoder:mainfrom
yltx:fix/working-state-color

Conversation

@yltx

@yltx yltx commented Aug 26, 2026

Copy link
Copy Markdown

Summary

  • use the panel fallback-bearing business color alias for working member and captain states
  • add a verification assertion that pins the working-state selector to that compatibility bridge

Root cause

The DOM and selector were correct, but the color used an unguarded host token. When that token is absent, the working label and currentColor glyph retain the tertiary gray base color.

Verification

  • pnpm typecheck
  • pnpm build
  • node scripts/verify.mjs (new assertion passes; existing Windows transient directory-lock EPERM remains environment-sensitive)

Copilot AI lite review requested due to automatic review settings August 26, 2026 13:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@spacexun2

Copy link
Copy Markdown
Contributor

主张逐点验证成立:基础规则落到 tertiary 灰、桥内 --dsw-alias-bg-fill-business 自带 #4d6bfe 字面兜底、WorkGlyphfill="currentColor" 会跟着 state 文字一起被救回来——verify 新断言的信息串两半都对得上。对 5fe388f apply-check 干净。一个后续线索(不建议塞进本 PR):面板 CSS 里还有约二十处直引裸 business token 做背景/描边,同样会在缺 token host 下退化,适合单开 issue 拉清单,本 PR 保持定点修复就好。

@yltx

yltx commented Aug 31, 2026

Copy link
Copy Markdown
Author

感谢详细复核与实机验证!\n\n关于面板其他裸引 business token 的地方,确实更适合后续单独拉 issue 系统性梳理和补齐 fallback alias,本 PR 保持定点修复。

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