Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add setting to disable find-as-you-type. #1303

Merged

Conversation

auouymous
Copy link
Member

The new setting defaults to enabled to preserve the current behavior of opening search fields in channel and episode lists when the user begins typing. Disabling the feature ignores the keypresses and requires ctrl-f or ctrl-shift-f to open or focus the search field.

The new setting defaults to enabled to preserve the current behavior of
opening search fields in channel and episode lists when the user begins
typing. Disabling the feature ignores the keypresses and requires ctrl-f
or ctrl-shift-f to open or focus the search field.
@tpikonen
Copy link
Contributor

How about also adding '/' as a shortcut for searching where the current focus is (either channels or episodes) when type-to-search is disabled? I'm probably going to add this kind of feature to the adaptive branch (vim-keys FTW) and it would be nice to be compatible.

@auouymous
Copy link
Member Author

See comments in #1256. The goal is to have actions the user can bind to keys. This patch is just to disable find-as-you-type until one of us implements the rest. It would even be acceptable to hard-code the keys and add key binding support later.

Copy link
Member

@elelay elelay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@auouymous auouymous merged commit 9bdc9d9 into gpodder:master Nov 9, 2022
@auouymous auouymous deleted the setting-to-disable-find-as-you-type branch November 9, 2022 03:42
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.

None yet

3 participants