[SOLVED] How to work GUI's?

[InfDev]

Hey. I'm new to developing with Forge and I've hit a bit of a dead-end. I've looked up quite a few videos on this but can't find one that actually works the best.

 

I'm trying to make a GUI (that's not a container just a GuiScreen) that would show buttons and whatever (haven't got to that part yet) I have imported all of my assets and nothing seems to be working. Here's my code

GuiModules.java (My GUI)

package me.infdev.mod.client.gui;

import org.lwjgl.opengl.GL11;

import net.minecraft.client.gui.GuiScreen;
import net.minecraft.util.ResourceLocation;

public class GuiModules extends GuiScreen {
	
	public GuiModules() {
		System.out.println("test");
	}
	
	@Override
	public void initGui() {
		super.initGui();
	}
	
	@Override
	public void drawScreen(int mouseX, int mouseY, float partialTicks) {		
		System.out.println("draw screen");
		drawDefaultBackground();

		GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
		
		this.mc.getTextureManager().bindTexture(new ResourceLocation("infdev/textures/gui/gui.png"));
		drawTexturedModalRect((this.width - 256) / 2, (this.height - 256) / 2, 0, 0, 256, 256);
		super.drawScreen(mouseX, mouseY, partialTicks);
	}
	
}

 

EventListener.java (where i call the displayGuiScreen())

package me.infdev.mod.client.events;

import me.infdev.mod.client.InfDev;
import me.infdev.mod.client.gui.GuiModules;
import net.minecraft.client.Minecraft;
import net.minecraftforge.client.event.RenderGameOverlayEvent;
import net.minecraftforge.client.event.RenderGameOverlayEvent.ElementType;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
import net.minecraftforge.fml.common.gameevent.PlayerEvent.PlayerLoggedInEvent;

public class EventListener {
	
	@SubscribeEvent
	public void onGameOverlayRender(RenderGameOverlayEvent e) {
		if (e.type == ElementType.TEXT) {
			InfDev.INSTANCE.fr.drawStringWithShadow("§cInfDev DEMO §b{" + InfDev.INSTANCE.MODID + ":" + InfDev.INSTANCE.VERSION + "}", 1, 1, 0xfffff);
		}
	}
	
	// Look here VVVVVV
	
	@SubscribeEvent
	public void onPlayerJoin(PlayerLoggedInEvent e) {
		System.out.println("event fired");
		Minecraft.getMinecraft().displayGuiScreen(new GuiModules());
	}
	
}

I do have a GuiHandler but I'm using displayGuiScreen() so I don't think I need to use it.

 

If you can see the things I've put out to the console, the only one's that worked were

"event fired" and

"test"

 

I am rather new to this so forgive me if I'm doing anything that is considered bad practice within FML or if I just seem like I'm acting dumb. If anybody can help, that'd be appreciated. TIA

Edited August 20, 2019 by InfDev

[Animefan8888]

10 minutes ago, InfDev said:

RenderGameOverlayEvent

You need to specify the type of RenderGameOverlayEvent you want to use. IE RenderGameOverlayEvent.Post or PRE

[InfDev]

Done that, got any other ideas that could make the GUI show? That did nothing (because that was for writing the text on the screen, a different thing I was doing)

[CAS_ual_TY]

You should use EntityPlayer#openGui

 

Im not sure if you want to open it once or have smth render the entire time. If its the first thing then use my mentioned method. 

[Animefan8888]

13 minutes ago, InfDev said:

Done that, got any other ideas that could make the GUI show? That did nothing (because that was for writing the text on the screen, a different thing I was doing)

Oh wrong event then lol. I think PlayerLoggedInEvent is too early to open a GUI so you need to use EntityJoinWorldEvent.

3 minutes ago, CAS_ual_TY said:

You should use EntityPlayer#openGui

This should be used if it has a contnainer. No reason to use it if it doesn't.

[InfDev]

Ok, I will try that. I'll let you know how it goes ?

[InfDev]

Didn't work either. Here's the code if it helps.

@SubscribeEvent
public void onPlayerJoinWorld(EntityJoinWorldEvent e) {
	if (e.entity instanceof EntityPlayer) {
		System.out.println("event fired");
		Minecraft.getMinecraft().displayGuiScreen(new GuiModules());
	}
}

 

One thing though, it called the sysout's twice.

[18:37:51] [Server thread/INFO]: [me.infdev.mod.client.events.EventListener:onPlayerJoinWorld:24]: event fired
[18:37:51] [Server thread/INFO]: [me.infdev.mod.client.gui.GuiModules:<init>:11]: test
[18:37:52] [Client thread/INFO]: [me.infdev.mod.client.events.EventListener:onPlayerJoinWorld:24]: event fired
[18:37:52] [Client thread/INFO]: [me.infdev.mod.client.gui.GuiModules:<init>:11]: test

 

[Animefan8888]

46 minutes ago, InfDev said:

e(new ResourceLocation("infdev/textures/gui/gui.png"));

Oh I'm bad. This isn't a proper ResourceLocation. A resource location expects a modid:textures/path/to/texture.png
At least in this instance.

[CAS_ual_TY]

52 minutes ago, Animefan8888 said:

This should be used if it has a contnainer. No reason to use it if it doesn't.

Pretty sure I read somewhere that you should do it regardless.. But doesnt matter.

 

OP, remember to register this event handler only on client side.

 

Also, dont use GL11.glColor

Use the GlStateManager instead.

[InfDev]

Ok, changed it to this 

this.mc.getTextureManager().bindTexture(new ResourceLocation(InfDev.MODID + "/textures/gui/gui.png"));

 

However, can't see it making a difference because the debug is pretty much saying that the GuiModules class isn't being instantiated for whatever reason and I don't know why. I have registered my events and I'm a bit confused.

Edited August 20, 2019 by InfDev

[InfDev]

Nevermind, all is fixed now. I managed to use threads to delay the show of the GUI by 2 seconds from the initial spawn into the world. Thanks all.