-
Notifications
You must be signed in to change notification settings - Fork 4
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
Team 2 UI alasdair #198
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…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.
…component or screen change functionality.
…ing the entities on screen. Does not yet handle incoming waves.
…rash issues re-entering the level.
…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.
AlasdairS4698737
requested review from
praneetdhoolia,
shiv-0831,
The-AhmadAA and
Mohamad11Dab
September 30, 2023 23:10
Mohamad11Dab
approved these changes
Oct 1, 2023
There was a problem hiding this 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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a pause ui button that pauses the game when clicked.
Notes: