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

Fix swapped front/rear view (reverted) #45669

Merged
merged 1 commit into from
Feb 2, 2021
Merged

Fix swapped front/rear view (reverted) #45669

merged 1 commit into from
Feb 2, 2021

Conversation

MrMinimal
Copy link
Contributor

Fixes #45668

@Calinou Calinou added this to the 4.0 milestone Feb 2, 2021
@akien-mga akien-mga requested a review from a team February 2, 2021 21:49
@akien-mga
Copy link
Member

Given the orientation of Left and Right views, I guess it makes sense for Front and Back to be swapped yeah.

Works well for a character set up this way to face forward (Z-):
Screenshot_20210202_230141

@Calinou Calinou added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Feb 2, 2021
@akien-mga akien-mga merged commit 6ae97bf into godotengine:master Feb 2, 2021
@akien-mga
Copy link
Member

akien-mga commented Feb 2, 2021

Thanks! And congrats for your first merged Godot contribution 🎉

@akien-mga
Copy link
Member

Cherry-picked for 3.2.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Feb 3, 2021
@JFonS JFonS mentioned this pull request Mar 22, 2021
@jitspoe
Copy link
Contributor

jitspoe commented Aug 31, 2021

So this does appear to be technically correct, but it's the opposite of how blender behaves. I guess blender is technically wrong, since "front" views the back. My expectation is that hitting 1 on the numpad will result in X being to the right. I wonder if we keep the correct naming but swap the key binds so that numpad 1 becomes the rear view?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Viewport front/rear view swapped
5 participants