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.

Sign up for GitHub

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

Jump to bottom

Feature/ortographic #157

Merged
merged 3 commits into from
Nov 16, 2022
Merged

Feature/ortographic #157

merged 3 commits into from
Nov 16, 2022

Conversation

ThomasJGabrielsen
Copy link
Contributor

No description provided.

Fueredoriku and others added 3 commits March 27, 2022 21:42
@Fueredoriku
Merge pull request #112 from hackerspace-ntnu/develop
70c441e 
NGA release!
@Fueredoriku
Merge pull request #134 from hackerspace-ntnu/develop
0c80c3d 
Release/steam
@ThomasJGabrielsen
Switched camera rendering mode to ortographic
abb76f9 
This change improves the visual layout of the game. Perspective changes in regards to altitude are still visible due to shadow rendering
Fueredoriku
Fueredoriku approved these changes Nov 16, 2022
View reviewed changes
Copy link
Contributor

@Fueredoriku Fueredoriku left a comment

Choose a reason for hiding this comment

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

Very much approve of this.

I personally think it improves the overall game-aesthetic as one can easier see the details of the models.

Additionally, the ortographic view actually makes it easier to see what's happening.

@Fueredoriku Fueredoriku merged commit 068ca6e into develop Nov 16, 2022
@Fueredoriku Fueredoriku deleted the feature/ortographic branch November 16, 2022 21:15
@ddabble
Copy link
Member

ddabble commented Nov 16, 2022

Would be nice to get approval from more members before merging this, as this drastically changes the look and feel of the whole game 😬

@Fueredoriku
Copy link
Contributor

Fueredoriku commented Nov 16, 2022 •
edited
Loading

We have 3 members approving currently, but I do agree you should've also been given time to respond

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fueredoriku Fueredoriku Fueredoriku approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ThomasJGabrielsen @ddabble @Fueredoriku