Skip to content

Conversation

@unfunco
Copy link
Owner

@unfunco unfunco commented Nov 16, 2025

Makes consistent the theme configuration so that this can in the future be configured via a config file.

@unfunco unfunco requested a review from Copilot November 16, 2025 13:03
@unfunco unfunco self-assigned this Nov 16, 2025
@unfunco unfunco added the adds a feature 💡 Improvements or additions. label Nov 16, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the theme configuration system to prepare for custom themes, making color handling consistent across both the TUI and CLI help documentation by introducing a unified PaletteColor type that handles conversion between hex colors for Lipgloss and RGBA for Fang.

  • Introduced PaletteColor type to unify color representation with methods for both Lipgloss and RGBA conversions
  • Added Config struct and parsing functions (FromConfig, MustFromConfig, parsePaletteColor) to support configuration-based theme creation
  • Replaced hard-coded ANSI color references with theme-based color references throughout the TUI
  • Renamed tabCount to TabCount (exported) for consistency with other exported Tab constants

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
main.go Updated customColorScheme to use new theme API with RGBA conversion methods, moved function to bottom of file
internal/theme/theme.go Complete rewrite introducing PaletteColor, Config types, hex color parsing, and new WorryStyle method
internal/theme/theme_test.go New test file covering color parsing, shorthand hex, invalid input, and ANSI rejection
internal/tui/tui.go Updated to use new theme API, renamed tabCount to TabCount, removed obsolete comment

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

@unfunco unfunco merged commit 44038ec into main Nov 16, 2025
1 check failed
@unfunco unfunco deleted the feat/theme-config branch November 16, 2025 13:10
@github-actions github-actions bot mentioned this pull request Nov 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adds a feature 💡 Improvements or additions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants