Antares
Spell Reviewer
- Joined
- Dec 13, 2009
- Messages
- 552
- You have a mix of tab and eight-space indents in your code, which messes up its formatting in anything other than vanilla World Editor.
- The animation for the counter action doesn't play when the SINGLE_COUNTER is set to true, because the hero will usually attack immediately after finishing the spell, overwriting the animation.
- The config is spread throughout the code. If you make the CounterCondition into a regular function, you can put it at the top of the script below the globals and have the config separated from the rest of the script. You could also make the damage formula into a function that you put into the config as well.
- The cooldown for the spell doesn't start when SINGLE_COUNTER is set to true and the first attack lands shortly after casting the spell.
- Listing the dependencies in your description would be nice.
- The spellcast could have a better visual representation. There's currently no spell effect or sound when you cast it. If the user is supposed to do that via the Channel ability, preset something that looks nice.
- I don't quite understand why "Disable other abilities" is enabled in the Channel spell.
This could be a really cool spell, but it needs some bugs fixed first before it can be approved. Some more eye-candy would also be in order.
Awaiting Update