Add arrow-key destination selection - #37
Merged
Merged
Conversation
While typing on desktop, Up/Down now move a highlighted selection through the ranked score rows; the engine hint mirrors the selection and Enter routes there, overriding the model's pick. Esc reverts to the model's choice, and any new keystroke re-classifies and drops the selection so a stale choice can never fire against a fresh ranking. Arrow keys keep their native caret behavior when no scores are shown. The routing overlay (?q= links / mobile chooser) gets the same treatment: arrows move the selection between override buttons and Enter confirms via the existing click delegation path. Documented in the `?` help overlay and README; covered by 7 new Playwright tests (selection movement, wrap-around, Enter override, Esc revert, stale-selection reset, caret preservation, overlay navigation). Co-Authored-By: Claude Fable 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
1 participant
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
Keyboard shortcuts for changing the routing destination, especially on desktop:
↑/↓move a highlighted selection through the ranked score rows; the engine hint mirrors the selection so you can see whereEnterwill go.Enterroutes to the highlighted engine, overriding the model's pick.Escreverts to the model's choice.?q=links / mobile chooser): arrows move the selection between override buttons, the engine-name display follows, andEnterconfirms via the same delegation path as a tap.?help overlay and README.Test plan
npm run lintclean.🤖 Generated with Claude Code