Skip to content

fix(desktop): readable startup error text in light mode - #243

Merged
ackness merged 1 commit into
mainfrom
claude/project-thread-uq6ee6
Oct 11, 2026
Merged

ackness merged 1 commit into
mainfrom
claude/project-thread-uq6ee6

Conversation

@ackness

@ackness ackness commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Requested by yy · project thread

Summary / 摘要

The desktop startup screen follows the system colour scheme, but its light-mode rules skipped the error block. When startup failed with the system in light mode, the error message (#a1a1aa) and hint (#d4d4d8) were drawn in near-white text on the #fafafa background, and the red error title and the spinner's centre dot were faint. The light-mode block now gives them dark colours (#52525b, #3f3f46, #dc2626, #71717a). Like the other rules in that block they carry !important, because the block comes before the base rules in the stylesheet.

Found while reviewing the v0.0.50 Windows splash change from #163. It applies on every platform.

Type of change / 变更类型

  • Bug fix / Bug 修复 (fix)

Verification / 验证方式

  • pnpm check: could not run in this environment. Turbo fails to spawn child processes (Exec format error), and the container has Node 22, not Node 26.
  • Tests of the changed packages: pnpm --filter-equivalent pnpm test in apps/desktop passes.
  • node scripts/changelog.mjs check: 1 valid fragment.

Not verified / 未验证: Nobody looked at the screen. The change is four CSS rules inside the existing prefers-color-scheme: light block.

Related issue / context / 关联

#163

Docs sync / 文档同步

  • A changelog fragment docs/changelog.d/splash-light-error-text.md
  • n/a for reference pages, guides and AGENTS.md

🤖 Generated with Claude Code

https://claude.ai/code/session_01HiABm7VuSQwevB8WsHx6EL


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HiABm7VuSQwevB8WsHx6EL
@ackness ackness self-assigned this Oct 11, 2026
@ackness
ackness marked this pull request as ready for review October 11, 2026 10:25
@ackness
ackness merged commit b81ddd9 into main Oct 11, 2026
6 checks passed
@ackness
ackness deleted the claude/project-thread-uq6ee6 branch October 12, 2026 08:52
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.

2 participants