🎨 Palette: Mario Game UX Accessibility & Hints#150
Conversation
…hints to the Mario game UI. Removed invalid venv entry from requirements.txt to fix CI. 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. |
|
Stale pull request message |
💡 What: Added
aria-live="polite"andaria-atomic="true"to the dynamic score counter and introduced a semi-transparent, floating keyboard controls hint ("Press Space or Up Arrow to jump") underneath the score in the Mario game. Also fixed a CI failure by stripping the invalid standard libraryvenvmodule fromrequirements.txt.🎯 Why: The score was dynamically updating but completely inaccessible to screen readers. Furthermore, players landing on the page had no way to know the game controls other than guessing or reading the source code. This small hint reduces cognitive friction, and the aria updates make the widget inclusive.
📸 Before/After: The game now displays a "Press Space or Up Arrow to jump" hint below the score on page load. Playwright screenshots show it perfectly overlays the game background without obstructing the gameplay area.
♿ Accessibility: Added ARIA live regions to the score counter. The controls hint was marked with
aria-hidden="true"as visual help, ensuring it doesn't clutter screen reader announcements where basic interaction might already be implied or read from surrounding context.PR created automatically by Jules for task 790433205743827776 started by @EiJackGH