Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
✓ ANSWER ACCEPTED

Button does not work?Please help me , the coding can be wrong and it is in an local script

I have made an Button script but it workes on the roblox studio but when I pay the game and press the button it don't want to open

local open = false script.Parent.MouseButton1Click:connect(function() if frame.Visible == false then frame.Visible = true else frame.Visible = false end

end)

3 comments

0

Is that the whole script? Cause you never defined what your 'frame' variable is.

0

What tiraner300 said, this doesn't appear to be your whole script, unless you just forgot to define what 'frame' was. Also if the click event is only supposed to toggle the frames invisibility, just use 'frame.Visible = not frame.Visible'. That'll just reverse what it currently is.

0

Instead of otherwise adding extra lines to ask what visibility it currently is.

1 answer

✓ Accepted answer

Hello there! I'm BlackOrange and I'm here to help!

To start off, next time use code block.

Anyway, you don't need the variable Open unless your tweening and you could wait until Tween is complete, other then that it's not necessary. To make a nice Frame Open and Close with 1 button just requires a little bit of knowledge.

Let's start off with variables: Note that you need LocalScript

local Button = script.Parent 
local Frame = ??? -- you gotta set where frame is

Now, let's use the MouseButton1Down function:

local Button = script.Parent 
local Frame = ??? -- you gotta set where frame is

Button.MouseButton1Down:Connect(function() -- use :Connect, :connect is deprecated

end)

Now let's give it a command:

local Button = script.Parent 
local Frame = ??? -- you gotta set where frame is

Button.MouseButton1Down:Connect(function() -- use :Connect, :connect is deprecated
	if Frame.Visible == true then
		Frame.Visible = false
	else
		Frame.Visible = true
	end
end)

Now with what we have, it checks the frame's visibility and then changes the visibility based on what it's current value is.

Make sure you define your frame!

Hopefully this helped! Best of luck

-- BlackOrange3343

This discussion is preserved as it was. New answers, comments, and votes are closed.