Why wont this local Script 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?

local frame = script.Parent.Parent:WaitForChild('Frame') script.Parent.MouseButton1Click:connect(function() frame.Visible = not frame.Visible

end)