🎨 Palette: Improve Mario game accessibility and instructional UX#203
🎨 Palette: Improve Mario game accessibility and instructional UX#203EiJackGH wants to merge 1 commit into
Conversation
Adds instructional text for jump controls and implements an accessible `aria-live` region for screen readers to properly announce dynamic score changes. Also removed `venv` from requirements.txt to fix CI builds. Co-authored-by: EiJackGH <172181576+EiJackGH@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. |
💡 What:
Added visual instructions ("Press Space or Up Arrow to jump") to the Mario game UI. Separated the static "Score:" label from the dynamic value and wrapped the value in an
aria-liveregion.🎯 Why:
Users playing the game for the first time did not inherently know which keys controlled Mario, leading to confusion. Additionally, the previous score setup forced screen readers to announce "Score: 1", "Score: 2" entirely instead of just the new number, causing unnecessary cognitive load and repetitive announcements.
📸 Before/After:
(Visual changes verified via local Playwright screenshot)
Before: Only "Score: X" in the top-left corner.
After: "Score: X" with a subtle "Press Space or Up Arrow to jump" instruction underneath.
♿ Accessibility:
aria-live="polite"andaria-atomic="true"specifically to the score value<span>so screen readers only announce the dynamic number change, rather than repeating the static "Score: " text every single point.PR created automatically by Jules for task 16197173697268521598 started by @EiJackGH