Scripting Helpers is winding down operations and is now read-only. More info→

Player List Menu, but nothing shows on the GUI when tested?

Asked by

8 years ago

01local Players = game:GetService("Players")
02Players.PlayerAdded:connect(function(Player)
03    for _,p in next, Players:GetPlayers() do
04        script.Parent.ScrollingFrame:ClearAllChildren()
05        local textButton = Instance.new("TextButton",script.Parent.ScrollingFrame)
06        textButton.BackgroundColor3 = BrickColor.Black().Color
07        textButton.BackgroundTransparency = 0.9
08        textButton.BorderColor3 = BrickColor.White().Color
09        textButton.Positon = UDim2.new(2, 0, 1, 0)
10        textButton.Size = UDim2.new(0, 200, 0, 25)
11        textButton.Font = Enum.Font.ArialBold
12        textButton.FontSize = Enum.FontSize.Size18
13        textButton.Text = p
14        textButton.TextColor3 = BrickColor.White().Color
15        textButton.Archivable = true
16    end
17end)

When run, a textbutton should appear with the players name on it.

Nothing shows up.

Please help.

Please no Remote Events.

1 answer

Answered by

xAtom_ik 574

8 years ago

Very buggy

I'm not sure if you intentionally did this, but your script is littered with errors.

1

You made a typo. Positon

2

Trying to set the text to a player object textButton.Text = p

3

Everytime someone joins, it creates a new text button with the name of the player looking at it. So if I tested it in multiplayer, it would be 'Player1, Player1' and for the other player, 'Player2, Player2' textButton.Text = p.Name

4

You have a very dodgy position there. That is way off the SCREEN! Also, you will need to see which llines are taken and that, etc.

5

Unreadable text. Your background transparency is a bit off. 0.9 makes the white text almost unreadable. I set it to 0.1 because that looks better.

The event

This is where the main issue is. Your PlayerAdded event is not firing. I tried putting it in Workspace and then just tweaked the code, and it worked fine.

Fixed code

01local Players = game:GetService("Players")
02 
03game:GetService("Players").PlayerAdded:connect(function(Player)
04    for _,p in pairs(Players:GetPlayers()) do
05        local textButton = Instance.new("TextButton",p.PlayerGui:WaitForChild("PlayerList").ScrollingFrame) -- Replace PlayerList with the name of your gui.
06        textButton.BackgroundColor3 = BrickColor.Black().Color
07        textButton.BackgroundTransparency = 0.1
08        textButton.BorderColor3 = BrickColor.White().Color
09        textButton.Position = UDim2.new(0, 0, 0, 0)
10        textButton.Size = UDim2.new(0, 200, 0, 25)
11        textButton.Font = Enum.Font.ArialBold
12        textButton.FontSize = Enum.FontSize.Size18
13        textButton.Text = Player.Name
14        textButton.TextColor3 = BrickColor.White().Color
15        textButton.Archivable = true
16    end
17end)

Thanks for reading!

Thanks for reading my answer. I hope it helped you. If it did, don't forget to upvote it or set it as the correct answer!

Think answer helped me alot in getting the right method. thanks. waifuSZN 123 — 8y

Always remember to double check your code. xAtom_ik 574 — 8y