Fading Player Unit Frames

[JDoubleU00]

I'd like to fade out the player unit frames when not in combat or target another unit. oUF_Fader is no longer being developed. The latest Kong UI Hider does not see any frames for oUF. Any suggestions? I've been using oUF_Phanx and oUF_Adirelle.

[zork]

--Variables
local frameList = {"frameName1","frameName2","frameName3"} --add the name of your frames here
local addon     = CreateFrame("Frame")
local _G        = _G
 
--ChangeFrameAlpha func
local function ChangeFrameAlpha(a)
  for _, frameName in next, frameList do
    if _G[frameName] then
      _G[frameName]:SetAlpha(a)
    end
  end
end
 
--OnEvent func
local function OnEvent(self,event)
  if InCombatLockdown() then
    ChangeFrameAlpha(1)
  else
    ChangeFrameAlpha(0.5)
  end
end
 
--RegisterEvent
frame:RegisterEvent("PLAYER_REGEN_ENABLED")
frame:RegisterEvent("PLAYER_REGEN_DISABLED")
frame:RegisterEvent("PLAYER_ENTERING_WORLD")
 
--SetScript
frame:SetScript("OnEvent", OnEvent)

If your oUF layouts do not generate a global frame name you can edit them to make one.

Additionally you can make the above work without using _G. You can do this by adding your unitframes by reference to the frameList table.

local addonName, addonTabe = ...
addonTable.frameList = addonTable.frameList or {}
 
--ChangeFrameAlpha func
local function ChangeFrameAlpha(a)
  for _, frame in next, addonTable.frameList do
    frame:SetAlpha(a)
  end
end
 
--inside of the unit creation (factory or template function)
tinsert(addonTable.frameList,self)

[Phanx]

You don't need global names at all... just iterate over oUF's internal list of frames:

for _, frame in next, oUF.objects do
    frame:SetAlpha(0.5)
end

Also, InCombatLockdown() is always false inside the PLAYER_REGEN_DISABLED event, since the purpose (or one of the purposes) of that event is to notify addons that combat is beginning so they can make changes for combat. Lockdown doesn't take effect until the event is finished being handled. Use UnitAffectingCombat("player") instead.

Also also, there's really no need to factor that out into multiple independent functions when there is only one chain of execution. Function calls are slow. Avoid them whenever posisble.

local f = CreateFrame("Frame")
f:RegisterEvent("PLAYER_ENTERING_WORLD")
f:RegisterEvent("PLAYER_REGEN_DISABLED")
f:RegisterEvent("PLAYER_REGEN_ENABLED")
f:SetScript("OnEvent", function()
     local a = UnitAffectingCombat("player") and 1 or 0.5
     for _, frame in next, oUF.objects do
          frame:SetAlpha(a)
     end
end)