search

ennvina / spellactivationoverlay

Spell Alerts for WoW Classic
MIT License
7 stars 19 forks source link

fix: show master marksman proc overlay #287

Closed stako closed 4 months ago

stako commented 4 months ago

overlay config table was using the wrong number of aura stacks - the proc aura doesn't have stacks

after reaching 5 stacks of "Ready, Set, Aim..." the aura is removed and replaced with a single stack of "Fire!"

ennvina commented 4 months ago

Quick tip: because the final buff (Fire!) has no stacks, there is no need to set stacks = 1 to the effect. Not setting any stacks will default to stacks = 0 which means "any number of stacks". I took the liberty of changing it in 82952d439df2befc5b8090c7c87210ec07740f2d.