✓ 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 = ???
Now, let's use the MouseButton1Down function:
local Button = script.Parent
local Frame = ???
Button.MouseButton1Down:Connect(function()
end)
Now let's give it a command:
local Button = script.Parent
local Frame = ???
Button.MouseButton1Down:Connect(function()
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
3 comments
Is that the whole script? Cause you never defined what your 'frame' variable is.
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.
Instead of otherwise adding extra lines to ask what visibility it currently is.