Feat/theme picker#543
Conversation
|
Both pending items from #452's review landed in one PR — The picker's One stale-merge note: Will merge as-is. |
|
Actually one ask before I merge — it's |
finished |
|
Thanks for the picker! I rebased it onto current |
Surfaces a SingleSelect modal for the terminal theme preference. Bare \`/theme\` opens the picker; \`/theme <name>\` keeps its existing "persist and report" behaviour. Moves the slash entry from the advanced group to setup and adds high-contrast to the arg completer. Adapted from J3y0r's PR — repointed onto current main (which had since reformatted App.tsx around the InflightProvider wrap from #566) and the count test in tests/ui-slash-suggestions.test.tsx updated from 12 → 11 advanced commands now that /theme has moved out of the advanced bucket. Thanks @J3y0r for the contribution.
Surfaces a SingleSelect modal for the terminal theme preference. Bare \`/theme\` opens the picker; \`/theme <name>\` keeps its existing "persist and report" behaviour. Moves the slash entry from the advanced group to setup and adds high-contrast to the arg completer. Adapted from J3y0r's PR — repointed onto current main (which had since reformatted App.tsx around the InflightProvider wrap from esengine#566) and the count test in tests/ui-slash-suggestions.test.tsx updated from 12 → 11 advanced commands now that /theme has moved out of the advanced bucket. Thanks @J3y0r for the contribution.
What
Added a theme picker selection page with only /theme commands
Why
As a supplement to the theme system
How to verify
run /theme
Checklist
npm run verifypasses locally (lint + typecheck + tests + comment-policy gate)Co-Authored-By: Claudetrailer in commitsCHANGELOG.md— release notes are maintainer-written at release time