Skip to content

🎨 Palette: Improve game responsiveness and visual feedback#18

Open
aidasofialily-cmd wants to merge 2 commits intomainfrom
palette-ux-improvement-10107469518797870887
Open

🎨 Palette: Improve game responsiveness and visual feedback#18
aidasofialily-cmd wants to merge 2 commits intomainfrom
palette-ux-improvement-10107469518797870887

Conversation

@aidasofialily-cmd
Copy link
Copy Markdown
Owner

💡 What:

  • Extracted UI rendering to a centralized render_ui function.
  • Added ANSI color codes (Green for score, Bold Red for Hard Mode) and bold text.
  • Replaced the busy-loop sleep_for with a dynamic poll() timeout for instant responsiveness.
  • Improved the "Game Over" screen with a clear banner.
  • Fixed Hard Mode toggle to update in-place instead of printing new lines.

🎯 Why:

  • The game felt unresponsive because the score only updated on the 1-second tick, even if the user clicked.
  • The interface was monochrome and cluttered when toggling modes.
  • The busy loop was inefficient.

♿ Accessibility:

  • Increased visual contrast using bold and colored text to highlight important game states (Score and Mode).
  • Standardized the UI to a single-line update, making it easier for users to track changes without scanning multiple lines of scrolling text.

PR created automatically by Jules for task 10107469518797870887 started by @aidasofialily-cmd

- Centralize UI rendering to provide immediate tactile feedback on input.
- Use ANSI colors (Green/Red) and bold text for better visual clarity.
- Optimize the game loop using a dynamic poll() timeout for better responsiveness and CPU efficiency.
- Ensure the Hard Mode toggle updates in-place without breaking the single-line UI.
- Add a clear 'Game Over' banner.

Co-authored-by: aidasofialily-cmd <247843425+aidasofialily-cmd@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

UX improvements:
- Centralize UI rendering to provide immediate tactile feedback on input.
- Use ANSI colors (Green/Red) and bold text for better visual clarity.
- Optimize the game loop using a dynamic poll() timeout for better responsiveness.
- Ensure the Hard Mode toggle updates in-place without breaking the single-line UI.
- Add a clear 'Game Over' banner.

CI fix:
- Replace incorrect Rust workflow with a C++ Build workflow using 'make'.
- Add a basic smoke test to the CI pipeline.

Co-authored-by: aidasofialily-cmd <247843425+aidasofialily-cmd@users.noreply.github.com>
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.

1 participant