Fix for Upgraded Seismic Stomp Visual Behavior, issue 2125/2126

[xTammaro]

Fix for Upgraded Seismic Stomp Visual Behavior

This fixes issue #2125 and potentially #2126. The new visual behaviour for Seismic Stomp upgraded ability:

Screen.Recording.2023-10-16.at.2.12.14.pm.mov

The changes include:

Only filling in the hexes of the creature that is hovered

Showing dashed hex for empty hexes

Cancelling the ability upon clicking the empty hex.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
ancientbeast	✅ Ready (Inspect)	Visit Preview	Oct 17, 2023 4:47am

[ghost]

👇 Click on the image for a new way to code review

Legend

[DreadKnight]

@xTammaro Heya! Did a bit of testing and found some issues:

• clicking path leading to first unit should trigger attack (see non-upgraded version or other ranged abilities)
• don't show multiple cancel cursors
  

[xTammaro]

I have fixed both issues!
Only one cancel cursor is shown, and only for hexes after the first creature in the path:

Also hovering over this first empty hex will only highlight the first creature, and clicking on it will attack the first creature:

Let me know if anything else needs tweaking, otherwise I think this should work as expected!

[xTammaro]

Sorry I have found an issue, this is not ready.

[xTammaro]

The issue is fixed, now works with multi-directional targets:

[DreadKnight]

@xTammaro Looks good to me. Well done! 👍🏻