Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
Why do I have to click it multiple times?
*I've been trying to create a **ScreenGUI ** for when the players connect into my game that tells them some basic info, but for it to close the button has to be pressed multiple times and i'm not sure why!*
I am a beginner and well..I don't know what to do.
01 | --0- |
02 | local WelcomeGUI = true |
03 | local Button = script.Parent |
04 | local GUI = script.Parent.Parent.Parent.Frame |
05 | --0- |
06 |
07 | function CloseIt() --Start of the function |
08 | if WelcomeGUI == true then |
09 | GUI:TweenPosition(UDim2.new(0, 400,0, 200),"Out","Bounce") |
10 | wait(0.15) |
11 | WelcomeGUI = false |
12 | elseif WelcomeGUI == false then |
13 | GUI:TweenPosition(UDim2.new(0, 400,0, -200),"Out","Bounce") |
14 | wait(0.15) |
15 | WelcomeGUI = true |
16 | end |
17 | end |
18 |
19 | Button.MouseButton1Down:connect(CloseIt) |
2 answers
Log in to vote
0
The problem is (I believe) related to your use of waits. Instead of using these waits, I would implement a debounce as it is more reliable:
01 | --0- |
02 | local WelcomeGUI = true |
03 | local Button = script.Parent |
04 | local GUI = script.Parent.Parent.Frame |
05 | local Debounce = true |
06 | --0- |
07 |
08 | function CloseIt() --Start of the function |
09 | if not Debounce then return end -- This will exit the function if debounce is false. |
10 |
11 | Debounce = false |
12 |
13 | if WelcomeGUI == true then |
14 | GUI:TweenPosition(UDim2.new(0, 400,0, 200),"Out","Bounce") |
15 | WelcomeGUI = false |
View all 25 lines...
Alternatively, you can use the override argument, like this:
1 | if WelcomeGUI == true then |
2 | GUI:TweenPosition(UDim2.new(0, 400,0, 200),"Out","Bounce",1,true) |
3 | WelcomeGUI = false |
4 | else |
5 | GUI:TweenPosition(UDim2.new(0, 400,0, -200),"Out","Bounce",1,true) |
6 | WelcomeGUI = true |
7 | end |
This will mean that the player doesn't have to wait for the tween to finish before clicking again.
