[1.8.9]Mouse not showing up in gui

[YoWazzup]

Hello,

 

I have a gui in my mod that shows me my current skill levels.

There is one problem however. when I open the gui i have no rendered cursor, it only shows up when i move my mouse outside of the minecraft window and move it back in. Is this a known bug? is there a solution?

 

thanks.

[Ernio]

You messed up. Code please.

[YoWazzup]

I'll just send my whole Gui class then, since I have no idea where the error is then.

 

 

public class GuiSkill extends GuiScreen {

private final int imageHeight = 129;
private final int imageWidth = 176;

private static final int SKILL_WIDTH = 64;
private static final int SKILL_HEIGHT = 32;

private static final int MINING_X = 4;
private static final int MINING_Y = 4;

private static final int WOOD_X = 4;
private static final int WOOD_Y = 37;



private static ResourceLocation guiTexture = new ResourceLocation(Reference.MODID + ":textures/gui/gui.png");

private GuiButton buttonDone;

private SkillMining mining;
private SkillWoodcutting woodcutting;

public GuiSkill() {

	mining = new SkillMining("mining");
	woodcutting = new SkillWoodcutting();
}

@Override
public void initGui() {
	buttonList.clear();
	Keyboard.enableRepeatEvents(true);

	buttonDone = new GuiButton(0, width / 2 - (98 / 2), 25 + imageHeight, 98, 20, I18n.format("gui.done", new Object[0]));
	buttonList.add(buttonDone);

}

int levelMining;
int currentExp;

@Override
public void updateScreen() {
	levelMining = mining.getCurrentLevel();
	currentExp = mining.getCurrentXp();
	buttonDone.visible = true;
}

@Override
public void drawScreen(int mouseX, int mouseY, float partialTicks) {
	super.drawScreen(mouseX, mouseY, partialTicks);
	GL11.glColor4f(1, 1, 1, 1);

	mc.getTextureManager().bindTexture(guiTexture);

	int offsetFromScreenLeft = (width - imageWidth) / 2;
	int offsetTopScreen = 20;
	drawTexturedModalRect(offsetFromScreenLeft, 20, 0, 0, imageWidth, imageHeight);

	//DRAWS STRINGS FOR MINING
	this.fontRendererObj.drawString("" + mining.getCurrentLevel(), offsetFromScreenLeft + 40, offsetTopScreen + 10, 0xFEFE00);
	this.fontRendererObj.drawString("" + mining.getCurrentLevel(), offsetFromScreenLeft + 51, offsetTopScreen + 24, 0xFEFE00);

	//DRWAS STRINGS FOR WOODCUTTING
	this.fontRendererObj.drawString("" + woodcutting.getCurrentLevel(), offsetFromScreenLeft + 40, offsetTopScreen + 40, 0xFEFE00);
	this.fontRendererObj.drawString("" + woodcutting.getCurrentLevel(), offsetFromScreenLeft + 51, offsetTopScreen + 54, 0xFEFE00);

	List<String> hoverText = new ArrayList<String>();

	if(isInRect(offsetFromScreenLeft + MINING_X, offsetTopScreen + MINING_Y, SKILL_WIDTH, SKILL_HEIGHT, mouseX, mouseY)) {
		hoverText.add("Mining: " + mining.levelToString());
		hoverText.add("Exp: " + mining.expToString());
		hoverText.add("Left: " + mining.neededToString());


	}
	if(isInRect(offsetFromScreenLeft + WOOD_X, offsetTopScreen + WOOD_Y, SKILL_WIDTH, SKILL_HEIGHT, mouseX, mouseY)) {
		hoverText.add("Woodcutting: " + mining.levelToString());
		hoverText.add("Exp: " + mining.expToString());
		hoverText.add("Left: " + mining.neededToString());


	}

	if(!hoverText.isEmpty()) {
		drawHoveringText(hoverText, mouseX , mouseY , fontRendererObj);
	}

}

public static boolean isInRect(int x, int y, int xSize, int ySize, int mouseX, int mouseY) {
	return ((mouseX >= x && mouseX <= x + xSize) && (mouseY >= y && mouseY <= y + ySize));
}

@Override
protected void mouseClickMove(int mouseX, int mouseY, int clickedMouseButton, long timeSinceLastClick) {
}

@Override
protected void actionPerformed(GuiButton button) throws IOException {
	if (button == buttonDone) {
		mc.displayGuiScreen((GuiScreen) null);
	}
}

@Override
public void onGuiClosed() {
}

@Override
public boolean doesGuiPauseGame() {
	return true;
}

}

 

[Ernio]

I don't think I can help you, I'd need other classes (git?) to test it myself, but I can say that there is no such problem in my 1.8.9 project.

 

Try removing everything from your methods and just leave them with super call. You either do some weird call somewhere else (doubt that) or your OS/workspace is somewhat messed up. Are you using some linux or mac by any chance?

[YoWazzup]

https://github.com/KeesNijenhuis/RuneCraft

[Ernio]

1. Guis are client side only. It is safe to mark them with @SideOnly(Side.CLIENT). That said:

ItemSkillChecker#onItemRightClick has logical mistake:

if(!worldIn.isRemote) { // call on server
		Runecraft.proxy.openGui(); // call to proxy which on dedic does nothing and on client opens GUI.
	}

Problem: while this will "kinda" work on SP (SP is Client.jar with integrated server), it will NOT work on dedicated MP.

This might be source of problem since you are opening gui from server thread.

 

Note: Move your Gui opening to GuiHandler - it is perfectly safe to return null on server and GuiScreen on client while calling player.openGui(...)

 

2. Well, I was expecting something more, but can't find shit. If error persists - resetup your workspace with latest forge and move over your sources. Cleanest way to go about it.

[YoWazzup]

Thanks for the reply.

 

What do you eaxactly mean by moving the gui opening to the guihandler? I can't call player.opgenGui because that requires an x, y and z which I don't have? How do I go about fixing this then?

[Ernio]

GuiHandler methods do not require x/y/z.

 

Pulled from other thread:

IGuiHandler works this way: (when you call player.openGUI)

On Client:

* Calls client method and opens Gui or null.

On Server:

* Calls server method and IF that method returns Container instance then:

** Container is opened on server.

** Packet is sent to client with data: id, x, y, z.

** Client calls client method and opens client Gui.

If the server method will return NULL - nothing happends (on client too).

 

Aditionally I must add: The "packet" that goes from server to client (when container is found) simply containes data: id, x, y, z; and while id is crucial to open proper gui on clinet - x/y/z are never used by internal code - which means that it is literally: "you have 3 ints here, use them if you want, but they can all be just random numbers."

[CatRabbit499]

Mouse.setGrabbed(false);

This tells LWJGL to show the cursor. The cursor is normally hidden (grabbed) when not in a GUI screen.

 

Edit: This works in forge 1.12 and since its LWJGL, it should work in 1.8.9 as well.

Edited July 2, 2017 by CatRabbit499
clarification of version testing

[shadowfacts]

Don't necro threads, this one is more than a year old.