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

Scan codes not set #35

Closed
glitchassassin opened this issue Dec 16, 2016 · 6 comments
Closed

Scan codes not set #35

glitchassassin opened this issue Dec 16, 2016 · 6 comments

Comments

@glitchassassin
Copy link
Collaborator

Certain applications with low-level functionality (like Citrix) check the scan code for keyboard events as well as the virtual key code. If the scan code isn't set, they will miss certain keys like the arrow keys (though other keys will work fine).

I'm running into this issue with the current release of keyboard. I'll submit a pull request in a bit.

@boppreh
Copy link
Owner

boppreh commented Dec 16, 2016

Thanks. Just note that some keys have no scan code (e.g. media keys).

glitchassassin added a commit to glitchassassin/keyboard that referenced this issue Dec 16, 2016
Fix for issue boppreh#35

Adds support for Citrix
@glitchassassin
Copy link
Collaborator Author

In that case the scan code will be set to zero, which is the default currently.

@glitchassassin
Copy link
Collaborator Author

I tried rewriting this to use SendInput instead of the older keybd_event, but for some reason it wasn't working so I just modified keybd_event to include the scan code. It seems to be working now in my tests.

@boppreh
Copy link
Owner

boppreh commented Dec 16, 2016

Did it fix your Citrix issue?

@glitchassassin
Copy link
Collaborator Author

Yep!

@boppreh
Copy link
Owner

boppreh commented Dec 20, 2016

@glitchassassin if you don't mind, I'm closing the issue. If you still experience problems please don't hesitate to reopen. And thanks again for the pull request.

@boppreh boppreh closed this as completed Dec 20, 2016
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