feat(icons): upgrade remaining status icons to modern atlases (PR DanderBot#31 set)

Add a legacy-path -> atlas map + DF:SetUpgradedStatusIcon(region, legacyPath):
prefers the modern atlas (sharper, matches Blizzard's raid frames) and falls
back to the legacy texture when absent, owning texcoord state so callers drop
their trailing SetTexCoord.

Routed every render site (live + test mode) through it:
ready check -> UI-LFG-ReadyMark/DeclineMark/PendingMark-Raid; summon ->
RaidFrame-Icon-Summon*; resurrection -> RaidFrame-Icon-Rez; phased ->
RaidFrame-Icon-Phasing, LFR eye -> RaidFrame-Icon-LFR; AFK ->
characterupdate_clock-icon; vehicle -> RaidFrame-Icon-Vehicle; MT/MA ->
RaidFrame-Icon-MainTank/MainAssist. Leader/assistant/master-looter and raid
target markers are intentionally left as-is.

Author: Krathe82

Frames/Bars.lua

@@ -2443,10 +2443,10 @@ function DF:UpdateReadyCheckIcon(frame)
     local readyCheckStatus = GetReadyCheckStatus(frame.unit)
 
     if readyCheckStatus == "ready" then
-        frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-Ready")
+        DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-Ready")
         frame.readyCheckIcon:Show()
     elseif readyCheckStatus == "notready" then
-        frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-NotReady")
+        DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-NotReady")
         frame.readyCheckIcon:Show()
     elseif readyCheckStatus == "waiting" then
         -- Check if player is AFK while waiting (enhanced ready check)
@@ -2460,9 +2460,9 @@ function DF:UpdateReadyCheckIcon(frame)
 
         if afkAccessible and isAFK then
             -- AFK state - show not ready icon (they likely won't respond)
-            frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-NotReady")
+            DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-NotReady")
         else
-            frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-Waiting")
+            DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-Waiting")
         end
         frame.readyCheckIcon:Show()
     else

Frames/Core.lua

@@ -715,3 +715,38 @@ function DF:SetIconTextureOrAtlas(region, value, l, r, t, b)
         end
     end
 end
+
+-- Legacy raid-frame status icon textures → their modern atlas equivalents.
+-- The atlas versions (used by Blizzard's own raid frames) are sharper; we keep
+-- the legacy path as the lookup key AND the fallback so nothing breaks if an
+-- atlas is ever absent.
+local LEGACY_STATUS_ICON_ATLAS = {
+    ["Interface\\RaidFrame\\ReadyCheck-Ready"]         = "UI-LFG-ReadyMark-Raid",
+    ["Interface\\RaidFrame\\ReadyCheck-NotReady"]      = "UI-LFG-DeclineMark-Raid",
+    ["Interface\\RaidFrame\\ReadyCheck-Waiting"]       = "UI-LFG-PendingMark-Raid",
+    ["Interface\\RaidFrame\\Raid-Icon-SummonPending"]  = "RaidFrame-Icon-SummonPending",
+    ["Interface\\RaidFrame\\Raid-Icon-SummonAccepted"] = "RaidFrame-Icon-SummonAccepted",
+    ["Interface\\RaidFrame\\Raid-Icon-SummonDeclined"] = "RaidFrame-Icon-SummonDeclined",
+    ["Interface\\RaidFrame\\Raid-Icon-Rez"]            = "RaidFrame-Icon-Rez",
+    ["Interface\\TargetingFrame\\UI-PhasingIcon"]      = "RaidFrame-Icon-Phasing",
+    ["Interface\\LFGFrame\\LFG-Eye"]                   = "RaidFrame-Icon-LFR",
+    ["Interface\\FriendsFrame\\StatusIcon-Away"]       = "characterupdate_clock-icon",
+    ["Interface\\Vehicles\\UI-Vehicles-Raid-Icon"]     = "RaidFrame-Icon-Vehicle",
+    ["Interface\\GroupFrame\\UI-Group-MainTankIcon"]   = "RaidFrame-Icon-MainTank",
+    ["Interface\\GroupFrame\\UI-Group-MainAssistIcon"] = "RaidFrame-Icon-MainAssist",
+}
+
+-- Render a known legacy status-icon texture as its modern atlas (sharper),
+-- falling back to the legacy file when the atlas isn't available. Owns the
+-- texcoord state, so call sites must NOT add a trailing SetTexCoord — passing
+-- the legacy path as the single source of truth is enough.
+function DF:SetUpgradedStatusIcon(region, legacyTexture)
+    if not region or not legacyTexture then return end
+    local atlas = LEGACY_STATUS_ICON_ATLAS[legacyTexture]
+    if atlas and C_Texture and C_Texture.GetAtlasInfo and C_Texture.GetAtlasInfo(atlas) then
+        region:SetAtlas(atlas)
+    else
+        region:SetTexture(legacyTexture)
+        region:SetTexCoord(0, 1, 0, 1)
+    end
+end

Frames/StatusIcons.lua

@@ -83,15 +83,15 @@ function DF:CreateStatusIcons(frame)
     -- ========================================
     frame.summonIcon = CreateStatusIcon(overlay, 16)
     frame.summonIcon:SetPoint("CENTER", frame, "CENTER", 0, 0)
-    frame.summonIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-SummonPending")
+    DF:SetUpgradedStatusIcon(frame.summonIcon.texture, "Interface\\RaidFrame\\Raid-Icon-SummonPending")
     frame.summonIcon.text:SetTextColor(0.6, 0.2, 1, 1)  -- Purple for summon
 
     -- ========================================
     -- RESURRECTION ICON (incoming res status)
     -- ========================================
     frame.resurrectionIcon = CreateStatusIcon(overlay, 16)
     frame.resurrectionIcon:SetPoint("CENTER", frame, "CENTER", 0, 10)
-    frame.resurrectionIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-Rez")
+    DF:SetUpgradedStatusIcon(frame.resurrectionIcon.texture, "Interface\\RaidFrame\\Raid-Icon-Rez")
     frame.resurrectionIcon.text:SetTextColor(0.2, 1, 0.2, 1)  -- Green for res
     frame.resurrectionIcon.unitFrame = frame
     frame.resurrectionIcon:EnableMouse(true)
@@ -131,16 +131,15 @@ function DF:CreateStatusIcons(frame)
     -- ========================================
     frame.phasedIcon = CreateStatusIcon(overlay, 16)
     frame.phasedIcon:SetPoint("TOPRIGHT", frame, "TOPRIGHT", -2, -2)
-    frame.phasedIcon.texture:SetTexture("Interface\\TargetingFrame\\UI-PhasingIcon")
-    frame.phasedIcon.texture:SetTexCoord(0.15625, 0.84375, 0.15625, 0.84375)
+    DF:SetUpgradedStatusIcon(frame.phasedIcon.texture, "Interface\\TargetingFrame\\UI-PhasingIcon")
     frame.phasedIcon.text:SetTextColor(0.5, 0.5, 1, 1)  -- Blue-ish for phased
 
     -- ========================================
     -- AFK ICON (unit is away from keyboard)
     -- ========================================
     frame.afkIcon = CreateStatusIcon(overlay, 32)
     frame.afkIcon:SetPoint("CENTER", frame, "CENTER", 0, 0)
-    frame.afkIcon.texture:SetTexture("Interface\\FriendsFrame\\StatusIcon-Away")
+    DF:SetUpgradedStatusIcon(frame.afkIcon.texture, "Interface\\FriendsFrame\\StatusIcon-Away")
     DF:SafeSetFont(frame.afkIcon.text, nil, 12, "OUTLINE")
     frame.afkIcon.text:SetTextColor(1, 0.5, 0, 1)  -- Orange for AFK
     -- Timer text (separate from main text, shown below/after)
@@ -155,7 +154,7 @@ function DF:CreateStatusIcons(frame)
     -- ========================================
     frame.vehicleIcon = CreateStatusIcon(overlay, 16)
     frame.vehicleIcon:SetPoint("BOTTOMRIGHT", frame, "BOTTOMRIGHT", -2, 2)
-    frame.vehicleIcon.texture:SetTexture("Interface\\Vehicles\\UI-Vehicles-Raid-Icon")
+    DF:SetUpgradedStatusIcon(frame.vehicleIcon.texture, "Interface\\Vehicles\\UI-Vehicles-Raid-Icon")
     frame.vehicleIcon.text:SetTextColor(0.4, 0.8, 1, 1)  -- Light blue for vehicle
 
     -- ========================================
@@ -339,8 +338,7 @@ function DF:UpdateSummonIcon(frame)
     end
 
     if showIcon then
-        frame.summonIcon.texture:SetTexture(texture)
-        frame.summonIcon.texture:SetTexCoord(0, 1, 0, 1)
+        DF:SetUpgradedStatusIcon(frame.summonIcon.texture, texture)
         ApplyIconSettings(frame.summonIcon, db, "summonIcon")
 
         -- Show as text or icon based on setting
@@ -411,7 +409,7 @@ function DF:UpdateResurrectionIcon(frame)
                 resCache[unit] = 1
                 resTimer = resTimer or C_Timer.NewTicker(0.25, ResTimerCleanup)
             end
-            frame.resurrectionIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-Rez")
+            DF:SetUpgradedStatusIcon(frame.resurrectionIcon.texture, "Interface\\RaidFrame\\Raid-Icon-Rez")
             frame.resurrectionIcon.texture:SetVertexColor(0, 1, 0, 1)
             ApplyIconSettings(frame.resurrectionIcon, db, "resurrectionIcon")
             frame.resurrectionIcon:Show()
@@ -420,15 +418,15 @@ function DF:UpdateResurrectionIcon(frame)
             -- Was casting, now stopped → pending accept (yellow)
             -- Store timestamp so we can expire after 60s
             resCache[unit] = GetTime()
-            frame.resurrectionIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-Rez")
+            DF:SetUpgradedStatusIcon(frame.resurrectionIcon.texture, "Interface\\RaidFrame\\Raid-Icon-Rez")
             frame.resurrectionIcon.texture:SetVertexColor(1, 1, 0, 0.75)
             ApplyIconSettings(frame.resurrectionIcon, db, "resurrectionIcon")
             frame.resurrectionIcon:Show()
             return
         elseif resCache[unit] and resCache[unit] ~= 1 then
             -- Still showing pending accept (check not expired)
             if (GetTime() - resCache[unit]) <= RES_ACCEPT_TIMEOUT then
-                frame.resurrectionIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-Rez")
+                DF:SetUpgradedStatusIcon(frame.resurrectionIcon.texture, "Interface\\RaidFrame\\Raid-Icon-Rez")
                 frame.resurrectionIcon.texture:SetVertexColor(1, 1, 0, 0.75)
                 ApplyIconSettings(frame.resurrectionIcon, db, "resurrectionIcon")
                 frame.resurrectionIcon:Show()
@@ -661,11 +659,9 @@ function DF:UpdatePhasedIcon(frame)
         -- cached == -1 means LFG (other party), anything else means phased
         local isLFG = (cached == -1)
         if isLFG and db.phasedIconShowLFGEye then
-            frame.phasedIcon.texture:SetTexture("Interface\\LFGFrame\\LFG-Eye")
-            frame.phasedIcon.texture:SetTexCoord(0.14, 0.235, 0.28, 0.47)
+            DF:SetUpgradedStatusIcon(frame.phasedIcon.texture, "Interface\\LFGFrame\\LFG-Eye")
         else
-            frame.phasedIcon.texture:SetTexture("Interface\\TargetingFrame\\UI-PhasingIcon")
-            frame.phasedIcon.texture:SetTexCoord(0.15625, 0.84375, 0.15625, 0.84375)
+            DF:SetUpgradedStatusIcon(frame.phasedIcon.texture, "Interface\\TargetingFrame\\UI-PhasingIcon")
         end
         ApplyIconSettings(frame.phasedIcon, db, "phasedIcon")
         ShowIconAsText(frame.phasedIcon, db.phasedIconText or "Phased", db.phasedIconShowText)
@@ -932,13 +928,12 @@ function DF:UpdateRaidRoleIcon(frame)
     if showIcon and role then
         local statusText = nil
         if role == "MAINTANK" then
-            frame.raidRoleIcon.texture:SetTexture("Interface\\GroupFrame\\UI-Group-MainTankIcon")
+            DF:SetUpgradedStatusIcon(frame.raidRoleIcon.texture, "Interface\\GroupFrame\\UI-Group-MainTankIcon")
             statusText = db.raidRoleIconTextTank or "MT"
         else
-            frame.raidRoleIcon.texture:SetTexture("Interface\\GroupFrame\\UI-Group-MainAssistIcon")
+            DF:SetUpgradedStatusIcon(frame.raidRoleIcon.texture, "Interface\\GroupFrame\\UI-Group-MainAssistIcon")
             statusText = db.raidRoleIconTextAssist or "MA"
         end
-        frame.raidRoleIcon.texture:SetTexCoord(0, 1, 0, 1)
         ApplyIconSettings(frame.raidRoleIcon, db, "raidRoleIcon")
         ShowIconAsText(frame.raidRoleIcon, statusText, db.raidRoleIconShowText)
         frame.raidRoleIcon:Show()
@@ -1115,7 +1110,7 @@ function DF:UpdateReadyCheckIconEnhanced(frame)
         return
     end
 
-    frame.readyCheckIcon.texture:SetTexture(texture)
+    DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, texture)
 
     -- Apply positioning
     local scale = db.readyCheckIconScale or 1.0

TestMode/TestMode.lua

@@ -1308,7 +1308,7 @@ function DF:UpdateTestIcons(frame, testData)
         if not db.readyCheckIconEnabled or db.testShowStatusIcons == false then
             frame.readyCheckIcon:Hide()
         elseif testData.isLeader then
-            frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-Ready")
+            DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-Ready")
 
             local scale = db.readyCheckIconScale or 1.0
             local anchor = db.readyCheckIconAnchor or "CENTER"
@@ -1342,7 +1342,7 @@ function DF:UpdateTestIcons(frame, testData)
             end
 
             if texture then
-                frame.centerStatusIcon.texture:SetTexture(texture)
+                DF:SetUpgradedStatusIcon(frame.centerStatusIcon.texture, texture)
 
                 local scale = db.centerStatusIconScale or 1.0
                 local anchor = db.centerStatusIconAnchor or "CENTER"
@@ -1517,7 +1517,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.readyCheckIconEnabled or db.testShowStatusIcons == false then
             frame.readyCheckIcon:Hide()
         elseif testData.isLeader then
-            frame.readyCheckIcon.texture:SetTexture("Interface\\RaidFrame\\ReadyCheck-Ready")
+            DF:SetUpgradedStatusIcon(frame.readyCheckIcon.texture, "Interface\\RaidFrame\\ReadyCheck-Ready")
 
             local scale = db.readyCheckIconScale or 1.0
             local anchor = db.readyCheckIconAnchor or "CENTER"
@@ -1544,7 +1544,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.summonIconEnabled or db.testShowStatusIcons == false then
             frame.summonIcon:Hide()
         elseif testData.centerStatus == "summon" then
-            frame.summonIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-SummonPending")
+            DF:SetUpgradedStatusIcon(frame.summonIcon.texture, "Interface\\RaidFrame\\Raid-Icon-SummonPending")
 
             local scale = db.summonIconScale or 1.0
             local anchor = db.summonIconAnchor or "CENTER"
@@ -1573,7 +1573,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.resurrectionIconEnabled or db.testShowStatusIcons == false then
             frame.resurrectionIcon:Hide()
         elseif testData.centerStatus == "resurrect" then
-            frame.resurrectionIcon.texture:SetTexture("Interface\\RaidFrame\\Raid-Icon-Rez")
+            DF:SetUpgradedStatusIcon(frame.resurrectionIcon.texture, "Interface\\RaidFrame\\Raid-Icon-Rez")
             frame.resurrectionIcon.texture:SetVertexColor(0, 1, 0)  -- Green = being cast
 
             local scale = db.resurrectionIconScale or 1.0
@@ -1603,7 +1603,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.phasedIconEnabled or db.testShowStatusIcons == false then
             frame.phasedIcon:Hide()
         elseif testData.isPhased then
-            frame.phasedIcon.texture:SetTexture("Interface\\TargetingFrame\\UI-PhasingIcon")
+            DF:SetUpgradedStatusIcon(frame.phasedIcon.texture, "Interface\\TargetingFrame\\UI-PhasingIcon")
 
             local scale = db.phasedIconScale or 1.0
             local anchor = db.phasedIconAnchor or "TOPRIGHT"
@@ -1633,7 +1633,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
             frame.afkIcon:Hide()
             if frame.afkIcon.timerText then frame.afkIcon.timerText:Hide() end
         elseif testData.isAFK then
-            frame.afkIcon.texture:SetTexture("Interface\\FriendsFrame\\StatusIcon-Away")
+            DF:SetUpgradedStatusIcon(frame.afkIcon.texture, "Interface\\FriendsFrame\\StatusIcon-Away")
 
             local scale = db.afkIconScale or 1.0
             local anchor = db.afkIconAnchor or "CENTER"
@@ -1696,7 +1696,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.vehicleIconEnabled or db.testShowStatusIcons == false then
             frame.vehicleIcon:Hide()
         elseif testData.inVehicle then
-            frame.vehicleIcon.texture:SetTexture("Interface\\Vehicles\\UI-Vehicles-Raid-Icon")
+            DF:SetUpgradedStatusIcon(frame.vehicleIcon.texture, "Interface\\Vehicles\\UI-Vehicles-Raid-Icon")
 
             local scale = db.vehicleIconScale or 1.0
             local anchor = db.vehicleIconAnchor or "BOTTOMRIGHT"
@@ -1725,7 +1725,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
         if not db.raidRoleIconEnabled or db.testShowStatusIcons == false then
             frame.raidRoleIcon:Hide()
         elseif testData.isMainTank and db.raidRoleIconShowTank ~= false then
-            frame.raidRoleIcon.texture:SetTexture("Interface\\GroupFrame\\UI-Group-MainTankIcon")
+            DF:SetUpgradedStatusIcon(frame.raidRoleIcon.texture, "Interface\\GroupFrame\\UI-Group-MainTankIcon")
 
             local scale = db.raidRoleIconScale or 1.0
             local anchor = db.raidRoleIconAnchor or "BOTTOMLEFT"
@@ -1745,7 +1745,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
                 frame.raidRoleIcon:SetFrameLevel(frame:GetFrameLevel() + frameLevel)
             end
         elseif testData.isMainAssist and db.raidRoleIconShowAssist ~= false then
-            frame.raidRoleIcon.texture:SetTexture("Interface\\GroupFrame\\UI-Group-MainAssistIcon")
+            DF:SetUpgradedStatusIcon(frame.raidRoleIcon.texture, "Interface\\GroupFrame\\UI-Group-MainAssistIcon")
 
             local scale = db.raidRoleIconScale or 1.0
             local anchor = db.raidRoleIconAnchor or "BOTTOMLEFT"
@@ -1793,7 +1793,7 @@ function DF:UpdateTestStatusIcons(frame, testData)
             end
 
             if texture then
-                frame.centerStatusIcon.texture:SetTexture(texture)
+                DF:SetUpgradedStatusIcon(frame.centerStatusIcon.texture, texture)
 
                 local scale = db.centerStatusIconScale or 1.0
                 local anchor = db.centerStatusIconAnchor or "CENTER"