add checks to reset buttons in construction menu on close

[VerinSenpai]

About the PR

Fixed an issue where closing the construction menu while erase or build mode is active leaves those functions toggled and keeps certain top menu functions and interactions disabled.

Why / Balance

The issue is that with erase mode active in particular, there is no indication that erase mode is still active as the window is gone and the top menu button has toggled off. This leads to confusion as you are unable to perform certain interactions and shortcuts with no indication as to what is causing it.

Technical details

I wanted to get some feedback on this as the implementation may be poor or the issue may be intentional and just in need of some tweaking. I moved the code handling EraseButtonToggled to its own function similar to BuildButtonToggled and then added checks for both to reset them upon closing the window.

Media

2024-09-19.22-56-12.mp4

Requirements

• I have read and am following the Pull Request and Changelog Guidelines.
• I have added media to this PR or it does not require an ingame showcase.

Breaking changes

No breaking changes to note.

Changelog

🆑

• fix: Build and erase modes no longer remain active upon closing the construction menu.

[Golinth]

God bless, as a new player this confused me so fucking much