Ahli / sc2xml

11 stars 1 forks source link

⚠️ Balance Implications ⚠️Liberator "AG - Target" ability's "Allow Movement" checkbox is not set #105

Open Joshua-Leibold opened 1 year ago

Joshua-Leibold commented 1 year ago

Over the years units that have morph abilities have been changed to not decelerate prior to morphing in order to make the unit feel more responsive (ie. Corruptor->Brood Lord, Overlord->Overseer etc.).

The liberator currently must decelerate prior to morphing to siege mode, even if the target siege zone is already within liberator casting range. This causes a phenomenon whereby players who order a liberator to siege at a location already within that liberator's cast range can have that siege order accidentally cancelled by a subsequent order to units in the same selection.

https://i.gyazo.com/e0507848d2790c6673669bbd09f19f42.mp4

https://i.gyazo.com/bd4d3ed4afdd252876f36b3eb9010098.mp4

Siege orders that are already within the liberator's cast range can be corrected to be uninterruptable by checking Liberator "AG - Target" ability's "Allow Movement" checkbox.

⚠️Balance Implications ⚠️ However, this results in shortening the total time between issuing a siege order and the siege zone becoming active for a moving liberator by up to ~0.5 seconds, since the time it takes to decelerate is removed. It would therefore be advisable to lengthen the time it takes a liberator to siege by a proportionate value in accompaniment with this change to maintain the current balance

Joshua-Leibold commented 1 year ago

XML Change to make Liberator morph act immediately when within casting range:

<CAbilEffectTarget id="LiberatorAGTarget">
        <Flags index="AllowMovement" value="1"/>
    </CAbilEffectTarget>