How to add a gui that opens when you press a button

[Skarte]

so i have been looking for a tutorial for that but everyone is expecting you to have a block/item to show the gui i just want to press a button i allready made the texture and the keybind just need to know how to make a gui not a container also how to add buttons

[Skarte]

it's a gui button

so all i do is just say if(keybind-ispressed) minecraft#displayguiscreen ?

 

 

[Skarte]

  On 11/3/2018 at 7:42 PM, diesieben07 said:

Where do you want this button to be?

In general, a non-inventory GUI is opened by calling Minecraft#displayGuiScreen with a new instance of your GuiScreen.

Expand  

it's a gui button

so all i do is just say if(keybind-ispressed) minecraft#displayguiscreen ?

 

this is my script

 

  Quote

public class MenuGui extends GuiScreen 
{
    private final ResourceLocation texture = new ResourceLocation(Refrence.MODID + ":textures/gui/Menu.png");
    private int menuwidth = 153, menuhight = 152;
   
    
    @Override
    public void drawScreen(int mouseX, int mouseY, float partialTicks) 
    {
        Minecraft.getMinecraft().renderEngine.bindTexture(texture);
        drawTexturedModalRect(0, 0, 0, 0, menuwidth, menuhight);
        super.drawScreen(mouseX, mouseY, partialTicks);
    }
    
    @Override
    public void initGui() 
    {
        super.initGui();
    }
    
    @Override
    protected void actionPerformed(GuiButton button) throws IOException 
    {
        super.actionPerformed(button);
    }
    
    @Override
    protected void keyTyped(char typedChar, int keyCode) throws IOException 
    {
        super.keyTyped(typedChar, keyCode);
    }
    
    @Override
    public boolean doesGuiPauseGame() 
    {
        return false;
    }
}

Expand  

 

Edited November 3, 2018 by Skarte

[Animefan8888]

  On 11/3/2018 at 7:54 PM, Skarte said:

so all i do is just say if(keybind-ispressed) minecraft#displayguiscreen ?

Expand  

Did you try it? Did it work? If not what happened?

[Skarte]

  On 11/3/2018 at 8:14 PM, Animefan8888 said:

Did you try it? Did it work? If not what happened?

Expand  

i allready tried that didn't work so if i had a mod only for this it would be like

the keybinding script

  Quote

    public static KeyBinding button = new KeyBinding("Menu", Keyboard.KEY_M, "something");

    public static void registerKeyBindings() 
    {
        ClientRegistry.registerKeyBinding(button);
    }

Expand  

and the gui script 

  Quote

    {
        if (KeyBindingHandler.button.isPressed()) 
        {
            Minecraft.getMinecraft().displayGuiScreen(new MenuGui());
        }
    }
    
    private final ResourceLocation texture = new ResourceLocation(Refrence.MODID + ":textures/gui/Menu.png");
    private int menuwidth = 153, menuhight = 152;
   
    
    @Override
    public void drawScreen(int mouseX, int mouseY, float partialTicks) 
    {
        Minecraft.getMinecraft().renderEngine.bindTexture(texture);
        drawTexturedModalRect(0, 0, 0, 0, menuwidth, menuhight);
        super.drawScreen(mouseX, mouseY, partialTicks);
    }
    
    @Override
    public void initGui() 
    {
        super.initGui();
    }
    
    @Override
    protected void actionPerformed(GuiButton button) throws IOException 
    {
        super.actionPerformed(button);
    }
    
    @Override
    protected void keyTyped(char typedChar, int keyCode) throws IOException 
    {
        super.keyTyped(typedChar, keyCode);
    }
    
    @Override
    public boolean doesGuiPauseGame() 
    {
        return false;
    }

Expand  

and in the pre init in the main script 

  Quote

     KeyBindingHandler.registerKeyBindings();

Expand  

 

[Animefan8888]

  On 11/3/2018 at 8:28 PM, Skarte said:

if (KeyBindingHandler.button.isPressed()) 
        {
            Minecraft.getMinecraft().displayGuiScreen(new MenuGui());
        }

Expand  

Where is this being called from?

[Skarte]

  On 11/3/2018 at 8:30 PM, Animefan8888 said:

Where is this being called from?

Expand  

the gui script

 

[Animefan8888]

  On 11/3/2018 at 8:36 PM, Skarte said:

the gui script

 

Expand  

From within the gui class? That is never going to be called. You have to use the KeyInputEvent in order to do this.

[Skarte]

  On 11/3/2018 at 8:37 PM, diesieben07 said:

What the hell is that supposed to mean? Java does not have a notion of "scripts". Please provide complete classes, not excerpts. 

Expand  

when i said script i meant class

 

  On 11/3/2018 at 8:37 PM, Animefan8888 said:

From within the gui class? That is never going to be called. You have to use the KeyInputEvent in order to do this.

Expand  

give me an exmaple please

[Animefan8888]

  On 11/3/2018 at 8:41 PM, Skarte said:

give me an exmaple please

Expand  

Here.

[Skarte]

  On 11/3/2018 at 8:42 PM, Animefan8888 said:

Here.

Expand  

like this (still in the gui class)

  Quote

    public class MyStaticForgeEventHandler 
    {
        @SubscribeEvent
        public void buttonGui(KeyInputEvent event) 
        {
            if (KeyBindingHandler.button.isPressed()) 
            {
                Minecraft.getMinecraft().displayGuiScreen(new MenuGui());
            }        
        }
    }

Expand  

 

[Animefan8888]

  On 11/3/2018 at 8:48 PM, Skarte said:

like this (still in the gui class)

Expand  

As long as it is getting registered? Is it working/did you test it?

[Skarte]

  On 11/3/2018 at 8:49 PM, Animefan8888 said:

As long as it is getting registered? Is it working/did you test it?

Expand  

iam sorry if i sound extra stuped it's because if you can't tell iam new to forge but what do you mean by registering it

[Animefan8888]

  On 11/3/2018 at 8:53 PM, Skarte said:

iam sorry if i sound extra stuped it's because if you can't tell iam new to forge but what do you mean by registering it

Expand  

It's in the link I posted, the event needs to be registered to the Event Bus.

[Skarte]

  On 11/3/2018 at 8:55 PM, Animefan8888 said:

It's in the link I posted, the event needs to be registered to the Event Bus.

Expand  

it didn't i think registring it like this

  Quote

         MinecraftForge.EVENT_BUS.register(new MenuGui());
 

Expand  

 

[Animefan8888]

  On 11/3/2018 at 9:06 PM, Skarte said:

it didn't i think registring it like this

Expand  

That is correct as long as that piece of code is being called.

[Skarte]

  On 11/3/2018 at 9:07 PM, Animefan8888 said:

That is correct as long as that piece of code is being called.

Expand  

 

  On 11/3/2018 at 9:08 PM, diesieben07 said:

KeyInputEvent is not the correct event to check for key bindings, it is used for raw LWJGL keyboard input.

To check for key bindings use ClientTickEvent (be sure to check TickEvent#phase or your code runs twice per tick), Minecraft already handles all the low level LWJGL stuff for you.

Expand  

it didn't work

[Skarte]

  On 11/3/2018 at 9:18 PM, diesieben07 said:

Post updated code.

Expand  

this is it also GuiMenu is the gui class

  Quote

    public class MyStaticForgeEventHandler 
    {
        @SubscribeEvent
        public void buttonGui(ClientTickEvent event) 
        {
            if (KeyBindingHandler.DBCR_MENU.isPressed()) 
            {
                Minecraft.getMinecraft().displayGuiScreen(new MenuGui());
            }        
        }
    }

Expand  

also if my code didn't work out is it possible to take my code and make it work out and give it to me but i think that would be asking for too much

[Skarte]

  On 11/3/2018 at 9:26 PM, diesieben07 said:

First of all:

 

Then: Use KeyBinding#isKeyDown, not isPressed.

 

Where is your registration code located? Is the event handler being called (use the debugger)?

Expand  

preinit in main class

[Skarte]

  On 11/3/2018 at 9:30 PM, diesieben07 said:

It would really help if you answered all of my questions and not just one of them.

Expand  

oh sorry but what is tickevent#phase 

and do you accept ?

  On 11/3/2018 at 9:23 PM, Skarte said:

also if my code didn't work out is it possible to take my code and make it work out and give it to me but i think that would be asking for too much

Expand  

 

[Skarte]

  On 11/3/2018 at 9:45 PM, diesieben07 said:

It's a field. Every tick event fires twice per tick, once before the vanilla logic and once after. If you do not check for the value of TickEvent#phase in your event handler your code will run in both of these cases, which is not what you want.

 

No.

Expand  

how do i stop it?

 

[Skarte]

  On 11/3/2018 at 9:50 PM, diesieben07 said:

Do I really need to explain to you how an if statement works?

Expand  

no you don't

just tell me what i need to write in the if statement