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

Game Over #103

Merged
merged 6 commits into from
Nov 15, 2023
Merged

Game Over #103

merged 6 commits into from
Nov 15, 2023

Conversation

thombruce
Copy link
Owner

@thombruce thombruce commented Nov 15, 2023

closes #59

By submitting this pull request, you agree to follow our Code of Conduct: https://github.com/thombruce/.github/blob/main/CODE_OF_CONDUCT.md


Internal use. Do not delete.

  • Tests passing
  • Coverage sufficient
  • Manual review
  • No A11y regression
  • Translations provided or not needed

@thombruce
Copy link
Owner Author

thombruce commented Nov 15, 2023

Issues

  • With the exception of the physics system, all other active game systems pause1

Footnotes

  1. This isn't a massive problem, but I would prefer for things like animation and UI elements to continue operating in part to show that the universe goes on without you in it, and in part because we'll eventually need these systems to continue working outside of the main active state in any case (e.g. when we add interior view(s) and other gameplay states).

@thombruce
Copy link
Owner Author

thombruce commented Nov 15, 2023

More Issues

  • Menu is unresponsive initially
  • "Enter" press can result in new game as soon as player quits

@thombruce
Copy link
Owner Author

"Enter" press just isn't working very well at all... Adding a second button has helped the menu, but it still isn't initially responding sometimes.

Navigating between focusables is working just fine. It is as if the actionable button press system just isn't running...

@thombruce
Copy link
Owner Author

...and I know exactly why.

@thombruce
Copy link
Owner Author

Basically, I had introduced a conflict between two systems each fighting to command the next_state. The GameOver next_state handling was still running every frame in the absence of the player...

This would probably be better handled by a single event and a one off system, but I'm going to deliver as is and leave that for future refactoring (the one off systems are a new feature in Bevy 0.12 and I haven't played with them yet; not sure how stable or experimental they still are).

@thombruce thombruce marked this pull request as ready for review November 15, 2023 03:11
@thombruce thombruce merged commit 9dc800f into main Nov 15, 2023
@thombruce thombruce deleted the feat/game-over branch November 15, 2023 03:12
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.

Game Over screen (or Player death handling)
1 participant