Skip to content

🎨 Palette: Polish CLI UX with score formatting and clean UI updates#133

Open
aidasofialily-cmd wants to merge 1 commit intomainfrom
palette-ux-polish-17684848214439175307
Open

🎨 Palette: Polish CLI UX with score formatting and clean UI updates#133
aidasofialily-cmd wants to merge 1 commit intomainfrom
palette-ux-polish-17684848214439175307

Conversation

@aidasofialily-cmd
Copy link
Copy Markdown
Owner

This PR introduces several micro-UX enhancements to the Speed Clicker terminal game to improve readability and visual polish.

💡 What: The UX enhancement added

  • Score Readability: Added a formatWithCommas helper to provide thousands separators for all score displays (Personal Best, live HUD, and Final Score).
  • Clean UI Updates: Replaced manual space padding in the "GO!" prompt and gameplay HUD with the ANSI "Erase in Line" (\033[K) sequence, ensuring a clean terminal state regardless of string length.
  • Achievement Context: Modified the post-game screen to display the previous personal best when a new record is set, providing meaningful context for the user's success.
  • Macro Consolidation: Removed the redundant RESET macro in favor of the standardized CLR_RESET.

🎯 Why: The user problem it solves

  • Large numeric scores were difficult to parse at a glance.
  • Manual padding can be brittle and leave trailing characters if strings shorten (though unlikely here, it's a best-practice for CLI tools).
  • Setting a new best felt slightly isolated without knowing the previous target that was surpassed.

♿ Accessibility: Any a11y improvements made

  • Improved cognitive accessibility by making large numbers easier to read.
  • Used standardized ANSI sequences for terminal control, ensuring better compatibility with various terminal emulators.

Verified with make and verify_ux.py.


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

- Implemented `formatWithCommas` helper for thousands separators in score displays.
- Replaced brittle manual space padding with ANSI 'Erase in Line' (\033[K) for robust HUD updates.
- Added previous personal best context to the final achievement screen.
- Consolidated terminal color macros and ensured consistent CLR_RESET usage.
- Verified cursor management and UI transitions via verify_ux.py.

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.

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