Skip to content

fix(tui): improve contrast and spinner cadence#64

Merged
BunsDev merged 1 commit into
mainfrom
codex/issue-55-tui-accessibility
Jun 11, 2026
Merged

fix(tui): improve contrast and spinner cadence#64
BunsDev merged 1 commit into
mainfrom
codex/issue-55-tui-accessibility

Conversation

@BunsDev

@BunsDev BunsDev commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

  • raise TUI muted/disabled theme colors so they meet AA contrast on their backgrounds
  • route composer/status dim text through the shared muted color
  • slow the animated status spinner to advance every two render ticks instead of every tick

Verification

  • cargo fmt -p claurst-tui --check
  • cargo test -p claurst-tui
  • cargo check -p claurst
  • git diff --check origin/main...HEAD

Fixes #55

Copilot AI review requested due to automatic review settings June 11, 2026 03:16
@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Jun 11, 2026 3:16am

@BunsDev BunsDev merged commit f6a61b9 into main Jun 11, 2026
2 checks passed
@BunsDev BunsDev deleted the codex/issue-55-tui-accessibility branch June 11, 2026 03:17

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.

Pull request overview

Improves TUI accessibility and visual comfort by increasing muted/disabled text contrast and reducing the effective flash rate of the streaming status spinner.

Changes:

  • Raised muted/disabled color values across themes to meet WCAG AA contrast targets on their intended backgrounds.
  • Routed dim status/composer text through the shared COVEN_CODE_MUTED constant for consistent styling.
  • Slowed the status spinner animation to advance every two render ticks and added a regression test for cadence.

Reviewed changes

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

File Description
src-rust/crates/tui/src/theme_colors.rs Updates per-theme disabled colors and adds a contrast-ratio test to enforce WCAG AA for muted/disabled text on representative theme backgrounds.
src-rust/crates/tui/src/render.rs Imports COVEN_CODE_MUTED, uses it for dim status/composer text, slows spinner frame advancement, and adds a test ensuring the new cadence.
src-rust/crates/tui/src/overlays.rs Updates the shared muted color constant used across overlay rendering to the higher-contrast value.

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

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.

TUI accessibility: muted-text contrast below WCAG AA and spinner flash rate

2 participants