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

fix: update keyboard shortcut to mv3 #2546

Merged
merged 5 commits into from Oct 24, 2023
Merged

Conversation

reneaaron
Copy link
Contributor

Describe the changes you have made in this PR

Action commands seem to have been moved with MV3:

https://developer.chrome.com/docs/extensions/reference/commands/#action-commands

@reneaaron
Copy link
Contributor Author

The keyboard shortcut doesn't work for me in chrome (at least not with the dev build) 🤔

@github-actions
Copy link

github-actions bot commented Jul 7, 2023

🚀 Thanks for the pull request!

Here are the current build files for testing:

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.

Don't forget: keep earning sats!

@bumi
Copy link
Collaborator

bumi commented Jul 10, 2023

does it work with this?

@reneaaron
Copy link
Contributor Author

No, only after setting it manually in the chrome settings:

chrome://extensions/shortcuts

Don't know if that is because this shortcut might already be used.

@pavanjoshi914
Copy link
Contributor

there is a bug in chrome, https://bugs.chromium.org/p/chromium/issues/detail?id=1478453#c_ts1695708024 it works for firefox currently maybe we merge it for now. since it will take time to get this fix from their side

@pavanjoshi914 pavanjoshi914 marked this pull request as ready for review October 24, 2023 05:29
@pavanjoshi914 pavanjoshi914 merged commit 9d9d0b0 into master Oct 24, 2023
5 of 6 checks passed
@pavanjoshi914 pavanjoshi914 deleted the fix/keyboard-command branch October 24, 2023 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants