I am trying to make a over head gui so if someone picks a team the team name will be over there head and on the game leaderboard this is what i got atm
valor = game.StarterGui.Choose.Frame.Valor mystic = game.StarterGui.Choose.Frame.Mystic instinct = game.StarterGui.Choose.Frame.Instinct remove = game.StarterGui.Choose game.Players.PlayerAdded:connect(onPlayerRespawned) function onPlayerRespawned(newPlayer) wait(1) if valor.clickedon then do gui = Instance.new("BillboardGui") gui.Parent=newPlayer.Character.Head gui.Adornee=newPlayer.Character.Head gui.Size=UDim2.new(3,0,2,0) gui.StudsOffset=Vector3.new(0,2,0) words = Instance.new("TextLabel") words.Text = Team Valor words.Position=UDim2.new(-0.125,0,-0.25,0) words.BackgroundTransparency = 1 words.Parent=gui wait(1) valor.Visible = Disable end end end function onPlayerEntered(newPlayer) newPlayer.Changed:connect(function (property) if (property == "Character") then onPlayerRespawned(newPlayer) end end) end game.Players.PlayerAdded:connect(onPlayerEntered)
Please can you fix it c;
Well, you have a rogue event connection on line 7 which is both above the target function (so it errors due to onPlayerRespawned not being a function, as it is nil). Remove that line.
On line 11 you are (I assume) attempting to check if the player has clicked on the object valor
. There are a number of issues with this:
1: There is no clickedon
property. You want to use events.
2: This is going to probably be a server-Script, and you don't get user input from a server-Script (you use a LocalScript for that)
3: valor
is the UI element in the StarterGui, which is not the same as what the players see (they see a clone of the contents of StarterGui)
You also have a syntax error on line 18 (words.Text = Team Valor
). Strings (text) needs to be enclosed in single quotes ('), or double quotes (") or (rarely) double brackets ([[ and ]]). Use words.Text = "Team Valor"
instead.
On line 23 you are attempting to assign Disable
to valor.Visible
. This is a boolean property, so you should pass in either true or false. Also, valor is inside StarterGui
, and as such any modifications to it will only affect players who respawn.
Also, you don't need do
with if
. All you are doing is creating an unnecessary code block. And use Player.CharacterAdded
instead of Player.Changed
for detecting respawning.