-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Docs: add "flip moon texture" into breakage file #12417
Conversation
This argument is moot. We would've had to document the orientation being the same as the sun's as well. To add to the never-ending chain of analogies: Consider XYZW vs WXYZ quaternion order. You have to document which one it is. You can't just say "because we've had to document XYZW we should WXYZ instead because it's obvious". IMO neither orientation is "obvious" or "more intuitive". Both would have to be documented. |
Nope. First, breaking every mod that actually cares about moon rotation does not magically become a good thing if documented. Second, only you seem to think there is a “right” way up. Rotation is pretty much arbitrary; not breaking things is preferable. Third, you already tried this once in #11902 and it was extensively discussed and ultimaltely rejected. So please cut the crap. |
once you have broken everything at least one time, will you consider making the world larger ? :( |
Yes, that should be added to this file Please note that this file is not binding, it's notes about things to consider when doing 6.0. I don't think the moon direction is important, either convention is fine, I just don't want to spend any more time talking about it |
Then close this PR. Simplest way to go about it: No (breaking) change is needed. |
If anyone cares about this when we get to 6.0 then we can have the discussion then. I cba to go through this discussion again for such a tiny issue |
Actually, I'll just unsubscribe from this thread |
I would be fine either way, so if any core dev wants this in I give them my approval. |
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.
<+sfan5> it will be reassesed anyway when the time actually comes so we can just add it to the file
As someone who both makes art and develop, the flipped moon still annoys me (yes, it is flipped if you've had to document it #11940). Considering the "controversial" tag in #11902, this should be an approach that makes everyone happy in the long run
To do
This PR is Ready for Review.
How to test
read