elementary / dock

A quick app launcher and window switcher for Pantheon and elementary OS
https://elementary.io
GNU General Public License v3.0
87 stars 24 forks source link

Feature: Show keyboard shortcuts in menu #148

Closed davidak closed 7 months ago

davidak commented 2 years ago

Problem

I accidentally pressed a keyboard shortcut (c) and lost all my work.

Proposal

Other menus show keyboard shortcuts, like the program Photos.

Screenshot from 2021-12-25 19-43-41

To be consistent, they should be displayed here too.

Prior Art (Optional)

No response

jeremypw commented 2 years ago

Which app and which shortcut caused loss of work? Any irreversibly destructive action should have some kind of warning.

davidak commented 2 years ago

@jeremypw i often have 500+ tabs open in Chromium and when i right click on the icon in the dock and press C, it triggers "Close All" which closes every window after another, so when i open the program again, it does not restore all my tabs! Here are more details and a video: https://github.com/elementary/dock/issues/147

@danrabbit is this issue also addressed by the rewrite?

jeremypw commented 2 years ago

@davidak Could you check your settings on Chromium to ensure that the "On Start Up" setting is to continue where you left off? I just tried this and the open tabs were saved even when closed using the context menu on the dock and the mnemonic "C". I agree however that there should be some indication what the shortcuts are on the dock menus. It also is unusual to have an unmodified key as an accelerator. ( I am running EOS7 btw). It seems odd to have keyboard shortcuts anyway as there is no obvious way to operate the dock using the keyboard otherwise.

davidak commented 2 years ago

Could you check your settings on Chromium to ensure that the "On Start Up" setting is to continue where you left off?

yes. it works when i close the browser using it's menu

danirabbit commented 7 months ago

Thanks for your report! We're doing a complete rewrite of the dock based on our recent UI study and this particular issue isn't able to be reproduced in the new version of the dock