Skip to content

Agent indicators follow the theme's accent; minimal by default (#4) - #5

Merged
Maxaubert merged 1 commit into
mainfrom
feat/4-indicator-follows-accent
Sep 19, 2026
Merged

Maxaubert merged 1 commit into
mainfrom
feat/4-indicator-follows-accent

Conversation

@Maxaubert

Copy link
Copy Markdown
Owner

Closes #4.

What changes

  • The indicator colours follow the theme. Working is the chrome's own accent (the colour the active tab's rule and the selected settings page already wear); finished is the theme's green, moved to the contrast floor on the theme's ground. The fixed orange belonged to one theme and clashed with the rest. termLook now stores the two colours as CHOICES, '' meaning "follow the theme", and lib/agentColors.ts resolves what is in force. A palette whose green IS its accent gets a fallback green, since two states in one colour is no indicator.
  • Your own colour still wins. Pick one and it is used; a "Follow theme" button beside the swatch gives it back. Picking a theme gives both back to the theme, as it resets the other extras.
  • Named for what they are: "Agent working indicator" and "Agent finished indicator", and each hint says whether it is following the theme or is your own colour.
  • Minimal is the default volume, not Full. Existing choices are kept: only a never-set value reads differently.
  • README hero screenshot reshot with the new default; spec and CLAUDE.md record the decisions.

Verification

  • npm run typecheck: clean. npm run lint: 0 errors (the same 7 react-refresh warnings).
  • npm test: 307 pass. New agentColors.test.ts asserts, for all 39 presets, that both colours clear 3:1 on the theme's ground and differ from each other, that working IS the chrome accent, and that the default theme wears Prism indigo #5b5bd6.
  • npm run e2e: 10 scenarios pass. indicator now proves the default is minimal, MEASURES the line's computed colour against --p-accent (91,91,214 both), then turns the volume to Full through Settings before it looks for the finished mark (which is Full's alone).
  • Packaged and installed on this machine; exe timestamp polled (22:56:50 -> 23:18:08), app relaunched.

Notes

  • Version stays 0.1.0: nothing has been released yet (release.yml and the real icon arrive with App icon and first release #2), so there is no published version to step past.
  • Unsigned.

🤖 Generated with Claude Code

… by default (#4)

Working is the chrome's own accent and finished is the theme's green, until the
user picks a colour of their own; "Follow theme" gives it back. Settings names
them Agent working indicator and Agent finished indicator. The volume defaults
to Minimal, where it was Full.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@Maxaubert
Maxaubert merged commit 37d3905 into main Sep 19, 2026
1 check passed
@Maxaubert
Maxaubert deleted the feat/4-indicator-follows-accent branch September 19, 2026 06:30
Maxaubert added a commit that referenced this pull request Sep 25, 2026
…ns, what is on disk (#67) (#68)

- Paste is sanitised: no ESC or control can end the bracketed paste (#1).
- Windows tools are started by full path; main leaves its launch folder (#3).
- Paths are quoted for the shell they go to; cd quoting is complete (#5, #7).
- The updater installs main's own offer only, pre-answers the close at the
  quit, hears a failed handoff and removes its temp folder (#14, #15, #19).
- tabs.json and window.json are written atomically (#18).
- Command-line paths resolve against the typing folder; a drive root works
  (#16, #17). The DWM helper's stdin errors are heard (#13).
- The review and the three-PR plan are in docs/.


Claude-Session: https://claude.ai/code/session_01LJbePcRzre7AusNzPNS2Bk

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
github-actions Bot pushed a commit that referenced this pull request Sep 25, 2026
…ns, what is on disk (#67) (#68)

- Paste is sanitised: no ESC or control can end the bracketed paste (#1).
- Windows tools are started by full path; main leaves its launch folder (#3).
- Paths are quoted for the shell they go to; cd quoting is complete (#5, #7).
- The updater installs main's own offer only, pre-answers the close at the
  quit, hears a failed handoff and removes its temp folder (#14, #15, #19).
- tabs.json and window.json are written atomically (#18).
- Command-line paths resolve against the typing folder; a drive root works
  (#16, #17). The DWM helper's stdin errors are heard (#13).
- The review and the three-PR plan are in docs/.


Claude-Session: https://claude.ai/code/session_01LJbePcRzre7AusNzPNS2Bk

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

Agent indicators follow the theme's accent; minimal by default

1 participant