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

Add ability to customize extension shortcut key #2

Open
hackel opened this issue Sep 22, 2017 · 19 comments
Open

Add ability to customize extension shortcut key #2

hackel opened this issue Sep 22, 2017 · 19 comments

Comments

@hackel
Copy link

hackel commented Sep 22, 2017

I would like the ability to define a custom shortcut key to activate the extension. Perhaps this is something Firefox needs to implement, the same way that Chrome has a page to manage hotkeys for extensions.

@reblws
Copy link
Owner

reblws commented Sep 26, 2017

I don't think this is possible right now, it doesn't look like Firefox has any support for changing extension hotkeys the same way Chrome does. Is there anything particular about the current hotkey that doesn't work for you?

@blakebutcher
Copy link

I'd also like this. My preference would be CMD+E to make it consistent with Vivaldi's Quick Commands feature.

@hackel
Copy link
Author

hackel commented Oct 25, 2017

In terms of why I'd like to change it, I would just like it to be consistent with other editors I use. Also, I just noticed that, at least on GitHub while I have this input focused, ctrl-shift-L inserts the following:


- [ ] 

@reblws
Copy link
Owner

reblws commented Oct 26, 2017

@hackel
Okay, that's pretty bad, I'll change it since I don't want the extension to interfere with anyone's workflow.

I originally chose Ctrl+Shift+L because it was closest to Ctrl+Shift+P (the Sublime Text default, currently opens incognito tabs in FF) and the idea was that those three keys would get your hands ready in a typing position.

I'm most likely going to switch the shortcut to Ctrl+E in the next couple of days, unless there are any objections or problems I find with it.

@ScriptSmith
Copy link

I quite like Ctrl+Shift+A which is the Intellij / Jetbrains* equivalent, however it seems that combination either selects all text, or opens Firefox's addons page depending on context.

I also read through the Mozilla's WebExtension docs and it does seem that the only way to customize shortcuts is to include duplicate entries in the manifest and use multiple command listeners - not really an ideal solution.

I'll try out Ctrl +E when the change is made, and if It doesn't work out I'll probably try out patches with other combinations.

reblws added a commit that referenced this issue Oct 31, 2017
@reblws
Copy link
Owner

reblws commented Oct 31, 2017

After trying Ctrl+E out, I'm concerned that it's way too close to the destructive commands (Ctrl+W - delete current tab, Ctrl+Q close entire window), the cost of missing the shortcut would be too much -- a tired user might accidentally delete their current browser window or close their current tab when all they wanted to do was navigate out of their current tab. Also current users might be confused about the old shortcut suddenly changing, and I can't think of an elegant way of notifying them about it.

Maybe it wouldn't be so bad to have multiple command listeners the way @ScriptSmith mentioned -- I'd define Ctrl+Shift+L available as the default browser action and Ctrl+E as an alternate keybinding that's left on by default but can be turned off in the settings, at least until Mozilla gets around to implementing custom shortcuts.

Here are the relevant bugzillas to watch out for:

https://bugzilla.mozilla.org/show_bug.cgi?id=1320332
https://bugzilla.mozilla.org/show_bug.cgi?id=1303384

@reblws
Copy link
Owner

reblws commented Nov 5, 2017

Ok after fumbling around with the shortcuts, it looks like browserAction.openPopup doesn't actually work if it's under a command listener, so the multiple shortcut idea is dead. I'll be switching the shortcut to Ctrl+Shift+F in the meantime and add a shortcut hint to search placeholder text.

EDIT: After testing Ctrl+Shift+F I'm finding it very nice compared to Ctrl+E. Ctrl+E is also already a default shortcut for activating the search bar in Chrome. I'd like to bring the extension over to the Chrome store when I feel like it's worthy of the $5 Google Tax, so I think the browser neutral shortcut the good way forward.

Sorry about all the indecision.......

@hiasl360
Copy link

hiasl360 commented Nov 6, 2017

Here on macOS the old shortcut "CMD" + "Shift" + "L" worked. The new shortcut "CMD" + "Shift" + "F" switches to fullscreen or window mode.

@thomcc
Copy link

thomcc commented Feb 24, 2018

In nightly it should be possible to implement this, now that bug 1421811 is fixed. There's even a handy library for it (which at the very least can serve as example code).

@reblws
Copy link
Owner

reblws commented Feb 24, 2018

Thanks @thomcc I'm looking into it.

@reblws reblws changed the title Add ability to customize shortcut key Add ability to customize extension shortcut key Mar 30, 2018
@nikitavoloboev
Copy link

So what is the shortcut to trigger Tab Search on macOS? I can't find it anywhere in the settings.

@cwagner22
Copy link

Hey @nikitavoloboev, it's ⌘⇧L but I'm using Karabiner-Elements to remap it for now

@nikitavoloboev
Copy link

Thanks @cwagner22. I tried ⌘⇧L but realized that I had to have the icon in the toolbar for it to work. Now it works. 🙂

@reblws
Copy link
Owner

reblws commented Jun 13, 2018

Hi all, sorry I haven't had the time to update the extension with this helpful feature. It'll be in the next update when I get around to it.

@chodorowicz
Copy link

chodorowicz commented Oct 6, 2018

cmd + shift + L doesn't work for some reason on SoundCloud tabs, I'd be great if it was customizable. FF still doesn't allow to customize shortcuts?

@androidfred
Copy link

@reblws Thanks for this well designed and useful addon!

In OSX there are a bunch of apps that like to hijack cmd + shift + L, and as if that wasn't enough a bunch of websites (eg Slack, Soundcloud) hijack it as well.

I'm commenting to encourage you to release this feature to make it even more useful =)

@fncll
Copy link

fncll commented Jan 25, 2019

Also jumping in to note that it would be extremely helpful to have a customizable shortcut! I should note that a) this is a great extension and b) I am currently using Keyboard Maestro to remap my preferred keyboard shortcut CMD-E to work.

@HaySayCheese
Copy link

HaySayCheese commented Feb 6, 2020

Guys, any plans to provide ability to change Ctrl+Shift+F in Firefox to some other shortcut? Unf. Ctrl+Shift+F collides with Google Docs (channging layout of the interface).

@reblws
Copy link
Owner

reblws commented Feb 13, 2020 via email

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

No branches or pull requests