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

Remove 'cancel' option from rocket attack select unit dialog #1386

Merged
merged 1 commit into from
Dec 18, 2016
Merged

Remove 'cancel' option from rocket attack select unit dialog #1386

merged 1 commit into from
Dec 18, 2016

Conversation

DanVanAtta
Copy link
Member

The cancel button stays on the same screen when selecting rocket damage targets, in effect it does nothing. This is a quick change to remove the button. A good next step/enhancement to the game would be to add the cancel back in, and have it so when cancel is clicked a player can pick a new territory to rocket attack. This looks to be a difficult change at the moment since the territory to attack is pretty baked into the unit target selection. Thus for now we remove the button that does nothing.

Addresses: #1209

The cancel button stays on the same screen when selecting rocket damage targets, in effect it does nothing. This is a quick change to remove the button. A good next step/enhancement to the game would be to add the cancel back in, and have it so when cancel is clicked a player can pick a new territory to rocket attack. This looks to be a difficult change at the moment since the territory to attack is pretty baked into the unit target selection. Thus for now we remove the button that does nothing.
@DanVanAtta DanVanAtta merged commit fee5af5 into triplea-game:master Dec 18, 2016
@DanVanAtta DanVanAtta deleted the remove_broken_cancel branch December 18, 2016 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant