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 #198

Merged
merged 19 commits into from
Oct 1, 2023
Merged

Team 2 UI alasdair #198

merged 19 commits into from
Oct 1, 2023

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 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.
Copy link
Contributor

@Mohamad11Dab Mohamad11Dab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ! Just make sure that you copy the full textures, atlases and sounds from ForestGameArea to AssetLoader before pushing to main.

@Mohamad11Dab Mohamad11Dab merged commit 936452a into team-2-ui Oct 1, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants