Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Why isn't this working?
When I click it, it won't work :/
1 | function onClick() |
2 | if game.StarterGui.ScreenGui.Frame.Visible ==false then |
3 | game.StarterGui.ScreenGui.Frame.Visible = true |
4 | else |
5 | game.StarterGui.ScreenGui.Frame.Visible = false |
6 | end |
7 | end |
8 | script.Parent.MouseButton1Click:connect(onClick) |
No ideas anymore help would be... well helpful :b
2 answers
Log in to vote
0
Don't use the StarterGui, use this:
1 | game.Players.LocalPlayer.PlayerGui |
Log in to vote
2
Since this is a Gui, you should usually use a LocalScript as it prevents server lag from the Gui, and it is only happening for that Player.
Now, a better way to do this would be to connect the function right away, like so:
1 | script.Parent.MouseButton1Click:connect(function() |
2 | end) |
Doing this can save a line, and a bit of time.
Your next error would be that you are going to the StarterGui instead of the PlayerGui, which can be done as so:
1 | Player = game.Players.LocalPlayer |
2 | PGui = Player.PlayerGui |
Everything else looks fine, good luck!