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

Print the keybindings used in the demo #73

Merged
merged 3 commits into from
Nov 15, 2024
Merged

Conversation

ll-nick
Copy link
Collaborator

@ll-nick ll-nick commented Nov 15, 2024

Closes #57

This adds a simple std::cout when launching the demo to print the keybindings to the terminal.

image

@ll-nick ll-nick requested a review from orzechow November 15, 2024 07:36
@ll-nick ll-nick self-assigned this Nov 15, 2024
@ll-nick
Copy link
Collaborator Author

ll-nick commented Nov 15, 2024

@orzechow I'm not great with colors, so feel free to adjust those ;-)

Copy link
Member

@orzechow orzechow left a comment

Choose a reason for hiding this comment

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

Great!

When running via docker compose up the keybindings had been printed after quitting the demo 😅
See ae89071 for making sure to flush the stream at least once.

@ll-nick
Copy link
Collaborator Author

ll-nick commented Nov 15, 2024

Nice catch, thanks for the fix :)

@ll-nick ll-nick merged commit 8e86842 into main Nov 15, 2024
1 check passed
@ll-nick ll-nick deleted the print_demo_keybindings branch November 15, 2024 13:24
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.

Explain PacMan GUI keybindings
2 participants