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

Map CTRL+SHIFT+M to minimize to tray icon (if enabled) [$25 awarded] #2254

Closed
apfelchips opened this issue Aug 31, 2018 · 6 comments · Fixed by #2402
Closed

Map CTRL+SHIFT+M to minimize to tray icon (if enabled) [$25 awarded] #2254

apfelchips opened this issue Aug 31, 2018 · 6 comments · Fixed by #2402

Comments

@apfelchips
Copy link

apfelchips commented Aug 31, 2018

Expected Behavior

Pressing Ctrl+W should close/minimize to tray the KeepassXC window

Current Behavior

Ctrl+W closes the Database, but keeps the window Open which goes against what most users expect to happen on the usual Window closing Shortcut.

Possible Solution

  • Remap Ctrl+W to close the current window / (minimize to tray as chosen in settings)
  • Remap Locking/Closing the Database to Shift+Meta+L (comp. Meta+L to lock the Screen on Windows)

Context

Give a more consistent UX when using shortcuts.

Debug Info

KeePassXC - 2.3.4

Operating system: ALL
CPU architecture: ALL
Kernel: ALL

@apfelchips apfelchips changed the title Remap Ctrl+W to close window instead of closing the DB Remap Ctrl+W Shortcut to close window instead of closing the DB Aug 31, 2018
@apfelchips
Copy link
Author

I've created a bug bounty on Bountysource for this:
https://www.bountysource.com/issues/62864169-remap-ctrl-w-to-close-window-instead-of-closing-the-db

@droidmonkey
Copy link
Member

I appreciate your enthusiasm, but you are not the decider for what different keyboard shortcuts are mapped to. In almost all tabbed applications, CTRL+W closes the currently open tab (in our case database). CTRL+M minimizes to tray and CTRL+Q quits. This would be best implemented as CTRL+SHIFT+M.

@apfelchips
Copy link
Author

apfelchips commented Aug 31, 2018

Maybe make these shortcuts reassignable then.
For me KeepassXC isn't really a tabbed application, especially if you only ever open one Database. Heck Ctrl+Tab doesn't even do what it's supposed to.

Coming from 1Password this one little shortcut drives me nuts.
Since on macOS Meta+M also doesn't minimize to tray but to a minimized window in the Dock.
(which is the correct behavior btw., I don't want the window to be minimized I want it to close, hence the ctrl+w, Keepass should just keep running in the background/tray so I can autofill and use the browser extension)

@droidmonkey droidmonkey changed the title Remap Ctrl+W Shortcut to close window instead of closing the DB Map CTRL+SHIFT+M to minimize to tray icon (if enabled) Sep 11, 2018
@droidmonkey
Copy link
Member

Well we are a tabbed application, even if you close all tabs the welcome screen comes up. CTRL+TAB being broken is captured in #1333.

@rjekker
Copy link

rjekker commented Oct 5, 2018

I'd like to contribute to the discussion from a different viewpoint: I use the keyboard for almost anything and the fact that there seems to be no way to minimize keepass with a shortcut drives me crazy. I need to grab the mouse and find this little tray icon to minimize the window, which is not user-friendly at all.

@droidmonkey
Copy link
Member

droidmonkey commented Oct 5, 2018

@rjekker what operating system are you using? I am not aware of many applications that have shortcuts to minimize. Most Linux window managers let you define custom keys to raise and lower windows. Either way we have CTRL+M to minimize and I will be adding CTRL+SHIFT+M to minimize to tray. Coming back from that state is a little trickier because it requires a global key shortcut (like auto-type uses).

@droidmonkey droidmonkey changed the title Map CTRL+SHIFT+M to minimize to tray icon (if enabled) Map CTRL+SHIFT+M to minimize to tray icon (if enabled) [$25 awarded] Dec 24, 2019
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 a pull request may close this issue.

3 participants