-
Notifications
You must be signed in to change notification settings - Fork 3
Key Guide & Pause Menu #7
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
Conversation
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
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.
Pull request overview
This PR adds a "Key Guide" feature that displays control instructions to users on their first launch. The implementation includes a new KeyGuide scene that shows a background image with keyboard controls, updates to the main menu to add a "Key Guide" button, and logic to automatically display the guide on first launch based on the save data.
Key changes:
- New KeyGuide scene implementation with automatic first-launch display
- Main menu integration with new "Key Guide" button between "Settings" and "Exit"
- First launch detection using
completed_levels == 0check in save data
Reviewed changes
Copilot reviewed 10 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/scenes/key_guide.cpp | Implements new KeyGuide scene with event handling and rendering |
| include/wforge/scene.h | Adds KeyGuide struct declaration |
| src/scenes/main_menu.cpp | Adds KEYGUIDE button enum, button descriptor, rendering, and navigation |
| src/main.cpp | Adds first launch detection and conditional KeyGuide scene initialization |
| include/wforge/save.h | Adds is_first_launch() method checking if completed_levels == 0 |
| src/save.cpp | Fixed typo: "relay" corrected to "rely" |
| assets/ui/main-menu.json | Adds keyguide button configuration and adjusts exit button position |
| assets/ui/key-guide.json | Configuration file for KeyGuide scene dimensions |
| assets/ui/key-guide.png | Background image displaying keyboard controls |
| assets/manifest.json | Adds asset loading entries for key guide image and configuration |
| CMakeLists.txt | Adds key_guide.cpp to build sources |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
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.
Pull request overview
Copilot reviewed 11 out of 15 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
Signed-off-by: ZTL-UwU <zhangtianli2006@gmail.com>
No description provided.