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

Can't trigger extension using keyboard shortcut in Firefox #48

Closed
husimo-tilleuls opened this issue Dec 20, 2022 · 16 comments
Closed

Can't trigger extension using keyboard shortcut in Firefox #48

husimo-tilleuls opened this issue Dec 20, 2022 · 16 comments
Labels
enhancement New feature or request
Milestone

Comments

@husimo-tilleuls
Copy link

Expected behaviour

Would like to be able to use the QOwnNotes web companion using keyboard shortcut Ctrl + Shift + B

Actual behaviour

It triggers the bookmarks bar (displays and hides).

Steps to reproduce

Maybe it's related to a French Firefox version, i dunno.

Output from the debug section in the settings dialog in QOwnNotes

Don't think it's relevant here but can send it if necessary

Relevant log output in the Log panel in QOwnNotes

Don't think it's relevant here but can send it if necessary

@husimo-tilleuls husimo-tilleuls changed the title Can't trigger extension using keyboard shorcut in Firefox Can't trigger extension using keyboard shortcut in Firefox Dec 20, 2022
@pbek
Copy link
Member

pbek commented Dec 20, 2022

I guess (your?) Firefox doesn't honor the default shortcut set here:

"__firefox__commands": {
"_execute_browser_action": {
"suggested_key": { "default": "Ctrl+Shift+B" }
}
}

@pbek
Copy link
Member

pbek commented Dec 20, 2022

I personally use Ctrl+Alt+B everywhere. Maybe I just set that as default too now...

@pbek pbek added the enhancement New feature or request label Dec 20, 2022
@husimo-tilleuls
Copy link
Author

Indeed : https://support.mozilla.org/en-US/questions/1317877

Ctrl+Shift+B now toggles the Bookmarks Toolbar.

Do you think it would be possible to implement other shortcuts like Chrome ?

@pbek
Copy link
Member

pbek commented Dec 20, 2022

What "other shortcuts like Chrome"?

@husimo-tilleuls
Copy link
Author

As listed here : https://github.com/qownnotes/web-companion#chrome ; there's only one listed for Firefox versus more for Chrome

@pbek
Copy link
Member

pbek commented Dec 20, 2022

Those just use HTML Access Keys, they should also work under Firefox, see https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/accesskey. Thank you, I'll add the link to the README.

@husimo-tilleuls
Copy link
Author

Okay, nice. I tried it the first one : https://github.com/qownnotes/web-companion/blob/develop/README.md?plain=1#L57

And Firefox triggers the menu bar on Alt key ;'(

@pbek pbek added this to the 22.12.0 milestone Dec 20, 2022
@pbek
Copy link
Member

pbek commented Dec 20, 2022

I personally use Ctrl+Alt+B everywhere.

You can set that shortcut too in the Firefox extension settings, those are well hidden in Firefox. 😬

image

@pbek
Copy link
Member

pbek commented Dec 20, 2022

And Firefox triggers the menu bar on Alt key ;'(

https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/accesskey says Alt + Shift (on Windows and Linux).

pbek added a commit that referenced this issue Dec 20, 2022
@husimo-tilleuls
Copy link
Author

Nice, it's working, thank you!

@pbek
Copy link
Member

pbek commented Dec 20, 2022

22.12.0

  • changed Firefox default shortcut to Ctrl + Alt + B since the browser blocked Ctrl + Shift + B
    (for #48)

There now is a new release, could you please test it and report if it works for you?

@husimo-tilleuls
Copy link
Author

Hello, thank you! It's not yet available in my Firefox and I don't have time right now to install it manually but I'll tell you ASAP ;-)

@pbek
Copy link
Member

pbek commented Dec 21, 2022

Mozilla needs some time for the review process... ;)

@pbek
Copy link
Member

pbek commented Dec 21, 2022

It's online...

@husimo-tilleuls
Copy link
Author

LGTM! Thank's a lot!

If there's any way to support your work I would be glad to!

@pbek
Copy link
Member

pbek commented Dec 22, 2022

Thank you. :) Reporting issues that creep up when browser vendors change things already helps a lot.

pbek added a commit to pbek/QOwnNotes that referenced this issue Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants