Naix v1.0 GUI DotA

[Daffa]

Reading the code makes me think: is this MUI?

 

[MyPad]

A lot of work must be done before this can be deemed useful.

Spoiler: Code Review

• General
  • Have a configuration section (usually a separate trigger) for each spell. Most of the spell values appear to be hardcoded.
• Rage
  • Rage (container) appears to be the custom Spell Immune ability that's .. meant to be hidden? You can hide it using the Command Button Position trick (0, -11), which is now intended behavior since 1.28+.
  • The spell itself is not MUI, since there appears to be no data attachment to the running timer.
• Feast
  • Refer to the General section for more details.
• Open Wounds
  • Open Wounds Cast should attempt to assign the level of the ability being cast as the reference level for the debuff being placed or update the value as the maximum between the previous value and the new level of the ability being cast.
  • Open Wounds Cast should not attach a new unit event every time it's cast, since a damage detection system is already being used by the spellpack.
  • Open Wounds Effect should use the reference level for the debuff instead of the level of the attacker's own ability.
• Infest
  • Infest Cast and Infest Effect should not share the same event.
    • Infest Cast should be used with the event "A unit begins casting an ability" only to determine whether the target is eligible or not.
    • Infest Effect does not need the stop order interruption block, since Infest Cast should ideally handle that already.
  • Infest Consume leaks a location handle. (ConsumePoint1)
  • Infest Select is neither MUI, nor does it handle additional events for players 13 to 24 in later patches. Just use the generic event instead.
  • There's no need to use Do Nothing as an action in Infest Move Loop, aside from padding the conditional blocks.