Trinket Tracker Swapper

Bug Fixes

Fixed: ADDON_ACTION_BLOCKED errors causing the popup to break during combat

The trinket selection popup was calling Hide() on a non-secure frame during combat, which WoW's protected environment does not allow. This caused a cascade of issues:

• The popup would stop opening on one trinket slot until you hovered the other slot first
• Queuing a swap in combat would corrupt the popup state, making it unresponsive after combat ended
• The hover-detection ticker was calling Hide() in combat every time the mouse left the popup area, silently breaking the UI over time (typically after ~1 hour of play)

How it was fixed:

• Added CombatHidePopup() / CombatShowPopup() helpers that use SetAlpha(0/1) + EnableMouse() instead of Hide()/Show() during combat lockdown — visually identical, but fully compliant with WoW's protection rules
• Added RefreshPopupBorders() to update queue highlight borders in-place without destroying and recreating the popup
• All code paths that previously called HideTrinketPopup() during combat now correctly branch between the combat-safe and out-of-combat paths
• On PLAYER_REGEN_ENABLED (combat end), the popup state is now fully reset before swap queues are processed, preventing stale anchor references from blocking future interactions