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 a master key option #21

Open
wants to merge 4 commits into
base: dotnet
Choose a base branch
from

Conversation

gq-omistarz
Copy link

Added a master key option which lets you toggle the feeding on/off while in game. Essential for Flight Simulators. Lets you use mouse yoke like in FSX.

  • The mouse position is saved when toggling off and restored when toggling on again. Avoids erratic yoke inputs.
  • Changeable master key.

Seems a bit finicky in MSFS 2020. Doesn't register at first press always. Not sure whats going on. Maybe to much input handling going on in the background.

Oliver Mistarz and others added 2 commits January 15, 2023 14:51
…ile in game. The mouse position is saved when toggling off and restored when toggling on again.
Comment on lines +238 to +239
if (active)
{
Copy link
Owner

Choose a reason for hiding this comment

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

We should probably add a similar conditional around the joystick.SetBtn call in HandleMouseDown. So that the joystick button won't be stuck in a "firing" state if someone uses the "master" toggle.

Copy link
Author

@gq-omistarz gq-omistarz Jan 15, 2023

Choose a reason for hiding this comment

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

Hmm, not sure how that might happen, since active exclusively wraps SetAxis and does nothing else. SetBtn would just continue to work at all times. I guess that's a matter of taste and also depends on the game. In Flight Sims it's more useful if everything else keeps working.

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