fix some incoming damage mistaken as outgoing, make sure all damage text has a color for its damage type

MikScrollingBattleText/MikCombatEventHelper.lua

@@ -901,9 +901,15 @@ nampowerHandlers["SPELL_DAMAGE_EVENT_SELF"] = function()
  local effectAuraStr = arg8
 
  -- Determine direction: player outgoing, or pet outgoing.
+ -- Special case: if the player is both caster and target (e.g. Hellfire self-damage,
+ -- spell reflects), route as incoming since the player is taking damage.
  local directionType
  if IsPlayerGUID(casterGuid) then
-  directionType = MikCEH.DIRECTIONTYPE_PLAYER_OUTGOING
+  if IsPlayerGUID(targetGuid) then
+   directionType = MikCEH.DIRECTIONTYPE_PLAYER_INCOMING
+  else
+   directionType = MikCEH.DIRECTIONTYPE_PLAYER_OUTGOING
+  end
   nampowerSpellDamageActive = true
  elseif IsPetGUID(casterGuid) then
   directionType = MikCEH.DIRECTIONTYPE_PET_OUTGOING

MikScrollingBattleText/MikScrollingBattleText.lua

@@ -1783,9 +1783,8 @@ function MikSBT.AddAnimation(animationEvent)
   animDisplayInfo.ColorG = animationEvent.EventSettings.FontSettings.Color.g;
   animDisplayInfo.ColorB = animationEvent.EventSettings.FontSettings.Color.b;
 
-  -- Override font color based on spell school damage type (outgoing only).
-  if animationEvent.DamageType and animationEvent.EventType
-     and string_find(animationEvent.EventType, "OUTGOING") then
+  -- Override font color based on spell school damage type.
+  if animationEvent.DamageType and animationEvent.EventType then
    local dtColor = GetDamageTypeFontColor(animationEvent.DamageType)
    if dtColor then
     animDisplayInfo.ColorR = dtColor.r