Skip to content

Commit 333aeb1

Browse files
jeremyederAmbient Code Botclaude
authored
fix(cli): improve contrast for dim phase rows in acpctl TUI (#1578)
## Summary - `PhaseColor()` used color `240` (very dark gray) for idle/completed/cancelled/default phases, making entire table rows nearly invisible on dark terminal backgrounds - Bumped to color `245` (~60% gray) which is readable while still looking subdued compared to active/running phases ## Test plan - [ ] Run `acpctl` TUI and verify idle/completed agent and session rows are readable on dark backgrounds - [ ] Verify active/running rows still stand out visually from idle rows 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Style** * Updated phase status colors in the terminal interface for improved visibility. Succeeded, completed, cancelled, idle, and default phase states now display with brighter coloring, enhancing readability and making status indicators more visually distinct during terminal operations. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Ambient Code Bot <bot@ambient-code.local> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 824b2fa commit 333aeb1

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

  • components/ambient-cli/cmd/acpctl/ambient/tui/views

components/ambient-cli/cmd/acpctl/ambient/tui/views/sessions.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,23 @@ import (
1414
//
1515
// pending -> yellow (33)
1616
// running / active -> orange (214)
17-
// succeeded / completed -> dim (240)
17+
// succeeded / completed -> light gray (245)
1818
// failed -> red (31)
19-
// cancelled / idle -> dim (240)
19+
// cancelled / idle -> light gray (245)
2020
func PhaseColor(phase string) lipgloss.Color {
2121
switch strings.ToLower(phase) {
2222
case "pending":
2323
return lipgloss.Color("33") // yellow
2424
case "running", "active":
2525
return lipgloss.Color("214") // orange
2626
case "succeeded", "completed":
27-
return lipgloss.Color("240") // dim
27+
return lipgloss.Color("245") // light gray — readable on dark backgrounds
2828
case "failed":
2929
return lipgloss.Color("31") // red
3030
case "cancelled", "idle":
31-
return lipgloss.Color("240") // dim
31+
return lipgloss.Color("245") // light gray — readable on dark backgrounds
3232
default:
33-
return lipgloss.Color("240") // dim
33+
return lipgloss.Color("245") // light gray — readable on dark backgrounds
3434
}
3535
}
3636

0 commit comments

Comments
 (0)