Why wont this local scrpt work?

I am trying to make a GUI that closes and opens. I am inserting this into the actual button, which would toggle it. It looks like this: >Frame TextLabel

Buttons frame Button Local script All of this is inside of a Screen GUI. If you don't understand the diagram above, please tell me and I will explain it! Also should the Screen GUI be visible when the player joins, or not?

1 local frame = script.Parent.Parent:WaitForChild('Frame') 2 script.Parent.MouseButton1Click:connect(function() 3 frame.Visible = not frame.Visible 4 5 end)