fabric icon indicating copy to clipboard operation
fabric copied to clipboard

Water/Foliage Color Modification Not Working

Open AViewFromTheTop opened this issue 1 year ago • 9 comments

Has stopped working for quite a while. Only seems to impact modded biomes.

AViewFromTheTop avatar Mar 15 '24 13:03 AViewFromTheTop

Have you got a minimal way to reproduce this? How are you adding the modded biome?

modmuss50 avatar Mar 16 '24 13:03 modmuss50

Have you got a minimal way to reproduce this? How are you adding the modded biome?

I'm registering them through datagen. We do have a custom FrozenBiome class, but it's the same as how vanilla builds its biomes- just streamlined into separate classes. Screenshot_20240316_211406

2024-03-16_21 15 42

BiomeModifications.create(WilderSharedConstants.id("modify_hot_water")).add( ModificationPhase.REPLACEMENTS, BiomeSelectors.all(), (selectionContext, modificationContext) -> { BiomeModificationContext.EffectsContext context = modificationContext.getEffects(); context.setWaterColor(0); context.setWaterFogColor(0); });

AViewFromTheTop avatar Mar 17 '24 01:03 AViewFromTheTop

Screenshot_20240316_211644 If a screenshot's easier, here it is. It only works on biomes we've added, nothing else.

AViewFromTheTop avatar Mar 17 '24 01:03 AViewFromTheTop

Seems to only impact 1.20.5 versions.

AViewFromTheTop avatar Mar 18 '24 16:03 AViewFromTheTop

I'm having almost the same issue in 1.20.5 release, I call this in my modinitializer but it doesn't change the grass color

        BiomeModifications.create(new Identifier(Bitty.MOD_ID, "birch_overhaul"))
                .add(ModificationPhase.REPLACEMENTS, biomeSelectionContext -> BittyConfig.birchOverhaul && biomeSelectionContext.hasTag(ConventionalBiomeTags.IS_BIRCH_FOREST), context -> {
                    context.getEffects().setGrassColor(10475854);
                });

dayofpi avatar Apr 27 '24 05:04 dayofpi