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

Re-allow entry keyboard shortcuts from search box #2850

Closed
DuBistKomisch opened this issue Mar 24, 2019 · 2 comments
Closed

Re-allow entry keyboard shortcuts from search box #2850

DuBistKomisch opened this issue Mar 24, 2019 · 2 comments

Comments

@DuBistKomisch
Copy link

Summary

This is explicitly broken in #2431:

Entry specific toolbar and menu items are disabled unless the entry row has focus (prevents unintended actions)

I love all the other changes in this PR, but previously I could open KeePassXC, log in, ctrl+f and start typing, then start ctrl+b, ctrl+c etc without ever having to touch the mouse. Since upgrading to 2.4.0 this is no longer possible. Can't even seem to mash tab to make it work.

Desired Behavior

Entry keyboard shortcuts should work on the highlighted entry in search results.

Possible Solution

Something like whitelist the search box being focused in addition to the entry itself, instead of just blanket disabling everything. Or at least add an option to disable this new "safe" behaviour.

Context

Used to work, broke my workflow! https://xkcd.com/1172/
Should generally be possible to use KeePassXC without needing the mouse.

@droidmonkey
Copy link
Member

droidmonkey commented Mar 24, 2019

Press down arrow before your normal shortcuts. That explicitly selects the first entry in the search results.

@DuBistKomisch
Copy link
Author

Yep, that seems to work I guess, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants