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.

Sign up for GitHub

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

Jump to bottom

Update StrategicPoint and complete tower buying flow #40

Closed
4 of 5 tasks
davcri opened this issue Oct 10, 2020 · 3 comments
Closed
4 of 5 tasks

Update StrategicPoint and complete tower buying flow #40

davcri opened this issue Oct 10, 2020 · 3 comments
Assignees
@davcri
Labels
gameplay

Comments

@davcri
Copy link
Member

davcri commented Oct 10, 2020 •
edited

Once #32 is done, we need to complete the buy_tower method in strategic-point-menu.gd.

It should:

  • close the StrategicPointMenu
  • remove money from the player
  • disable input (in the future we should implement a logic to show an tower upgrade menu)

Also we need to improve the behaviour of the StrategicPoint:

  • close the StrategicPointMenu by clicking outside of it
  • when the user closes the StrategicPointMenu and they didn't buy the tower, make sure that the tower is removed
    • eg: click on the StrategicPointMenu, click on a Slot to preview the tower, click again on the StrategicPointMenu to hide the menu. Now the tower should not be visible
@davcri davcri self-assigned this Oct 11, 2020
@davcri
Copy link
Member Author

davcri commented Oct 11, 2020

I'm working on this one.

davcri added a commit that referenced this issue Oct 11, 2020
@davcri
Fix most of the points of issue #40.
6cd93be 
Add state variable to Tower
@davcri
Copy link
Member Author

davcri commented Oct 11, 2020

There is one point that I need to finish, but I'll wait for #32 to be done

@davcri
Copy link
Member Author

davcri commented Oct 26, 2020

Done with 1210932

@davcri davcri closed this as completed Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davcri davcri

Labels
gameplay
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@davcri