Repository navigation
Add Console Themes, saved look settings and activity motion - #870
Merged
Merged
Conversation
Preserve the Theme routes and authenticated Stop routes. Use the shared Command watcher for Inbox, time-page and peek updates, retaining Theme activity and polling recovery. Apply the saved Pause motion preference on the Stop page and cover it alongside reduced motion. Validated with cargo test --locked --quiet, cargo clippy --all-targets -- -D warnings, cargo fmt --check, and all 32 Console browser tests.
This was referenced Oct 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Six built-in Themes, each in auto/light/dark, share the Console layout and embed all ten font families locally.
Closes #824
Evidence
After: all 18 choices render, settings persist across browsers, and HTTP/CLI tests cover comment preservation, account attribution, CSRF and rejected unrelated keys. Both write routes have CLI pairs.
After: those same commands pass, with three retained regression tests. AA checks cover every built-in text/background token pair, rendered text and black/white solid tags; Focus also checks APCA.
cargo test --quiet,cargo fmt --check, andcargo clippy --all-targets -- -D warningspass. All 30 Console Playwright tests pass (npm test --prefix console, using an isolated copy of the built binary).Test seams
Review coverage
Independent Standards and Spec reviews against
issue-776at7788650, followed by one review of the fixes. Both axes read all 52 implementation/regression paths then present, with no generated-output exclusions.Unread changed file on both axes:
src/setup.rs. Final validation found that its existing complete-config fixture omitted the new default; its only change addshidden_sections = []to that fixture. This test-only update occurred after both review follow-ups.Unaddressed findings
Merge Danger
Door: two-way
Blast Radius: Console
Adds
console.hidden_sectionsand accepts all built-in Theme names. The shared look writer changes only the chosen Theme and hidden sections; Pause motion stays in the browser.Built with codex · gpt-6.1-sol · high