Jump to content

[1.7.2][Solved]Cannot Close GUI


SoBiohazardous

Recommended Posts

Hello all,

So i have a gui that opens when you press a keybind, and it's supposed to close when you hit the keybind again. However, the GUI never closes. Here is my code:

 

KeyHandler

package sobiohazardous.dashboard.handler;

import org.lwjgl.input.Keyboard;

import sobiohazardous.dashboard.client.gui.GuiOverlay;
import cpw.mods.fml.client.FMLClientHandler;
import cpw.mods.fml.client.registry.ClientRegistry;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.InputEvent.KeyInputEvent;
import net.minecraft.client.Minecraft;
import net.minecraft.client.gui.GuiChat;
import net.minecraft.client.settings.KeyBinding;

public class DKeyHandler
{
/** Key index for easy handling */
public static final int CUSTOM_INV = 0;
/** Key descriptions; use a language file to localize the description later */
private static final String[] desc = { "key.tut_inventory.desc" };
/** Default key values */
private static final int[] keyValues = { Keyboard.KEY_Y };
private final KeyBinding[] keys;

public boolean isOn = false;


public DKeyHandler()
{
	keys = new KeyBinding[desc.length];
	for (int i = 0; i < desc.length; ++i)
	{
		keys[i] = new KeyBinding(desc[i], keyValues[i], "key.tutorial.category");
		ClientRegistry.registerKeyBinding(keys[i]);
	}
}

/**
 * KeyInputEvent is in the FML package, so we must register to the FML event
 * bus
 */
@SubscribeEvent
public void onKeyInput(KeyInputEvent event)
{
	// FMLClientHandler.instance().getClient().inGameHasFocus
	if (!FMLClientHandler.instance().isGUIOpen(GuiChat.class))
	{
		if (keys[CUSTOM_INV].isPressed() && !isOn)
		{
			//TutorialMain.packetPipeline.sendToServer(new OpenGuiPacket(TutorialMain.GUI_CUSTOM_INV));
			isOn = true;
		}		

		else if(keys[CUSTOM_INV].isPressed() && isOn)
		{
			isOn = false;
		}			

		else
		{
			isOn = false;
		}
	}
}
}

 

OverlayHandler (Basically an FML Event handler)

package sobiohazardous.dashboard.handler;

import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;

import org.lwjgl.opengl.GL11;

import sobiohazardous.dashboard.Dashboard;
import sobiohazardous.dashboard.client.gui.GuiOverlay;
import sobiohazardous.dashboard.lib.DReference;
import sobiohazardous.dashboard.util.DUtil;
import net.minecraft.client.Minecraft;
import net.minecraft.client.gui.FontRenderer;
import net.minecraft.client.gui.GuiIngame;
import net.minecraft.client.gui.GuiScreen;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.event.RenderGameOverlayEvent;
import cpw.mods.fml.client.FMLClientHandler;
import cpw.mods.fml.common.FMLCommonHandler;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;

public class DashboardOverlayHandler {

private Minecraft mc;

public DashboardOverlayHandler(Minecraft mc)
{
	this.mc = mc;
}

@SubscribeEvent
public void renderGameOverlay(RenderGameOverlayEvent.Pre event)
{		
	if(Dashboard.keyHandler.isOn && !FMLClientHandler.instance().isGUIOpen(GuiScreen.class))
	{
		mc.displayGuiScreen(new GuiOverlay());
	}
	if(!Dashboard.keyHandler.isOn && FMLClientHandler.instance().isGUIOpen(GuiOverlay.class))
	{
		mc.displayGuiScreen((GuiScreen)null);
	}
	System.out.println(Dashboard.keyHandler.isOn);
}

}

 

Thanks in advance.

Link to comment
Share on other sites

In 1.7, you can no longer close GUIs directly from the KeyBinding class, you have to do it in the GUI itself:

@Override
protected void keyTyped(char c, int keyCode) {
super.keyTyped(c, keyCode);
// 1 is the Esc key, and we made our keybinding array public and static so we can access it here
if (c == 1 || keyCode == KeyHandler.keys[KeyHandler.CUSTOM_INV].getKeyCode()) {
mc.thePlayer.closeScreen();
}
}

Link to comment
Share on other sites

In 1.7, you can no longer close GUIs directly from the KeyBinding class, you have to do it in the GUI itself:

@Override
protected void keyTyped(char c, int keyCode) {
super.keyTyped(c, keyCode);
// 1 is the Esc key, and we made our keybinding array public and static so we can access it here
if (c == 1 || keyCode == KeyHandler.keys[KeyHandler.CUSTOM_INV].getKeyCode()) {
mc.thePlayer.closeScreen();
}
}

It seemed it doesn't work. However, when I press the keybind, the GUI closing name plays, the mouse snaps to the middle, but it never closes.

Link to comment
Share on other sites

The problem is you open another GUI immediately due to the way you are setting your keys / isOn. You shouldn't need to have the renderOverlayEvent at all - just open the GUI when the key is pressed.

 

Btw, KeyBinding#isPressed() will still return true even after the key has been released for a certain amount of time; you want KeyBinding.getIsKeyPressed(), but you shouldn't even need that:

@SubscribeEvent
public void onKeyInput(KeyInputEvent event) {
// gets you the key binding key code, if you want it
int kb = Keyboard.getEventKey();

// Keyboard.getEventKeyState() returns true if the key is down, i.e. pressed
// you can either use that in combination with the kb, or just use getIsKeyPressed
// I've got Minecraft instance stored in a local variable from my constructor

if (mc.inGameHasFocus) {
// alternatively: if (kb == keys[CUSTOM_INV].keyCode)
if (keys[CUSTOM_INV].getIsKeyPressed()) {
// sending an open GUI packet to server will also open the GUI client-side
yourPacketDispatcher.sendToServer(new OpenGuiPacket(yourGuiID));
}
}

 

The way you've got yours set up is quite unnecessary and not very functional, as you've just experienced.

Link to comment
Share on other sites

The problem is you open another GUI immediately due to the way you are setting your keys / isOn. You shouldn't need to have the renderOverlayEvent at all - just open the GUI when the key is pressed.

 

Btw, KeyBinding#isPressed() will still return true even after the key has been released for a certain amount of time; you want KeyBinding.getIsKeyPressed(), but you shouldn't even need that:

@SubscribeEvent
public void onKeyInput(KeyInputEvent event) {
// gets you the key binding key code, if you want it
int kb = Keyboard.getEventKey();

// Keyboard.getEventKeyState() returns true if the key is down, i.e. pressed
// you can either use that in combination with the kb, or just use getIsKeyPressed
// I've got Minecraft instance stored in a local variable from my constructor

if (mc.inGameHasFocus) {
// alternatively: if (kb == keys[CUSTOM_INV].keyCode)
if (keys[CUSTOM_INV].getIsKeyPressed()) {
// sending an open GUI packet to server will also open the GUI client-side
yourPacketDispatcher.sendToServer(new OpenGuiPacket(yourGuiID));
}
}

 

The way you've got yours set up is quite unnecessary and not very functional, as you've just experienced.

 

Works great, thanks!

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.