Skip to content

Fix: Max Health Style not rendering when Absorb Style is disabled - #1917

Merged
EllesmereGaming merged 1 commit into
EllesmereGaming:mainfrom
svart2521:absorbs-max-health-style
Sep 2, 2026
Merged

Fix: Max Health Style not rendering when Absorb Style is disabled#1917
EllesmereGaming merged 1 commit into
EllesmereGaming:mainfrom
svart2521:absorbs-max-health-style

Conversation

@svart2521

Copy link
Copy Markdown
Contributor

Bug: https://discord.com/channels/585577383847788554/1543885241318637619

Issue: Raid Frames' Max Health Style overlay (e.g. for max-HP-reduction debuffs like the Den of Nalorakk curse) never rendered unless Absorb Style was also enabled, even though it's meant to be independent like Heal Absorb Style/Bar. Two separate early-return gates in UpdateAbsorb bailed out of the shared overlay frame whenever Absorb Style was off, without ever checking whether Max Health Style itself was on.
Fix: Added a maxHealthOn flag checked by both gates so the function keeps painting when Max Health Style is enabled on its own. Also scoped the shield-absorb-specific bar painting (previously reached via fallthrough) behind styleOn so it doesn't re-show the unstyled shield-absorb overlay when Absorb Style is off.

Bug:

Issue: Raid Frames' Max Health Style overlay (e.g. for max-HP-reduction debuffs like the Den of Nalorakk curse) never rendered unless Absorb Style was also enabled, even though it's meant to be independent like Heal Absorb Style/Bar. Two separate early-return gates in UpdateAbsorb bailed out of the shared overlay frame whenever Absorb Style was off, without ever checking whether Max Health Style itself was on.
Fix: Added a maxHealthOn flag checked by both gates so the function keeps painting when Max Health Style is enabled on its own. Also scoped the shield-absorb-specific bar painting (previously reached via fallthrough) behind styleOn so it doesn't re-show the unstyled shield-absorb overlay when Absorb Style is off.
@EllesmereGaming
EllesmereGaming merged commit 311e68d into EllesmereGaming:main Sep 2, 2026
1 check passed
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