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

Hitting Escape in a database pw entry closes the database #11199

Open
RJVB opened this issue Aug 26, 2024 · 5 comments · May be fixed by #11221
Open

Hitting Escape in a database pw entry closes the database #11199

RJVB opened this issue Aug 26, 2024 · 5 comments · May be fixed by #11221

Comments

@RJVB
Copy link

RJVB commented Aug 26, 2024

Overview

Hitting escape when a database PW entry widget has the focus/input causes the database to be closed.

Steps to Reproduce

  1. Open one or more databases
  2. click in the PW (or key file) entry widgets
  3. hit the Escape key

Expected Behaviour

Nothing happens, or the entry gets cleared, or the focus is relinquished, anything but what happens now.

Actual Behaviour

The database is closed.

Context

This is confusing as the DB unlock screen looks way too similar between different databases. If you have multiple open that all live in the same directory (so the full paths shown in small font under Unlock KeePassXC Database are hard to tell apart) you can end up trying to "break in" into one of your own DBs, or you end up making modifications to the wrong one.

KeePassXC - 2.7.6
Revision: 9e11923

Operating System: Linux
Desktop Env: KDE
Windowing System: X11

PS: on a similar note, it can be confusing too that Ctrl-W closes the DB and not the window; a shortcut manager feature would be really welcome!

@RJVB RJVB added the bug label Aug 26, 2024
@droidmonkey
Copy link
Member

Ctrl+W is the common shortcut to close a tab, Ctrl+Q is the common shortcut to quit the application. I agree the ESC key is too easy to smack and there is no question as to user intent.

This PR may help you with distinguishing between your databases: #10819

@droidmonkey droidmonkey added the ux label Aug 26, 2024
@droidmonkey droidmonkey self-assigned this Aug 26, 2024
@droidmonkey droidmonkey added this to the v2.8.0 milestone Aug 26, 2024
@RJVB
Copy link
Author

RJVB commented Aug 26, 2024 via email

@WinnetouL
Copy link

Hey @RJVB,
just for my understanding. You open your database, then you click on any entry of the listed entries, so that it has the focus and the corresponding details appear at the bottom. And then you hit ESC and the database closes, so you have to unlock it again? Did I got that correctly?

@RJVB
Copy link
Author

RJVB commented Aug 30, 2024 via email

@WinnetouL
Copy link

ahh thanks I see!

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

Successfully merging a pull request may close this issue.

4 participants
@RJVB @droidmonkey @WinnetouL and others