Make sure this is a LocalScript on the client. I recommend using script.Parent…MouseButton1Down, and Visible.
If you send a picture of the way your GUI looks in the explorer, I’ll send you a script that should do what you want
script.Parent.Name.MouseButton1Down:Connect(function() -- make this reference the button
script.Parent.Frame.Visible = true; -- make this reference your frame
end)
Don’t forget, this must be in a local script, preferably in the GUI its self.
Also, make sure it’s in a local script. I agree with sloss, try doing script.Parent instead of referencing it that way. It’s easier and errors are more easily detectable.
The MouseButton1Down event fires when the user presses their left Mouse button down on the GUI object.
If you are looking for an event requiring the user to press and RELEASE their left mouse on a GUI in order for the event to fire, consider using GuiButton.MouseButton1Click .
From what i noticed, the “MouseButton1Down” wants you to HOLD the left mouse button for it to fire.