-
-
Notifications
You must be signed in to change notification settings - Fork 469
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
feat: Search improvement #5874
feat: Search improvement #5874
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
21b60e3
to
6386cc8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a full review
I like the Hotkeys "Search keybind" functionality
I like that I can CTRL+F to find a hotkey name
I don't like that I can CTRL+F to find the hotkey keybinding
I don't like it searching through accounts/nicknames/highlights/ignoresfilters/logs
I'm torn on searching through commands with this
Closed in favor of #5884 |
This pr adds the functionality of searching for fields on the tables we have in the settings pages. You can for example search for the name you gave a hotkey and it'll show up. Or you can search for "whisper" and find that you have a highlight for it in the highlights tab!
Partially fixes #5872