search

ennvina / spellactivationoverlay

Spell Alerts for WoW Classic
MIT License
6 stars 18 forks source link

a small suggestion to include in the paladin #319

Open Lennrj opened 1 month ago

Lennrj commented 1 month ago

I have a print here of a weakaura that I use in the paladin, but it has low graphic quality, it would be interesting for you to add it to your addon with a better resolution if you want, it would be very good for the inquisition, thank you for your work!

Captura de tela 2024-06-15 184041

ennvina commented 1 month ago

Hi @Lennrj and thank you for your feedback. Which effect are you referring to? It looks like you have Divine Purpose, which already exists in the addon.

Pocokk commented 1 month ago

I think he means the bottom part, the half-circle where the number "30.2" is located.

ennvina commented 1 month ago

If that's the case, then it's out of scope of the addon. The addon is there to tell players "this rare proc happened, you might want to cast this spell or ability that you probably wouldn't have cast otherwise". Inquisition is a buff to keep up over the course of a fight. There are addons designed to track that, such as WeakAuras.

On the other hand, SpellActivationOverlay could track when the paladin does not have Inquisition, in which case it would glow the Inquisition button to tell "hey, it's time you cast it". But I don't think it's a good idea to spam this information, as there are times where the paladin has no Inquisition buff and should save their Holy Power for some other time (e.g. burst phase) or for other spells (e.g. Templar's Verdict if the fight ends too soon™).