Game only detects if mouse is on my first index

[izuchuku]

I'm just messing around, so i'm attempting to make a menu.
I'm using tables to create the buttons, and basically right now I'm just trying to make it so that the game can detect when the mouse hovers over the button.
So to test this out, there is some text in the right corner that tests if mouse hover is true. I tried to do this using a for loop, but it only shows true for the first button and not the rest. I'm fairly new at programming and I can't figure out how to fix it.

[KayleMaster]

You have a typo:

Code: Select all

function newBtn(x,y,text)
	table.insert(menuBtn,{x=x,y=x,text=text,width=100,height=30,mouseover=false}) 
end

Check out y=x, should be y=y.
Also:

Code: Select all

function love.update(dt)
		mousex=love.mouse.getX()
		mousey=love.mouse.getY()
		bool = 'false'
		for i,v in ipairs(menuBtn) do 
			if mousex>=v.x-v.width/4 and mousex<=v.x+v.width and mousey>=v.y and mousey<=v.y+v.height then
				v.mouseover=true
				bool="true"
				print("true")
			else
				v.mouseover=false
			end
		end
end

Removed bool = false from else branch and moved it before the loop. If it was in the else, it means that if one button set bool = 'true' and the next doesn't have mouseover, it will set bool = 'false'. The other way is to put a break where you set bool = 'true'.

To fix your draw now, just remove *i at the v.y at both loops. EDIT: probably also at mousepressed if you want that to work

[izuchuku]

Thanks man! Appreciate the help, works perfectly now.

[KayleMaster]

No problem, welcome to the forums!
I'd change just this one more thing also:

Code: Select all

local menuBtn, btnSpecs = {}, {}
local bool = 'false'
local font

local function newBtn(x,y,text)
	table.insert(menuBtn,{x=x,y=y,text=text,width=100,height=30,mouseover=false}) 
end

function love.load()
	btnSpecs.x=100
	btnSpecs.y=100
	newBtn(100,100,"Start")
	newBtn(100,200,"Options")
	newBtn(100,300,"Quit")
	font=love.graphics.newFont("Segoe UI.ttf",20)
end

function love.update(dt)
		local mousex=love.mouse.getX()
		local mousey=love.mouse.getY()
		bool = 'false'
		for i,v in ipairs(menuBtn) do 
			if mousex>=v.x-v.width/4 and mousex<=v.x+v.width and mousey>=v.y and mousey<=v.y+v.height then
				v.mouseover=true
				bool="true"
				print("true")
			else
				v.mouseover=false
			end
		end
end
function love.mousepressed(x,y,button)
	for i,v in ipairs(menuBtn) do
		if x>=v.x-v.width/4 and x<=v.x+v.width and y>=v.y*i and y<=v.y*i+v.height then		
			print("true")			
		else
			print("false")			
		end
	end
end
function love.draw()
	love.graphics.setFont(font)
	love.graphics.print(bool, 500,500)
	for i,v in ipairs(menuBtn) do
		love.graphics.setColor(25,255,25)
		love.graphics.rectangle('fill',v.x-v.width/4,v.y,v.width,v.height)
	end

	for i,v in ipairs(menuBtn) do
		love.graphics.setColor(255,255,255)
		love.graphics.print(v.text,v.x,v.y)
	end

end

I localised all used variables, because variable declaration without local are by default global (they're located in the _G table).
I also localised the function newBtn, since it's used in this file only. But now love.load accesses newBtn before it's declared, so we have to move it so it's first.
Or you can do something like this:

Code: Select all

...
local newBtn = function() end

function love.load()
	btnSpecs.x=100
	btnSpecs.y=100
	newBtn(100,100,"Start")
	newBtn(100,200,"Options")
	newBtn(100,300,"Quit")
	font=love.graphics.newFont("Segoe UI.ttf",20)
end

function newBtn(x,y,text)
	table.insert(menuBtn,{x=x,y=y,text=text,width=100,height=30,mouseover=false}) 
end
...

But this only works because love.load is called only after the whole main.lua has been loaded, If I were to comment out function love.load() and end it would not error but you'll have no buttons because they were called with the empty function. So basically remember that order is important.