Open minhty-fresh opened 1 month ago
Thanks for your PR! We'll take a look at it hopefully this weekend :+1:
Could I suggest instead of "nightNewMoon", having the "night" split into the following types? (You can use CODEC.either(...))
"night": "hex color"
or
"night": {
"full_moon": "hex color",
"new_moon": "hex color"
}
This would keep compatibility with player's resource packs and any biomes that dont need this format.
(and obviously, this transition needs to be configurable, you can definitely enable it by default because it's a great idea to have in vanilla anyways)
Oooh, I really like your suggestion for the night format, let me work on figuring that out, I'm pretty much brand new to minecraft mod dev. I'll also add a config option for enabling the feature.
Okay, I got it working with your new suggested format, and I also added a config option to disable moon phase color transitions. There is a slight bit of messiness caused by the use of Either<String, NightColors>. It seems required to have that as a property in FogColor for serialization, but also we want to be able to manually instantiate FogColor and Either can't be instantiated, so I had to do a bit of a workaround.
https://github.com/IMB11/Fog/issues/48
This pull request adds an optional config option in the fog definition for defining "nightNewMoon" which represents the fog color at night during a new moon. If provided, the fog color at night will transition between "night" and "nightNewMoon" fog color depending on the moon phase.
Here is what it looks like if "/generated/assets/c/fog_definitions/tag/biome/is_desert.json" is changed to
(all other configs untouched)
Other biomes' fog colors are unchanged at night and are not affected by moon phase.