Repository navigation
Switches for on/off settings, one-line helper copy - #149
Merged
Merged
Conversation
Settings rows and save prompts carried multi-sentence helper text that buried the controls. Instant-apply toggles now use a Switch, and the descriptions are cut to one sentence in both locales. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Off track and thumb now contrast in both themes; stock tokens left the off state near-invisible in light mode. - Panel auto-scroll checkboxes become small switches, matching the settings rows. - Settings switches reference their description via aria-describedby. - Restore the macOS Dock and the speakers consequence in copy, clarify the app-language step, shorten the remaining mock save prompts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
|
Review follow-up:
Lint, both tsc configs, build and |
Language, suggestions, mock hints and transcript steps showed a subtitle that restated the field's own helper line directly beneath it. Those steps now carry only a title; the transcript step's title says what it asks. Hide-taskbar helper no longer repeats its label. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
|
Second review follow-up:
Lint, both tsc configs, build and |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The taskbar helper told Windows users it hides a Dock icon. The wizard column jumped sideways on the one step tall enough to scroll. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
|
Manual pass in the running app at 840x600 (Windows), EN + RU, light + dark:
Found and fixed in 551d7d8:
|
Reserving the gutter on one side stopped the jump but left every step 7.5px left of centre. Reserve it on both edges. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Closes #148
components/ui/switch.tsx(shadcn Switch, using the existingradix-uidependency - no new package).en.tsandru.ts. Keys and signatures are unchanged.Checks
pnpm linttsc(app + electron)pnpm buildpnpm test:main(includes the ui-language test for Russian strings left in English)🤖 Generated with Claude Code