-
Notifications
You must be signed in to change notification settings - Fork 27
#36 #146
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
base: main
Are you sure you want to change the base?
#36 #146
Conversation
Signed-off-by: Matt <mjcantin@yahoo.com>
Signed-off-by: Matt <mjcantin@yahoo.com>
Signed-off-by: Matt <mjcantin@yahoo.com>
Fixed checkstyle errors
Removed .idea directory from gitignore. Signed-off-by: Matt <mjcantin@yahoo.com>
…asciititle Revert "Fixed checkstyle errors"
Added line to display the number of remaining guesses in Jotto
user story 26
Removed a line that prints the number of guesses remaining. Signed-off-by: Matt <mjcantin@yahoo.com>
add borders to launcher menu MetroCS#36
Signed-off-by: agina2004 <72883535+agina2004@users.noreply.github.com>
jody
left a comment
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.
Please replace the title with a meaningful statement of what this PR does.
There are 23 Commits, many of which appear to be irrelevant to the issue. Please clean up this PR.
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 enhances the visual presentation of the game launcher menu by adding a border around the menu options, creating a more structured and polished UI.
- Added top and bottom border lines to frame the menu
- Prefixed all menu items with a consistent left border character
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| System.out.println("_____________________________"); | ||
| for (int i = 0; i < this.games.size(); i++) { | ||
| System.out.printf("%d. %s\n", | ||
| System.out.printf("| %d. %s\n", |
Copilot
AI
Nov 1, 2025
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.
The dynamically generated game menu items use a left border '|' prefix, but the right border is missing. This creates an asymmetric box that doesn't properly close on the right side. Consider adding the right border by modifying the format string to calculate padding and include a closing '|' character, ensuring all menu items have consistent width and proper borders on both sides.
Added borders to launcher menu
#36
closed # 36