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

Change Speed Shortcut #40

Closed
HalBenHB opened this issue Feb 20, 2023 · 4 comments
Closed

Change Speed Shortcut #40

HalBenHB opened this issue Feb 20, 2023 · 4 comments

Comments

@HalBenHB
Copy link

I'm using your Chrome extension and have a Turkish Q Keyboard. So, I need to press "ğ" and "ü" to change speed. I want to control it via my mouse. I checked your code and what the extension actually waits is KeyCode.219 and KeyCode.221. I want to key these codes to my mouse by AutoHotkey and X-Mouse Button Control, but cannot. I would really like to see a way to remap these keys from your plugin.

@gmertes
Copy link
Owner

gmertes commented Feb 22, 2023

@HalBenHB Thanks for the suggestion, but I don't have any immediate plans to make the keys configurable.

AutoHotkey and X-Mouse Button Control

In principle you should be able to achieve what you want to do with these. Is the problem that you're unable to map the Turkish ğ and ü to a button on the mouse? Are you able to bind the keys [ and ] to a mouse button? Those are the intended keybinds, but the keyCode interface is mapping it to the physical keys rather than the characters (so that correspond to ğ and ü on your keyboard). If you're able to use [ and ] on your mouse, I should be able to fix your problem by using the key character values.

@gmertes
Copy link
Owner

gmertes commented Feb 22, 2023

@HalBenHB

I updated the keyboard event handler so that it now listens to both the physical keys and the character values. So in your case, the playback controller will work with both ğ/ü and [/].

I created a test release here: https://github.com/gmertes/NflxMultiSubs/releases/tag/v2.3.4-alpha

Can you test this for me? Download the zip for chrome or firefox from the release page. You can follow the install instructions here (click on manual install at the bottom). If you bind the keys [ and ] on your mouse it should work now.

@HalBenHB
Copy link
Author

HalBenHB commented Feb 23, 2023

I uninstall current version from my Chrome and installed your test release. It gives this error "Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details."

image
image

However, it works on Netflix by "ğ" and "ü" on my keyboard. Also, it now supports X-Mouse Button Control when I remap "[" and "]" (not ğ and ü)

image

For your notice, I double checked with these programs before uninstall the current version. X-Mouse Button Control didn't work again for both "ğ/ü" or "[ / ]". However, remapping "ğ/ü" worked fine with AutoHotkey.

#IfWinActive Netflix
XButton1::ğ
XButton2::ü

Still, I will continue from your test version because I don't want to run an AutoHotkey script. XMBC is always on and it automatically change its mapping when I'm on a Window called Netflix. So, thanks for your update.

@gmertes
Copy link
Owner

gmertes commented Feb 23, 2023

Glad to hear it works now, and thanks for testing! That manifest error is normal.

I will push this release (v2.3.4) to the store soon, when that's done I suggest to uninstall the test version and install from the Chrome store again so that you get automatic updates in the future.

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

No branches or pull requests

2 participants