Strange glitch with gui colors

[minecraftbigfoot]

After using this code:

this.drawString(Minecraft.getMinecraft().fontRendererObj, "5", width - 65, 0 + 20,  color);

It also makes my hunger bar go completely white, the color is set to 0xFFFFFF.

 

How do I fix this?

Tried gl11 didnt do much.

 

thanks!

[Jay Avery]

Where do you call that code?

[minecraftbigfoot]

public class GuiClickerCount extends Gui
{
	public GuiClickerCount(Minecraft clientIn)
	{
		ItemStack click = Minecraft.getMinecraft().thePlayer.inventory.getCurrentItem();
		int color = 0xFFFFFF;
		
		if(click != null && click.getItem() == BrandyItems.clickerItem)
		{

			GL11.glPushMatrix();
			{

				ScaledResolution scaled = new ScaledResolution(clientIn);
				int width = scaled.getScaledWidth();
			
				if(GuiClicker.count == "0")
				{
					this.drawString(Minecraft.getMinecraft().fontRendererObj, "No Clicks!", width - 65, 0 + 20,  color);
				}
				else if(GuiClicker.count == null)
				{
					this.drawString(Minecraft.getMinecraft().fontRendererObj, "No Clicks!", width - 65, 0 + 20,  color);
				}
				else
				{
					this.drawString(Minecraft.getMinecraft().fontRendererObj, "Clicks: " + GuiClicker.count, width - 65, 0 + 20,  color);
				}
				
            }
			GL11.glPopMatrix();

		}
		
	}

}

This is the full class, it is called here:

	@SubscribeEvent
	public void onRenderGui(RenderGameOverlayEvent.Post event)
	{
		if (event.getType() != ElementType.EXPERIENCE) 
		{
			new GuiClickerCount(Minecraft.getMinecraft());
		}
	}

 

[minecraftbigfoot]

Just now, diesieben07 said:

You should only draw when the event type is ALL, not when it's anything but EXPERIENCE.

Thank you, Sir.

In case someone views this later, should now be:

 

@SubscribeEvent
	public void onRenderGui(RenderGameOverlayEvent.Post event)
	{
		if (event.getType() == ElementType.ALL) 
		{
			new GuiClickerCount(Minecraft.getMinecraft());
		}
	}

 

[minecraftbigfoot]

Just now, diesieben07 said:

Oh, and as for code style: Drawing things in a constructor is ... well, ... terrible.

Yeah, for my other item I chose to not do it in a constructor, ngl only drawing things in that constructor is because I can't be bothered to move anything aha