Radar v1.1

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Radar v1.1 | Reviewed by Maker | 10th Mar 2013

NEEDS FIX

The ability is MUI The ability is leakless The ability has a fitting effect The tooltip is nice The ability does not detect all units. I placed 100 peasants near the caster and 7 were left without the buff The importing instructions could say that the ability and dummy need to be set correctly in the trigger, as the can refer to wrong objects when importing the ability into another map You could rename the hashtable variable, the name is too generic Increase the follow through time a bit You could null the location variables you remove to avoid minor leaks

[tr]

Old review Radar v1.0 | Reviewed by Maker | 6th Mar 2013 NEEDS FIX The ability is MUI The ability is leakless The ability has a fitting effect The tooltip is nice The ability does not detect invisible units The importing instruction could say that the ability and dummy need to be set correctly in the trigger, as the can refer to wrong objects when importing the ability into another map Radar Loop should be initially off You could rename the dummy Faerie Fire ability so it is not confused with the original You might not need to order the dummy to cast FF on itself Increase the follow through time a bit [tr]

 

[Jazztastic]

Reserved in case I need more space. I never do.

This is a depart from most of my spells in that it does no damage. It is simply an interesting idea that I was playing with. I could see some use for it though.

 

[pred1980]

looks cool ^^

 

[Jazztastic]

Updated to v1.1

Changed the fearie fire name, changed the importing instructions, fixed a few redundancies in the code.

I played for about 15 minutes with different methods of invisible detection but none of them would work (including truesight, wards, dust, and farsight). The truesight would "lag", and by the time the invisible unit was visible the wave had already passed. I couldn't find a real solution to this problem. As it is, the radar is currently a visible units only ability.

 

[Almia]

Add prefixes.

edit

Prefixes is a must because we can experience variable collision due to their names.