Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Add keyboard shortcut #5089

Closed
ianb opened this issue Oct 29, 2018 · 3 comments
Closed

Add keyboard shortcut #5089

ianb opened this issue Oct 29, 2018 · 3 comments
Milestone

Comments

@ianb
Copy link
Contributor

ianb commented Oct 29, 2018

We want to use Ctrl+Shift+S (or Command+Opt+S on Mac). This previously opened the devtools debugger, but that has been disabled (per this comment).

We'll need to open a Bugzilla ticket to remove this from devtools. We'll have to experiment with whether we can override it in our add-on before they remove it.

See also previous discussion in #2491

@ianb ianb added this to the Sprint 25 (65-2) 🎃 milestone Oct 29, 2018
@jaredhirsch
Copy link
Member

Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1502924 to track the devtools work

@jaredhirsch
Copy link
Member

We'll have to experiment with whether we can override it in our add-on before they remove it.

According to mdn, we're out of luck:

If a key combination is already used by the browser (for example, "Ctrl+P"), or by an existing add-on, then you can't override it. You will be allowed to define it, but your event handler will not be called when the user enters it.

If any other addon tries to use our key combination, they're going to be in for a surprise. Might be good to figure out if/where the Firefox key combinations are documented (MDN? Mozilla wiki?) and make sure we're included in the big list.

@johngruen johngruen modified the milestones: Sprint 25 (65-2) 🎃, Sprint 26 (65-3) Nov 12, 2018
punamdahiya pushed a commit to punamdahiya/screenshots that referenced this issue Nov 30, 2018
TIL Can't specify 'option' as a webextension hot key.
Just go with ctrl + shift + s, which will be command + shift + s on mac.
punamdahiya pushed a commit to punamdahiya/screenshots that referenced this issue Nov 30, 2018
TIL Can't specify 'option' as a webextension hot key.
Just go with ctrl + shift + s, which will be command + shift + s on mac.
@ianb ianb modified the milestones: Sprint 27 (66-1), Firefox 66 Dec 20, 2018
@ianb ianb closed this as completed in dc35fa0 Jan 24, 2019
ianb added a commit that referenced this issue Jan 24, 2019
@Rob--W
Copy link

Rob--W commented May 31, 2019

FYI, users can change shortcuts of their add-ons, via the "Manage Extensions Shortcuts" menu on the gears button at about:addons.

System add-ons such as Screenshots were not shown, but this is going to change in Firefox 69, by https://bugzilla.mozilla.org/show_bug.cgi?id=1553416

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

No branches or pull requests

5 participants