ssborbis / ContextSearch-web-ext

Search engine manager for modern browsers
329 stars 37 forks source link

[Request] Add shortcut key for menu item "Add to ContextSearch" #529

Closed wordpure closed 1 year ago

wordpure commented 2 years ago

1.42 Added shortcut keys for search-related menu items, but the menu item "Add to ContextSearch" of the search box does not have it. I hope to add one to it.

图片

ssborbis commented 2 years ago

Got it. Changes will be in the next release.

ssborbis commented 2 years ago

Fix should be live in 1.42.1+

wordpure commented 2 years ago

Great work.

But the default shortcut &C of ContextSearch is duplicated with the browser menu item - Copy (&C), we have to press &C twice to trigger the menu item of ContextSearch.

图片

图片

Is it possible to switch to a key that is easier to click but less likely to conflict with the existing menu items, for example, &A?

图片

ssborbis commented 2 years ago

Is it possible to switch to a key that is easier to click but less likely to conflict with the existing menu items, for example, &A?

I think you're right in that the default should be something other than a character already used in the default menu. Since the user can change it, it's not exactly a big deal, but I'll swap it for Z maybe.

(edit) Since the same key reused by this addon for multiple menus, I figure something generic like Z would be more appropriate than A which seems to imply Add to

wordpure commented 2 years ago

Yes, Z does work better than A as the default shortcut. I switched to A as the trigger key for Context Search because another extension was already using Z.