Why doesnt this Local Script work?

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

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?