[1.15] Issues With Button

[Swirls]

Hello there,

 

I added a addWidget method to instance of InventoryScreen but it seems not to work. I don't know what might be causing the issue.

Perhaps its my event handler. 

 

@Mod.EventBusSubscriber(modid = StudyingMod.MOD_ID, bus = Bus.MOD, value = Dist.CLIENT)
public class SurvivalInventory 
{
	
	@SubscribeEvent
	public static void addCustomButtonToInventory(GuiScreenEvent.InitGuiEvent.Pre event)
	{
	  if (event.getGui() instanceof InventoryScreen)
	  {
	    event.addWidget(new Button(8, 12, 6, 3, "test", new CustomAction()));
	  }
	}

	public static class CustomAction implements IPressable {
	  public void onPress(Button button) {

	   }
	}
}

 

If anyone knows what the issue might be, help would be really appreciated.

 

Thanks in forward!

[loordgek]

7 minutes ago, Swirls said:

bus = Bus.MOD

thats the wrong bus, remove it

Edited August 22, 2020 by loordgek

[Swirls]

1 minute ago, loordgek said:

thats the wrong, bus remove it

Should i remove it completely?

[loordgek]

yes

[Draco18s]

(or you can do bus=Bus.FORGE, but there's only two busses, and the Forge bus is the default bus).

 

Bus.Mod is for lifecycle events (FMLPreInit, Registry, etc)

[Swirls]

14 minutes ago, Draco18s said:

(or you can do bus=Bus.FORGE, but there's only two busses, and the Forge bus is the default bus).

 

Bus.Mod is for lifecycle events (FMLPreInit, Registry, etc)

Again, it's not working for some reason. I tried changing x, y of the button but didn't help. At this point i really don't know what to do

[Danebi]

Just now, Swirls said:

Again, it's not working for some reason. I tried changing x, y of the button but didn't help. At this point i really don't know what to do

Can you show your code?

[Swirls]

3 minutes ago, Danebi said:

Can you show your code?

Sure,

This is the code after I removed Bus.MOD and changed x,y,height,width of button

@Mod.EventBusSubscriber(modid = StudyingMod.MOD_ID, bus = Bus.FORGE, value = Dist.CLIENT)
public class SurvivalInventory 
{
	
	@SubscribeEvent
	public static void addCustomButtonToInventory(GuiScreenEvent.InitGuiEvent.Pre event)
	{
	  if (event.getGui() instanceof InventoryScreen)
	  {
	    event.addWidget(new Button(154, 33, 31, 14, "test", new CustomAction()));
	  }
	}

	public static class CustomAction implements IPressable {
	  public void onPress(Button button) {

	   }
	}
}

 

Edited August 22, 2020 by Swirls

[Danebi]

Is the event called?

[Swirls]

1 minute ago, Danebi said:

Is the event called?

Once again, I've been told on discord that event is called with event handler.

[Swirls]

41 minutes ago, Danebi said:

Is the event called?

Isnt the event called with event handler?

[poopoodice]

Use post otherwise the button you added will get removed afterwards.

Edited August 23, 2020 by poopoodice

[Swirls]

1 minute ago, poopoodice said:

Use post otherwise the button you added will get remove afterwards.

Sorry, but i dont understand 

[poopoodice]

Quote

.InitGuiEvent.Pre

use .Post, which is called after all buttons are cleared.

[Swirls]

3 minutes ago, poopoodice said:

use .Post, which is called after all buttons are cleared.

So basically instead of .Pre i would use .Post

[poopoodice]

That's what I think.

[Draco18s]

20 minutes ago, Swirls said:

Once again, I've been told on discord that event is called with event handler.

Isnt the event called with event handler?

Have you confirmed that it is being called? is the question, not where are YOU calling it?

Have you verified that your code, is in fact, being executed.

Use a breakpoint and the debugger.

Edited August 23, 2020 by Draco18s

[Swirls]

13 minutes ago, poopoodice said:

That's what I think.

Its working after changing from .Pre to .Post, thank you very much for your time!