Moderator
M
Moderator
12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.
IcemanBo: Too long as NeedsFix. Rejected.
Reviewed by Maker, HealingRays1.00, 26th Jan 2012
- You spam effects
- Leaks a location when you create an effect in the looping trigger
- Make tooltips look more like standard Blizzard ability tooltips
- Use Starts the effect of an ability event
- You only need one point variable for the ability, not two
- Filter out dead units from being healed
- Remove invulnerability ability from the dummy, it has Locust
- Add importing instructions into the map file
If the only purpose of the dummy is to provide the custom value so you know which instance you're dealing with, then in my opinion it would be better to use Hanky's dynamic indexing and ditch unit indexer and the dummy unit.