GUI doesn't appear even when enabled by localscript

Help and Feedback Scripting Support
#1

Hi there, I have this script that toggles a GUI from a click of a button. Problem is, is that the GUI doesn’t actually appear however it shows that it is enabled in the properties. I’ve seen this is as a common issue with StarterGUI, however I’ve tried their solutions like re-defining the variable to local players then playergui however I can’t seem to get them working neither.

I’ll leave the original script here.

local open = true
local gui = game.StarterGui.FPS.Frame
local button = script.Parent.button

button.MouseButton1Click:Connect(function()
	if open == true then
		gui.Visible = true
		open = false
	else
		gui.Visible = false
		open = true
	end
end)
1 Like
#2

Can you provide your attempt at using PlayerGui? Using StarterGui won’t change the player’s current Gui.

#3 
local open = true
local gui = game.Players.LocalPlayer.PlayerGui:WaitForChild("FPS").Frame
local button = script.Parent.button

button.MouseButton1Click:Connect(function()
	if open == true then
		gui.Visible = true
		open = false
	else
		gui.Visible = false
		open = true
	end
end)
#4

When referencing UI objects, you need to use relative hierarchy (script.Parent.Parent -- etc.) instead of accessing it through game. This is because UI objects replicate to each player in the game.

3 Likes
#5

Maybe it’s because PlayerGui didn’t exist yet. Try changing this:

local gui = game.Players.LocalPlayer.PlayerGui:WaitForChild("FPS").Frame

to this:

local PlayerGui = game:GetService("Players").LocalPlayer:WaitForChild("PlayerGui")
local Frame = PlayerGui:WaitForChild("FPS").Frame
#6

I see now. The other guy seemed to work for me but I’ll use this later when I need to organise my scripts really badly.

#7

StarterGui is just a placeholder for all GUI objects that will be used in your game. The UI that is shown on a players screen is called the PlayerGui, and is a seperate copy of StarterGui. Therefore, you’re script won’t work because you’re referencing the wrong thing.

Just switch it to player.PlayerGui or use script.Parent in order to access the players UI

#8

PlayerGui, like LocalPlayer, implicitly exists to the client. WaitForChild on PlayerGui isn’t necessary.

1 Like
#9

That seemed to do the trick. It’s a bit messy but it works.

#10 
local player = game.Players.LocalPlayer
local playergui = player:WaitForChild("PlayerGui")
local gui = playergui.FPS
local buttonframe = script.Parent
local button = buttonframe.button

button.MouseButton1Click:Connect(function()
	gui.Visible = not gui.Visible
end)

Is this cleaner?

1 Like