KGpanels target color script help!

[Soulcleaver]

Hi, im making an ui and i want my target panel to be hostile colored.. aka "faction colored" and class color if its a friendly target :P my script only uses class color but i need help to make it better:P

on load:

self:RegisterEvent("PLAYER_TARGET_CHANGED")

on event:

if UnitExists("target") then
local _, Class = UnitClass("target")
local Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1}
self.bg:SetVertexColor(Color.r, Color.g, Color.b, self.bg:GetAlpha())
end

Any help appriciated

[Fizzlemizz]

Something like:

local Color = { r=1, g=0, b=0 } -- red for any hostile
if UnitExists("target") then
	if not UnitCanAttack("player", "target") then
		local _, Class = UnitClass("target")
		Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1}
	end
	self.bg:SetVertexColor(Color.r, Color.g, Color.b, self.bg:GetAlpha())
else
	self.bg:Hide() -- or set it to some generic colour
end

[Soulcleaver]

Originally Posted by Fizzlemizz Something like: local Color = { r=1, g=0, b=0 } -- red for any hostile if UnitExists("target") then if not UnitCanAttack("player", "target") then local _, Class = UnitClass("target") Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1} end self.bg:SetVertexColor(Color.r, Color.g, Color.b, self.bg:GetAlpha()) else self.bg:Hide() -- or set it to some generic colour end

Thank you, ive tried this and this did not work how i wanted it to work. I got a image here of what happened ingame with this script!

I pasted the script in "onEvent" and kept the same "onLoad" script as my original post, is the mistake there?

http://i.imgur.com/YXpwBBk.jpg

Hope this helps you find the mistake either you or i did :P

[Fizzlemizz]

This might be closer to what you'r looking for, it doesn't account for tapped or dead.

if UnitExists("target") then
	local r, g, b
	local playerFaction, targetFaction = UnitFactionGroup("player"), UnitFactionGroup("target")
	if targetFaction == "Neutral" then -- level less than 20 Panda or 55-58 DK
		-- r, g, b, = 1, 1, 1 -- whatever color works for you
	elseif targetFaction and playerFaction ~= targetFaction then -- enemy faction NPC or player
		r, g, b = 1, 0, 0 -- red
	elseif UnitIsPlayer("target") then -- friendly player
		local _, Class = UnitClass("target")
		local Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1}
		r, g, b = Color.r, Color.g, Color.b
	else -- non "factioned" NPC
		r, g, b = UnitSelectionColor("target") -- your "reaction" to the target (neutral, friendly, die you sob)
	end
	self.bg:SetVertexColor(r, g, b)
else -- target out of range
	self.bg:SetVertexColor(0, 0, 0) -- or hide or...
end

Someone might have better or more definitive solution.

[Soulcleaver]

Originally Posted by Fizzlemizz This might be closer to what you'r looking for, it doesn't account for tapped or dead. if UnitExists("target") then local r, g, b local playerFaction, targetFaction = UnitFactionGroup("player"), UnitFactionGroup("target") if targetFaction == "Neutral" then -- level less than 20 Panda or 55-58 DK -- r, g, b, = 1, 1, 1 -- whatever color works for you elseif targetFaction and playerFaction ~= targetFaction then -- enemy faction NPC or player r, g, b = 1, 0, 0 -- red elseif UnitIsPlayer("target") then -- friendly player local _, Class = UnitClass("target") local Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1} r, g, b = Color.r, Color.g, Color.b else -- non "factioned" NPC r, g, b = UnitSelectionColor("target") -- your "reaction" to the target (neutral, friendly, die you sob) end self.bg:SetVertexColor(r, g, b) else -- target out of range self.bg:SetVertexColor(0, 0, 0) -- or hide or... end Someone might have better or more definitive solution.

http://i.imgur.com/VLZVVue.jpg

Okay this seem to work how i want it to be, but i can't get the colors to match :S

As you can see in the preview, this is what my bars look like, they don't match my hp bars! I've included the colors i want the bars to be on the certain targets like neutral(yellow), green/faction/friendly mob(green) and hostile (red).

I tried to edit the "r,g,b" values i could in the script but they still didn't match the colors..

This is how my script looks like after i edited something, what am i doing wrong?

OnEvent

if UnitExists("target") then
local r, g, b
local playerFaction, targetFaction = UnitFactionGroup("player"), UnitFactionGroup("target")
if targetFaction == "Neutral" then -- level less than 20 Panda or 55-58 DK
-- r, g, b, = 247, 236, 54
elseif targetFaction and playerFaction ~= targetFaction then -- enemy faction NPC or player
r, g, b = 247, 88, 54
elseif UnitIsPlayer("target") then -- friendly player
local _, Class = UnitClass("target")
local Color = RAID_CLASS_COLORS[Class] or {r = 1, g = 1, b = 1}
r, g, b = Color.r, Color.g, Color.b
else -- non "factioned" NPC
r, g, b = UnitSelectionColor("target") -- your "reaction" to the target (neutral, friendly, die you sob)
end
self.bg:SetVertexColor(r, g, b)
else -- target out of range
self.bg:SetVertexColor(0, 0, 0)
end

OnLoad

self:RegisterEvent("PLAYER_TARGET_CHANGED")

[Phanx]

If your goal is just "make this thing the same color as that other thing, always" then you shouldn't be rewriting tons of code -- just grab the color from "that other thing" and apply it to "this thing":

hooksecurefunc(ThatOtherThing, "SetStatusBarColor", function(self, ...)
     ThisThing:SetVertexColor(...)
end)

You should be able to put that in your OnLoad script, as long as "that other thing" already exists when your panel loads.

Also, the above code assumes that "that other thing" is actually a StatusBar object. If it's just a texture, you'd need to hook SetVertexColor instead of SetStatusBarColor.