[1.16.4] Dyeable armor color represent on item

[Aligatetcolt]

Goodday,

 

I am making an armor mod and i want to make some items dyeable just like leather. I already have the dye working on the item models but now i want it also to represent the color on the item. but i cant really figure out how.

 

i saw it had something to do with a item color handler and iitemcolor but i dont realy understand it. does someone maybe have an example on how it should work.

 

this is what i have now and i keep getting a stackoverflow error.

 

@Mod.EventBusSubscriber(modid = ColtCosmeticArmorMod.MOD_ID, bus = Bus.MOD)
public class ModColorHandeler {
	@SubscribeEvent
	@OnlyIn(Dist.CLIENT)
	public static void registerItemColourHandlers(final ColorHandlerEvent.Item event) {
		final BlockColors blockColors = event.getBlockColors();
		final ItemColors itemColors = event.getItemColors();

		final IItemColor itemColourHandler = (stack, tintIndex) -> {
			return itemColors.getColor(stack, tintIndex);
		};
        itemColors.register(itemColourHandler, ccamItems.FLOWER_CROWN.get());
    }
}

 

It is my first time making a minecraft mod. Btw i mostly learn things by example.

Edited March 9, 2021 by Aligatetcolt

[Aligatetcolt]

Alright thanks for the reply.

 

So i deleted the @OnlyIn

 

But as the second part. How would i actually set that color that i get on to the item i want it on. which is in this case the Flower Crown.

[Aligatetcolt]

Thanks alot.

 

just to clarify. I will now have to take that data i got from the script i send before and then set that as nbt data into the item that needs that.  but is that supposed to be in a different script or in the color manager script i send before.

 

 

[Aligatetcolt]

Sorry for my bad habit of naming things. i got used to using unity alot and a new file is called a script.

 

What i meant was, The color data i got from the code snippet earlier. i need to add that then to the nbt data with the ItemStack#getOrCreateChildTag(<your mod ID>) that you send. Right?

 

The only thing i dont really know at the moment is where would i add to the items nbt data. in the place where i register the items or in a new file.

[Aligatetcolt]

Yeah the exact same. just like minecraft does with leather. just put the item in the crafting table with dye and then it is colord. it works on the models but still not in the inventory screen

 

i send an pic of what i have right now as the item. idk if i was clear enough but i want to change the white part to the color that it was dyed.

[Aligatetcolt]

I am not 100%. i have basically been doing everything by trail and error. and i couldn't figure out on how and what i was supposed to do. and i am not sure on how to check if it is the right color because right now it still give that stackoverflow error.

[Aligatetcolt]

public class ModColorHandeler {
	@SubscribeEvent
	public static void registerItemColourHandlers(final ColorHandlerEvent.Item event) {
		final BlockColors blockColors = event.getBlockColors();
		final ItemColors itemColors = event.getItemColors();

		final IItemColor itemColourHandler = (stack, tintIndex) -> {
			return itemColors.getColor(stack, tintIndex);
		};
        itemColors.register(itemColourHandler, ccamItems.FLOWER_CROWN.get());
    }

Here you go. i havent changed much yet because i didn't fully understand what i was supposed to change yet.

[Aligatetcolt]

I got it working. still thanks for your help. it was still realy helpfull

[zero8034]

I'm trying to essentially do the same thing as you have but in 1.18.2 i dont see any of the comments to help with this could ya help me to do what ya did?

Edited Wednesday at 05:51 AM by zero8034
wanted to say the version im working on

[zero8034]

iv done as best i can and gotten to the same stackoverflow error 

[zero8034]

i also finally managed to get it working heres my finished colorHandler that works currently for 1.18.2 and from what i can tell should work post version 1.12

 

//code start

package com.zer0_the_wolf.layeredarmor.item.custom;

import com.zer0_the_wolf.layeredarmor.item.LayeredArmoritems;

import net.minecraft.client.color.item.ItemColor;

import net.minecraft.client.color.item.ItemColors;

import net.minecraft.nbt.CompoundTag;

import net.minecraft.world.item.DyeColor;

import net.minecraft.world.item.DyeableArmorItem;

import net.minecraft.world.item.DyeableLeatherItem;

import net.minecraft.world.item.ItemStack;

import net.minecraftforge.client.event.ColorHandlerEvent;

import net.minecraftforge.eventbus.api.SubscribeEvent;

public class ColorHandlers

{

@SubscribeEvent public static void registerItemColors(ColorHandlerEvent.Item event)

{

event.getItemColors().register(ColorHandlers::getColor,

LayeredArmoritems.LEATHERCHAINHELMET.get(),

LayeredArmoritems.LEATHERCHAINCHESTPLATE.get(),

LayeredArmoritems.LEATHERCHAINLEGGINGS.get(),

LayeredArmoritems.LEATHERCHAINBOOTS.get());

}

private static int getColor(ItemStack pStack, int tintIndex)

{

CompoundTag compoundtag = pStack.getTagElement("display");

if (tintIndex == 0)

{

return compoundtag != null && compoundtag.contains("color", 99) ? compoundtag.getInt("color") : 10511680;

}

return 0xFFFFFF;

}

}

//end code

do note that you have to add a listener should be same place you added a listener for the event bus. mine looks like.

eventBus.addListener(ColorHandlers::registerItemColors);