Skip to content

feat(k8s-ui): dark/light toggle for logs viewer#358

Merged
nadaverell merged 1 commit intomainfrom
feature/logs-dark-toggle
Mar 18, 2026
Merged

feat(k8s-ui): dark/light toggle for logs viewer#358
nadaverell merged 1 commit intomainfrom
feature/logs-dark-toggle

Conversation

@nadaverell
Copy link
Copy Markdown
Contributor

@nadaverell nadaverell commented Mar 18, 2026

Summary

  • Add dark/light mode toggle button (Sun/Moon icon) to the logs viewer toolbar
  • Defaults to dark mode, persisted to localStorage
  • Sets colorScheme on the container so light-dark() CSS variables resolve correctly regardless of host app's theme
  • Explicit monospace font stack (SF Mono, Cascadia Code, Fira Code, Menlo, Consolas) instead of relying on host's --font-mono

Test plan

  • Toggle dark/light in logs viewer — background, text, toolbar should all switch
  • Refresh page — preference should persist

- Logs default to dark mode, toggle persisted to localStorage
- Sun/Moon button in toolbar to switch
- Explicit monospace font stack (SF Mono, Cascadia Code, Fira Code, etc.)
@nadaverell nadaverell requested a review from hisco as a code owner March 18, 2026 23:56
@nadaverell nadaverell merged commit 64d41eb into main Mar 18, 2026
5 of 6 checks passed
@nadaverell nadaverell deleted the feature/logs-dark-toggle branch March 18, 2026 23:57
@nadaverell nadaverell restored the feature/logs-dark-toggle branch March 19, 2026 00:16
nadaverell added a commit that referenced this pull request Mar 19, 2026
- Logs default to dark mode, toggle persisted to localStorage
- Sun/Moon button in toolbar to switch
- Explicit monospace font stack (SF Mono, Cascadia Code, Fira Code, etc.)
nadaverell added a commit that referenced this pull request Mar 19, 2026
- Logs default to dark mode, toggle persisted to localStorage
- Sun/Moon button in toolbar to switch
- Explicit monospace font stack (SF Mono, Cascadia Code, Fira Code, etc.)
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