ennvina / spellactivationoverlay

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

Effect Rework #238

Closed ennvina closed 3 months ago

ennvina commented 4 months ago

The current aura system makes life difficult to add new effects. This system was probably well suited for the early days of the addon, but the introduction of new functionalities over time has made more and more complex to add new effects.

A rework will pave the future of the addon, hopefully helping for the imminent release of Cataclysm, where there are a lot of effect to add, update, or exclude to Cataclysm players.

The main idea is to introduce an 'effect' object that describes an effect:

(the list may not be exhaustive)

This object will serve many purposes, such as:

Please note: the goal is not to remove all existing auras overnight. The current system will still exist. As a matter of fact, the current system may still hold the basement, and will be used by complex code such as handlers.

ennvina commented 3 months ago

First implementation in #245. Improved many times afterwards, especially in #259, #271 and #273. It can still be (and will be) improved. The issue can be closed, as the main functionality is there. Some documentation is available at https://github.com/ennvina/spellactivationoverlay/wiki/Overlay#High-level