ClickDetector to GUI Help

Help and Feedback Scripting Support
#1

  1. What do you want to achieve? I want to have a GUI pop up when you click a part.

  2. What is the issue? I got it to work but it only works once. For example, if I click the part once it works. I close out of the gui and click it a second time but it doesn’t work the second time.

  3. What solutions have you tried so far? I have tried looking on the dev forum, and multiple videos but no luck.

Here is some details. I put a click detector in a part, and a script in the click detector. Here is the code I used.

script.Parent.MouseClick:Connect(function(player)
player:WaitForChild(“PlayerGui”).DoorGUI.DoorSystem.Visible = true
end)

Like I said above, it works only once, but when you close out of the gui and try it again, it doesn’t work. Thanks for reading, btw! :slight_smile:

#2

script.Parent.MouseClick:Connect(function(p)
p:WaitForChild(“PlayerGui”).DoorGUI.DoorSystem.Visible = true
end)

script.Parent.Button1Down:Connect(function()
script.Parent.Parent.Visible = false
end)

1 Like
#3

Did you check if your exit button makes it visible or deletes the GUI

1 Like
#4

When you click the x button it makes visible = false

#5

dont waitforchild just do if p.PlayerGui then <code here> end

1 Like
#6

if that doesnt work do if p:FindFirstChild("PlayerGui") then <codehere> end

1 Like
#7

@romeanyguy10 Welcome to the forums!

The script you are using can we improved, this is a local script inside the x button:

local Part = #The part with the ClickDetector 
local ClickDetector = Part.ClickDetector

local GUI = #the GUI to become visible and invisible
local XButton = script.Parent

ClickDetector.MouseClick:Connect(function()
    GUI.Visible = true
end)

XButton.Button1Down:Connect(function()
    GUI.Visible = false
end)

@iSyriux You should utilize waitforchild because there is no guarantee that the playergui has been loaded yet. Please be sure before tou assume things.

#8

It didn’t work when I tried to test. I tried to write “ClickDetector.Clicked” but when I write “Clicked,” it isn’t identifying anything.

1 Like
#9

Oops, that was a typo, it’s MouseClick, not Clicked.

#10

It didn’t work. Again when I typed MouseClick it didn’t identify anything. There were no red lines which was good but when I typed MouseClick it didn’t identify anything. Thanks for helping btw! :smiley:

#11

Did you fill in the missing variables?

#12

Yep I filled them in correctly.

#13

Can you paste your GUI variable for me?

#14

frame = game.StarterGUI.DoorGUI.DoorSystem

#16

Okay! Let me try it real quick…

#18

Just to make things clear. A local script inside of the x button?

#19

I have the same problem, I only got a GUI to pop up once but I could not open it again after closing it.

I hope you find the answear to fix it though! :smiley:

#20

Correct. Also, replace the frame variable with this:

frame = game.Players.LocalPlayer:WaitForChild(“PlayerGui”).DoorGUI.DoorSystem

#21

Didn’t work. When I was typing frame.Visible, it would not even identify frame. Also, when I was writing MouseClick, it didn’t identify that either. Lastly in the output I saw this “- Infinite yield possible on ‘Players.romeanyguy10:WaitForChild(“PlayerGUI”)’” - Could that mean something?

#22

Is the code in a server script or local script?

example2
example2831×297 25.1 KB

Server scripts look like the normal paper, local scripts have the little player icon on them