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

Team 2 UI alasdair #197

Closed
wants to merge 19 commits into from
Closed

Team 2 UI alasdair #197

wants to merge 19 commits into from

Conversation

AlasdairS4698737
Copy link
Contributor

@AlasdairS4698737 AlasdairS4698737 commented Sep 29, 2023

Added a pause ui button that pauses the game when clicked.
Notes:

  • Have not implemented prevention of clicking the pause button multiple times to make multiple pause menus.
  • Have not paused the wave spawn timer, team 4 said they are adding a pause method to the wave spawning task, so this will be implemented once I'm able to access that.
  • Once team 3 has updated the ButtonFactory to be static, I'll use it to improve the appearance of the pause menu buttons.

Death7Bed and others added 19 commits September 15, 2023 23:19
…ameScreen, creating a pause button in the top right of the game screen
…etSelectButton, PauseMenuMainMenuButton, PauseMenuTimeStopComponent and PauseMenuRenderComponent to act as components for a pause menu entity. These components do nothing presently, and the pause menu entity is not implemented yet.
…. Added functionality to PauseMenuStopTimeComponent to stop time when created, and resume time when disposed. Added functionality to MainGamePauseDisplay to create a pause menu entity when the pause button is clicked.
…ing the entities on screen. Does not yet handle incoming waves.
…UE: The wave timer entity isn't present in MainGameScreen, so it can't be passed to the pause menu at present. Fixing this is the next objective for this task.
…ned. ISSUE: The wave timer entity isn't present in MainGameScreen, so it can't be passed to the pause menu at present. Fixing this is the next objective for this task."

This reverts commit e095797.
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.

4 participants