How to figure out what device the player is currently using

[Kittylitterking123]

Hello!

So I have two GUI labels that show the leaderstats for mobile players. However, my script that detects if the player is on mobile or not doesn’t seem to be working.

local UserInputService = game:WaitForChild("UserInputService")

local IsMobile = UserInputService.TouchEnabled
local IsConsole = UserInputService.GamepadEnabled
local IsComputer = UserInputService.KeyboardEnabled

local Player = game.Players.LocalPlayer
local leaderstats = Player:WaitForChild("leaderstats")
local Time = leaderstats:WaitForChild("Time")
local Age = leaderstats:WaitForChild("Age")

local TimeLabel = script.Parent:WaitForChild("Time")
local AgeLabel = script.Parent:WaitForChild("Age")

repeat task.wait() until game:IsLoaded()

while task.wait(1) do
	TimeLabel.Text = "Time: " .. Time.Value	
	AgeLabel.Text = "Age: " .. Age.Value
end

if IsMobile == true then
	TimeLabel.Visible = true 
	AgeLabel.Visible = true
elseif IsComputer then
	TimeLabel.Visible = false 
	AgeLabel.Visible = false
elseif IsConsole then
	TimeLabel.Visible = false 
	AgeLabel.Visible = false
end

If you think you know what went wrong, please let me know. Thank you and have a wonderful day!

[ItsEGT (EGT)]

Have you checked the output for any errors?

[Kittylitterking123]

There are none. It’s completely empty.

[ItsEGT (EGT)]

Have you tried using print() to make sure the code is even running?

[Kittylitterking123]

I think I fixed it! The script wasn’t getting past the while task.wait() loop, so I moved it to the bottom of the script and it worked! Thanks for the help, though!

[ItsEGT (EGT)]

Ah yeah, didn’t look at that part of the script. You should consider using GetPropertyChangedSignal() to listen for changes in value objects.

Documentation

[Kittylitterking123]

@ItsEGT

You should consider using GetPropertyChangedSignal() to listen for changes in value objects.

How would I use it in this case?

[ItsEGT (EGT)]

Time:GetPropertyChangedSignal("Value"):Connect(function()
	TimeLabel.Text = "Time: " .. Time.Value
end)

Instead of running a while loop every second, you can update the TextLabel right when the value is changed.

[Kittylitterking123]

That IS a much better way! And I think it will save memory! Thanks for the tip!