[SOLVED][1.8][FORGE] Ability to close a gui with a custom key?

[HappyKiller1O1]

So, after getting my custom inventory to work, I find it a tad bit annoying not having the ability to close it with my custom key ®. I have tried sending a packet to close the container (which I believe would tell the client to close the gui also), but the KeyInputEvent is not called while in a gui.

 

Here is my key handler class:

http://pastebin.com/jA9eTDub

 

And my Packet to close the container:

http://pastebin.com/uNNQyzVp

[HappyKiller1O1]

Thank you for that, so, I put that in the keyTyped method in my GUI class. The only problem seems to be that, it will close and re-open. I find this to be that the KeyInputEvent gets called multiple times per press. Is there a way to counter this?

[coolAlias]

KeyInputEvent fires both when the key is pressed and when the key is released; Keyboard#getEventKeyState() will be true if it was pressed, and false if it was released, or you can use the ClientTickEvent as diesieben suggests.

 

And yes, you can put the closing logic in your Gui's #keyTyped method if you want, or in the key handling section. I like to put it in my Gui so that the key doesn't close other screens (you can prevent that by checking the current screen's type, but it's easier to just put it in your Gui).

[HappyKiller1O1]

@diesieben07 Basically, I close the screen in my gui because KeyInputEvent doesn't fire in a gui (I don't know if that is normal, but that's what I found).

 

@coolAlias Thank you! I now check if the custom key is down, and if the current container is not my own before sending the packet to open the gui. Here's an example for those experiencing something similar:

 

int kb = Keyboard.getEventKey();
boolean isDown = Keyboard.getEventKeyState();

if(kb == keys[CUSTOM_INV].getKeyCode() && isDown && !(player.openContainer instanceof YourContainer)) {
System.out.println(player.openContainer);

if(player.openContainer instanceof YourContainer) {
	//Unused		
}else {
        PacketHandler.network.sendToServer(new PacketOpenGui(CUSTOM_INV));
	System.out.println("Packet was sent");
}
}

[HappyKiller1O1]

That's what I did. I posted the fix to the code that was conflicting (reopening the gui after closing it). When I'd attempt to close the gui, I never checked in my KeyHandler if the key was down. So, when the key was released, it would send the packet to reopen the gui. Thus, constantly reopening it after closing it.