WebMemex / webmemex-extension

📇 Your digital memory extension, as a browser extension
https://webmemex.org
Other
208 stars 45 forks source link

Keyboard shortcuts #140

Closed clmay closed 4 years ago

clmay commented 4 years ago

As a user, I want to be able to use keyboard shortcuts to:

Additionally, on pages that do not support snapshots, I want the "take snapshot" shortcut to have no effect.

I will try to find time to dig into Codetour.md to see how the extension works.

If anyone has thoughts on how to implement this, please let me know. I've always liked the Settings feature for the Firefox extensions Video Speed Controller, would hope to implement something similar.

Treora commented 4 years ago

Fantastic. Note that the manifest already defines two shortcuts:

When I configured this (roughly three years ago, maybe things changed) I found it difficult to get this working well. If the shortcut conflicts with another one, it appears to silently be ignored, and it seemed impossible to know if this is the case, nor to let the user change the shortcut either. I chose the key combinations empirically as these seemed to work in both Firefox and Chromium (at least on Linux).

Now you remind me of its existence, I updated the code tour a bit. :)

Treora commented 4 years ago

At least, we could make the popup put focus on the ‘take snapshot’ button when opened, so one can press Alt+M, Space to take a snapshot.

I implemented this change in commit 8304985. Does this solve your needs?

Closing this issue for now; but feel free to follow up if you have a better solution!