Skip to content

Fix debuff icons persisting on dead frames#85

Merged
DanderBot merged 3 commits into
DanderBot:mainfrom
Krathe82:fix/debuffs-persisting-on-dead-frames
May 17, 2026
Merged

Fix debuff icons persisting on dead frames#85
DanderBot merged 3 commits into
DanderBot:mainfrom
Krathe82:fix/debuffs-persisting-on-dead-frames

Conversation

@Krathe82

Copy link
Copy Markdown
Contributor

Summary

When a unit dies, WoW does not fire UNIT_AURA — so the aura cache retains the pre-death aura list and any debuff icons that were showing before death remain on the dead frame until something else triggers an aura update.

Two-part fix:

  • Features/Auras.lua UpdateAuraIconsDirect: Added a dead/ghost guard at the top of the function. If UnitIsDeadOrGhost(unit) is true, all aura icons are immediately hidden and the function returns early — same code path as the existing "no cache" quick-out.
  • Frames/Update.lua UpdateHealthFast: Added a call to UpdateAuras_Enhanced in the death path (after ApplyDeadFade), so the new dead guard fires at the moment of death rather than waiting for the next external aura event.

Fixes https://danders-lab.netlify.app/bugs/863

Krathe82 added a commit to Krathe82/DandersFrames that referenced this pull request May 15, 2026
Add edge-detect flag (dfLastKnownDead) so UpdateAuras_Enhanced is only
called once on the dead transition, not on every subsequent UNIT_HEALTH
event while the unit remains dead. Clears on the alive branch alongside
ResetDeadFade so revival re-arms the flush for the next death.
@DanderBot DanderBot merged commit db7ddb4 into DanderBot:main May 17, 2026
@Krathe82 Krathe82 deleted the fix/debuffs-persisting-on-dead-frames branch June 4, 2026 12:42
Krathe82 added a commit to Krathe82/DandersFrames that referenced this pull request Jun 6, 2026
Krathe82 pushed a commit to Krathe82/DandersFrames that referenced this pull request Jun 6, 2026
…on-dead-frames

Fix debuff icons persisting on dead frames
Krathe82 pushed a commit to Krathe82/DandersFrames that referenced this pull request Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants