From 740940dece098733e9ca179e227e121acd906ed2 Mon Sep 17 00:00:00 2001 From: Brues <5278969+brues-code@users.noreply.github.com> Date: Sun, 21 Jun 2026 00:11:20 -0500 Subject: [PATCH] actionbar: defer to SuperCleveRoidMacros for managed macros In ButtonMacroScan, when SuperCleveRoidMacros manages the slot (CleveRoids.IsManagedAction), leave spellslot/booktype unset and return early. This lets the button's icon, cooldown, and tooltip flow through the addon's hooked GetActionTexture / GetActionCooldown / GameTooltip:SetAction so they follow the active conditional dynamically, instead of being frozen to the first statically-scanned spell. --- modules/actionbar.lua | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/modules/actionbar.lua b/modules/actionbar.lua index 334c7c3f..659f7581 100644 --- a/modules/actionbar.lua +++ b/modules/actionbar.lua @@ -370,6 +370,16 @@ pfUI:RegisterModule("actionbar", function () if not self.scanmacro then return end if pfUI.bars.skip_macro then return end + -- SuperCleveRoidMacros: for macros it manages, leave spellslot/booktype unset + -- so the button's icon, cooldown, and tooltip flow through the hooked + -- GetActionTexture / GetActionCooldown / GameTooltip:SetAction and follow the + -- active conditional dynamically, instead of being frozen to the first + -- statically-scanned spell. + if CleveRoids and CleveRoids.IsManagedAction and CleveRoids.IsManagedAction(self.id) then + self.spellslot, self.booktype = nil, nil + return + end + local kind, slot = GetActionInfo(self.id) self.spellslot, self.booktype = nil, nil if kind == 'macro' then