Skip to content

Add type-ahead search handling#216

Merged
SUPERCILEX merged 3 commits into
SUPERCILEX:masterfrom
Aditya190803:feature/type-ahead-search
Apr 21, 2026
Merged

Add type-ahead search handling#216
SUPERCILEX merged 3 commits into
SUPERCILEX:masterfrom
Aditya190803:feature/type-ahead-search

Conversation

@Aditya190803

@Aditya190803 Aditya190803 commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Adds type-ahead search to the clipboard menu: when the search field is not focused and the user starts typing, focus moves to search and the typed characters are inserted immediately. This also keeps the existing keyboard shortcuts working by returning handled key events consistently.

@SUPERCILEX

Copy link
Copy Markdown
Owner

Hmmmm, I'm worried about consistency. I like that you've made sure this can't interfere with other shortcuts, but what if we wanted to add a shortcut like Shift+F to favorite an entry? Then suddenly you can't type F to focus the search which is why I went with an explicit focus shortcut.

Is there a reason the / shortcut is annoying to use?

@Aditya190803

Copy link
Copy Markdown
Contributor Author

I get your point about consistency.

I added this mainly for speed since I often start typing instinctively after remembering a keyword, and the extra / step breaks that flow.

Happy to adjust if you’d prefer keeping it strictly shortcut-driven or gating this behind a setting.

@SUPERCILEX

Copy link
Copy Markdown
Owner

gating this behind a setting

Yeah, I think an off-by-default setting would work for me.

@Aditya190803

Copy link
Copy Markdown
Contributor Author

@SUPERCILEX
I have updated it.

@SUPERCILEX

Copy link
Copy Markdown
Owner

Sorry, did you forget to push? I don't see any changes besides the merge from master.

@Aditya190803

Copy link
Copy Markdown
Contributor Author

Sorry, I pushed it to a different local branch and commented without verifying first.

@SUPERCILEX SUPERCILEX merged commit 1d22d3d into SUPERCILEX:master Apr 21, 2026
1 check passed
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.

2 participants