Skip to content

feat(ui): add theme toggle - #2

Merged
urzeye merged 2 commits into
mainfrom
feature/theme-toggle
Jun 3, 2026
Merged

feat(ui): add theme toggle#2
urzeye merged 2 commits into
mainfrom
feature/theme-toggle

Conversation

@urzeye

@urzeye urzeye commented Jun 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • add dark/light theme palette support and persist the selected mode
  • add a global T shortcut to toggle themes from the main TUI
  • apply theme colors to the main shell, header, list, details, status, hint, and form chrome
  • document the new theme toggle in English and Chinese READMEs
  • include UI settings and theme mode tests

Tests

  • GOCACHE=/private/tmp/lazyssh-gocache go test ./...
  • GOCACHE=/private/tmp/lazyssh-gocache GOLANGCI_LINT_CACHE=/private/tmp/lazyssh-golangci-cache ./bin/golangci-lint run
  • GOCACHE=/private/tmp/lazyssh-gocache go build -o /private/tmp/lazyssh-theme-check ./cmd

@urzeye

urzeye commented Jun 3, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e6fb00dfcf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/adapters/ui/tui.go Outdated
@urzeye
urzeye merged commit 1ecc3b9 into main Jun 3, 2026
2 checks passed
@urzeye
urzeye deleted the feature/theme-toggle branch June 3, 2026 15:19
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.

1 participant