Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Done and Skip Turn buttons do not accurately reflect command state. #4479

Closed
pakfront opened this issue Jun 2, 2023 · 1 comment · Fixed by #4489
Closed

Done and Skip Turn buttons do not accurately reflect command state. #4479

pakfront opened this issue Jun 2, 2023 · 1 comment · Fixed by #4489
Assignees
Labels
Bug GUI User Interface

Comments

@pakfront
Copy link
Collaborator

pakfront commented Jun 2, 2023

Environment

0.49.13

Description

if you have a fallen mech, you can't just hit "get up" and then click move. It's grayed out. You have to get up and move or it stays Grey unless you do random clicking. Stated another way: if you get up but don't leave your hex, the move button is grayed out.

As an addition, changing the "Done" button to "Fire", leads to the odd situation where if you twist a torso, the skip firing button becomes greyed out, so you have to click "Fire" to end that unit's turn, even though it hasn't fired.

Note: this is currently using the exact same logic that the nag screen used, but clearly this is not correct.

@pakfront pakfront added Bug GUI User Interface labels Jun 2, 2023
@pakfront pakfront self-assigned this Jun 2, 2023
@SimonLandmine
Copy link

SimonLandmine commented Jun 5, 2023

Additionally, if you turn on the spot using keyboard commands, but remain in the same hex, it does not recognise that as a Move, so the 'Skip Movement' button remains highlighted rather than the 'Move' button. This is probably a linked issue, which is why I'm raising in this bug.
(Oddly, this does not happen if you use the SHIFT+CLICK approach to turning on the spot.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug GUI User Interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants