Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
When I try to remove a GUI it doesn't work and no error message is given, why?
I am trying to make a GUI disappear when I click it but nothing happens. I put print("") to see if it works and I get the first print and not the second one. The StarterGUI is disabled but not the PlayerGUI one. Why?
1 | ImageGui = script.Parent |
2 | OnorOff = game.StarterGui.ScreenGui |
3 |
4 | ImageGui.MouseButton1Click:Connect(function(player) |
5 | print("This.Works") --Prints to see if the code ran through here |
6 | OnorOff.Enabled = false |
7 | game.Players:WaitForChild("player.Name").PlayerGui.ScreenGui.Enabled = false |
8 | print("This.Also.Works") --Prints to see if the code ran through here |
9 | end) |
3 answers
Log in to vote
1
You are removing the GUI but the true GUI is in your player
1 | ImageGui = script.Parent |
2 | --OnorOff = game.StarterGui.ScreenGui -- nope |
3 | OnorOff = game:GetService("Players").LocalPlayer.PlayerGui.ScreenGui -- yes |
4 | ImageGui.MouseButton1Click:Connect(function() |
5 | print("This.Works") --Prints to see if the code ran through here |
6 | OnorOff.Enabled = false |
7 | game:GetService("Players").LocalPlayer.PlayerGui.ScreenGui.Enabled = false |
8 | print("This.Also.Works") --Prints to see if the code ran through here |
9 | end) |
0
Why do we need to put :GetService()? New to the scripting game so confused about this stuff Audiimo 105 — 7y
0
^That's wrong. GetService is basically the WaitForChild for getting a service like Players or StarterGui. PyccknnXakep 1225 — 7y
0
Also, line 7 wasn't needed. PyccknnXakep 1225 — 7y
1
Okie ^ greatneil80 2647 — 7y
Log in to vote
0
First you need to get the player's version of the GUI. This basically means that whatever was in the StarterGui is cloned into the player's PlayerGui, so in your script your not indexing the player's version of the GUI.
1 | local imagegui = script.Parent |
2 | local onoroff = game:GetService("Players").LocalPlayer.PlayerGui:FindFirstChild("ScreenGui") |
3 |
4 | imagegui.MouseButton1Click:Connect(function() --the 'player' parameter isn't needed. |
5 | onoroff.Enabled = not onoroff.Enabled --not reverses something, so if the player clicks the button once, it disables the gui. when the player clicks it again, it enables gui. you can change this to false if you want to though. |
6 | end) |
Hopefully this made sense to you, and if it did please accept my answer!
Log in to vote
-1
On line 7, I don't think you are supposed to put quotation marks around player.Name. So it should look like this:
1 | ImageGui = script.Parent |
2 | OnorOff = game.StarterGui.ScreenGui |
3 |
4 | ImageGui.MouseButton1Click:Connect(function(player) |
5 | print("This.Works") --Prints to see if the code ran through here |
6 | OnorOff.Enabled = false |
7 | game.Players:WaitForChild(player.Name).PlayerGui.ScreenGui.Enabled = false |
8 | print("This.Also.Works") --Prints to see if the code ran through here |
9 | end) |
0
OMG GOOD POINT greatneil80 2647 — 7y
0
yep rilgundam 65 — 7y
0
idk if ur supposed to get the service of players or not rilgundam 65 — 7y
0
it works both ways ^ greatneil80 2647 — 7y
0
When you don't get a service, your code has a change of erroring, since the service has not been defined yet, but unlike Workspace, Workspace is always defined, so you never need to use :GetService for it. hiimgoodpack 2009 — 7y