Skip to content

Conversation

@maimedleech
Copy link

Key up for this extended key code was not working with at least one target app (the game Elite Dangerous). This change makes AddKeyUp work like AddKeyDown, and that fixed the issue.

@Gh0stWalk3r
Copy link
Member

Hello @maimedleech
Thanks a lot for spotting the issue and providing a PR for it.
Unfortunately I can't merge it at the current state as it will change the TargetFrameworks, which is at the moment not intended for this project.
If you are so kind to revert those changes, I am willing to merge it 😄


<PropertyGroup>
<TargetFrameworks>net40;netstandard2.0</TargetFrameworks>
<TargetFrameworks>net5.0;net6.0;netcoreapp3.1</TargetFrameworks>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert changes to TargetFrameworks as it will break compatibility with older applications.

@maimedleech
Copy link
Author

Sorry about that. Replaced with a PR from a different branch.

@maimedleech maimedleech closed this Jan 4, 2022
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