-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Fix selfie mode movement #1127
base: master
Are you sure you want to change the base?
Fix selfie mode movement #1127
Conversation
Oh a whole file format managed to slip by me. I will look at that soon. |
This comment was marked as outdated.
This comment was marked as outdated.
Maybe it was using different newline characters? |
This comment was marked as outdated.
This comment was marked as outdated.
I got it. |
Co-authored-by: HifiExperiments <[email protected]>
I had just noticed that mouse look is also inverted while in selfie mode. I just snuck in the fix with this pull. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good from a CR perspective, idk if the mouse look thing is a matter of preference for folks?
Currently in Selfie mode (View 2 on the keyboard) if you were to press either "A" or "D" keys with the intention of moving your character, your avatar will be moving in a unexpected way.
This pull adds an additional check to see if the user is in "Selfie Mode" and will move the character in the expected direction.
In effect, this mirrors the current movement direction. So in Selfie Mode you will be moving to the left in the perspective of your character when you press the "D" key.
Fixes #821
Closes #135. Unless I am misreading this issue, this would have been a problem solved a long while ago. What is described is not what I observe.