Skip to content

keyToggle and keyTap not working on MacOS when the current language is non-Latin#294

Merged
octalmage merged 1 commit into
octalmage:masterfrom
hristoterezov:master
Mar 3, 2018
Merged

keyToggle and keyTap not working on MacOS when the current language is non-Latin#294
octalmage merged 1 commit into
octalmage:masterfrom
hristoterezov:master

Conversation

@hristoterezov

Copy link
Copy Markdown
Contributor

keyToggle and keyTap are working only with ASCII chars.

The dictionary with chars and the key codes is generated using the current language and if this chars are not ASCII we won't be able to find the corresponding key code that we need for executing keyToggle. This will result in not printing any chars.

With this patch, no matter of the current language the dictionary will be constructed only from ASCII chars. This fixes the issue.

@hristoterezov

Copy link
Copy Markdown
Contributor Author

Hey,

Any news on this PR?

@octalmage

Copy link
Copy Markdown
Owner

I'll take a look this weekend, thanks for the PR!

@octalmage

Copy link
Copy Markdown
Owner

Sorry everyone, releasing and building is currently difficult since I have to spin up a bunch of VMs. I'm going to work on getting TravisCI publishing so it's easier to test and deploy. Thanks for the PR!

@octalmage octalmage merged commit e34f6b1 into octalmage:master Mar 3, 2018
@ghost

ghost commented Sep 6, 2018

Copy link
Copy Markdown

@octalmage
I'm using "robotjs": "^ 0.5.1" after this PR is merged, but Segmantation Fault seems to come out if it is Japanese input source.

It works properly if it is set to "US" input source, but is this PR compatible with Japanese input source?

image

huanvb pushed a commit to huanvb/robotjs that referenced this pull request Jan 13, 2022
huanvb added a commit to huanvb/robotjs that referenced this pull request Jan 13, 2022
Related octalmage#294

Co-authored-by: Vu Ba Huan <vbhuan@cmc.com.vn>
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.

2 participants