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

Added MapVirtualKey calls in AddKeyDown and AddKeyUp to fix compatibility issues with some applications. #13

Closed
wants to merge 21 commits into from

Conversation

TChatzigiannakis
Copy link
Collaborator

Sometimes, key up and key down events from InputSimulator are not received by certain applications, unless the Scan field of the KEYBDINPUT struct is set. (Sadly, this happens even if the KEYEVENTF_UNICODE flag is not specified.)

The proper value for this field can be retrieved using the MapVirtualKey native function (from the user32.dll library), which I've declared in the NativeMethods class.

@sharwell
Copy link

sharwell commented Mar 8, 2018

@TChatzigiannakis Are you interested in separating your changes so they can be incorporated back into the root project? This pull request was sent from your master branch so it's picked up all the other changes to added to it after this pull request was originally sent.

@michaelnoonan Are you interested in reviewing proposals and incorporating changes?

@TChatzigiannakis
Copy link
Collaborator Author

TChatzigiannakis commented Mar 8, 2018

@sharwell I absolutely am willing to merge any changes I've made back into the root project. However, we haven't heard from @michaelnoonan at all — especially with respect to the MapVirtualKey issue, which has been asked and even solved by others before. Thus, I've assumed that the original creator has abandoned the maintenance of the root project.

@sharwell
Copy link

sharwell commented Mar 8, 2018

I filed #15 with hopes that it gets seen. 🤞

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