Skip to content

feat(board): align TUI card colors with web board status palette#3521

Merged
dgageot merged 2 commits into
docker:mainfrom
dgageot:worktree-board-299a81ea0ca8bb29
Jul 8, 2026
Merged

feat(board): align TUI card colors with web board status palette#3521
dgageot merged 2 commits into
docker:mainfrom
dgageot:worktree-board-299a81ea0ca8bb29

Conversation

@dgageot

@dgageot dgageot commented Jul 8, 2026

Copy link
Copy Markdown
Member

The TUI board was using arbitrary colors that didn't correspond to the web board's status conventions, and the starting/running colors were swapped from what they should be. Paused cards were also shown in orange rather than a neutral color.

This change introduces a statusColor helper as a single source of truth for status-to-color mapping in the card renderer. Card borders and status lines now use the same semantic palette as the web board: starting/loading/attaching render in blue (info), running in orange (warning), waiting in green (success), paused in white, and error in red. The project accent badge color and the primary highlight border on selected cards are unaffected.

A table-driven test pins the full statusColor contract, including the unknown-status fallback, so future changes to the mapping can't silently regress.

dgageot added 2 commits July 8, 2026 08:21
Introduces a statusColor helper so border and status line share one
source of truth: starting=blue, running=orange, waiting=green,
paused=white, error=red.

Assisted-By: Claude
@dgageot
dgageot requested a review from a team as a code owner July 8, 2026 06:28

@docker-agent docker-agent 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.

Assessment: 🟢 APPROVE

The PR cleanly introduces a statusColor helper to centralize status-to-color mapping and aligns the TUI card colors with the web board's semantic palette. The implementation is consistent with the PR description, the test covers all named statuses plus the unknown-status fallback, and no bugs were found in the changed code.

Diff stats: 2 files changed, 106 diff lines reviewed.

@aheritier aheritier added area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only. labels Jul 8, 2026
@dgageot
dgageot merged commit 932960c into docker:main Jul 8, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants