Can you help me fix this script?

Help and Feedback Scripting Support
#14

I suggest using a debounce than using the property Visible.

#15

im really suspecting the camera is the prob lel print this again


script.Parent.MouseButton1Click:Connect(function()
    print("Button clicked")
	if script.Parent.Next.Visible == false then
        print("Turning visible")
		script.Parent.Next.Visible = true
	else
        print("Turning invisible")
		workspace.Camera.CameraSubject = game.Players.LocalPlayer.Character
		script.Parent.Next.Visible = false
print("Done")

	end
end)
#16

Thanks ill try this now, ill let you know if “Done” gets printed in the output.

#17

Then the problem is likely with
workspace.Camera.CameraSubject = game.Players.LocalPlayer.Character
Try changing it to
workspace.Camera.CameraSubject =game.Players.LocalPlayer.Character:FindFirstChild("Humanoid")

#18

or Here’s a script using a debounce:

 local debounce = false
 
 script.Parent.MouseButton1Click:Connect(function()
     print("Button clicked")
 	if debounce == false then
         print("Turning visible")
 		script.Parent.Next.Visible = true
 	elseif debounce == true then
         print("Turning invisible")
 		workspace.Camera.CameraSubject = game.Players.LocalPlayer.Character
 		script.Parent.Next.Visible = false
 	end
end)
1 Like
#19

So you are 100% sure that there is not overlapping with open and next? I suspect that the next button is stealing the input.

local Button = script.Parent

Button.MouseButton1Click:Connect(function()
	if not Button.Next.Visible then
		Button.Next.Visible = true
	else
		if Button.Next.Visible then
			workspace.Camera.CameraSubject = game.Players.LocalPlayer.Character
			Button.Next.Visible = false
		end
	end
end)
#20

Thanks ill give this a try now.

#21

Try this:

script.Parent.MouseButton1Click:Connect(function()
	if script.Parent.Next.Visible == false then
		script.Parent.Next.Visible = true
	else
		if script.Parent.Next.Visible == true then
			workspace.CameraType = Enum.CameraType.Scriptable 
			workspace.CurrentCamera.CameraSubject = game.Players.LocalPlayer.Character
			script.Parent.Next.Visible = false
		end
	end
end)
1 Like
#22

WAIT! THAT SCRIPT WON’T WORK. Try this updated one.


script.Parent.MouseButton1Click:Connect(function()
	if script.Parent.Next.Visible == false then
		script.Parent.Next.Visible = true
	else
		if script.Parent.Next.Visible == true then
			workspace.CurrentCamera.CameraType = Enum.CameraType.Scriptable 
			workspace.CurrentCamera.CameraSubject = game.Players.LocalPlayer.Character
			script.Parent.Next.Visible = false
		end
	end
end)
#23

Ill double check now, but im positive there isn’t.

#24

Thanks ill try this one, and ill let you know if it works.

#25

Sadly, this script didn’t work either.

#26

What happen? Did the camera move? Did it error?

#27

I got the output error again “ActivateCameraController did not select a module”.

#28

“ActivateCameraController did not select a module” - Help and Feedback / Scripting Support - DevForum | Roblox

1 Like
#29

Try this:


script.Parent.MouseButton1Click:Connect(function()
	if script.Parent.Next.Visible == false then
		script.Parent.Next.Visible = true
	else
		if script.Parent.Next.Visible == true then
			workspace.CurrentCamera.CameraType = Enum.CameraType.Scriptable 
			workspace.CurrentCamera.CameraSubject = game.Players.LocalPlayer.Character.Humanoid
			script.Parent.Next.Visible = false
		end
	end
end)
#30

Thanks ill try this one, and ill let you know if it works.

#31

Instead of using an else and then later using an if, I would use an elseif. Other than that the code looks decent to me, but I am tired and probably just haven’t been able to notice the mistakes yet.

#32

I think this will work, if it does then mark this as a solution.

script.Parent.MouseButton1Click:Connect(function()
	if script.Parent.Next.Visible == false then
		script.Parent.Next.Visible = true
	elseif script.Parent.Next.Visible == true then
		wait()
		game.Workspace.CurrentCamera.CameraSubject = game.Players.LocalPlayer.Character:WaitForChild("Humanoid")
		script.Parent.Next.Visible = false
	end
end)
2 Likes
#33

Yes, this works thanks for that!

1 Like