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

FF7: Fix Bahamut Zero and Supernova not displaying correctly with lighting #603

Merged

Conversation

CosmosXIII
Copy link
Contributor

@CosmosXIII CosmosXIII commented Sep 16, 2023

Summary

Fixed Bahamut Zero and Supernova not displaying correctly when lighting enabled. The lighting shader assumes an input with valid normals. So for any model that has no valid normals I changed it to fallback to the standard shader.

Motivation

Because it's time to have a bug-free lighting mode from start to finish :)

ACKs

  • I have updated the Changelog.md file
  • I did test my code on FF7
  • I did test my code on FF8

@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Sep 16, 2023
@julianxhokaxhiu julianxhokaxhiu added this to the 1.17.0 milestone Sep 16, 2023
@julianxhokaxhiu julianxhokaxhiu changed the title Fixed Bahamut Zero and Supernova not displaying correctly when lighting enabled Fix Bahamut Zero and Supernova not displaying correctly with lighting Sep 16, 2023
Copy link
Owner

@julianxhokaxhiu julianxhokaxhiu left a comment

Choose a reason for hiding this comment

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

I rephrased a little the Changelog, if it's fine by you I'm ok merging it :) Let me know and thanks!

@julianxhokaxhiu julianxhokaxhiu changed the title Fix Bahamut Zero and Supernova not displaying correctly with lighting FF7: Fix Bahamut Zero and Supernova not displaying correctly with lighting Sep 16, 2023
@CosmosXIII
Copy link
Contributor Author

I rephrased a little the Changelog, if it's fine by you I'm ok merging it :) Let me know and thanks!

Sure, no problem. Thanks!

@julianxhokaxhiu julianxhokaxhiu merged commit 791daac into julianxhokaxhiu:master Sep 16, 2023
1 check passed
@julianxhokaxhiu
Copy link
Owner

Thank you!

@julianxhokaxhiu julianxhokaxhiu added bug Something isn't working and removed enhancement New feature or request labels Sep 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants