fix for toggle regression

[rianconley]

If your pull request tries to address a specific issue from the repository, please reference to it.
Don't forget to include a description of the changes proposes. https://discord.me/AncientBeast

Hey @DreadKnight , here's the pull request for the issue #1634. Let me know if there's any regression issues.

---

IssueHunt Summary

Referenced issues

This pull request has been submitted to:

• #1634: passive icon ability toggling regression

---

[gitpod-io]

[DreadKnight]

@rianconley Did a bit of testing and found one problem so far: when scrolling into one of the directions or pressing the passive icon, the "Canceled" state is not triggered (having no ability selected, allowing for unit movement), but if scrolling in the other direction, it works just as intended.

[rianconley]

hey @DreadKnight , can you elaborate on the expected scroll behavior? Is it when you use the mouse wheel which cycles through the abilities? Can you explain how the canceled should work if you press passive or "scroll"? Thanks.

[DreadKnight]

@rianconley After going through all the usable abilities once, the "Canceled" state should be triggered, resulting in no usable ability selected, being able to just move the unit around.
Currently using the passive ability icon and scrolling towards the hand it doesn't do that.

[rianconley]

Hey @DreadKnight , please pull the latest from AB-1634 pull request. The update addresses canceling ability after cycling..

[DreadKnight]

@rianconley I've tested it, seems to work fine this time, well done!