🎨 Palette: Improve game responsiveness and UI cleanliness#15
🎨 Palette: Improve game responsiveness and UI cleanliness#15aidasofialily-cmd wants to merge 1 commit intomainfrom
Conversation
- Refactor game loop in src/main.cpp to use poll() with dynamic timeout. - Trigger immediate display updates on user input for better tactile feedback. - Clean up terminal output by integrating mode status into the score line. - Fix broken Rust CI by replacing it with a C++ workflow using make. - Add 'test' target to Makefile for automated verification. Co-authored-by: aidasofialily-cmd <247843425+aidasofialily-cmd@users.noreply.github.com>
|
👋 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 New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
💡 What: Refactored the game loop to use
poll()with dynamic timeout and immediate display updates. Also fixed the CI configuration and added atesttarget.🎯 Why: The game felt unresponsive because scores only updated on the timer tick. The UI was also cluttered by mode toggle messages.
♿ Accessibility: Improved tactile feedback for keyboard users by providing immediate visual confirmation of keypresses.
📸 Before/After:
PR created automatically by Jules for task 15783760359809981948 started by @aidasofialily-cmd