🎨 Palette: Add keyboard instructions and dynamic score accessibility#172
🎨 Palette: Add keyboard instructions and dynamic score accessibility#172EiJackGH wants to merge 1 commit into
Conversation
- Adds an overlay explaining keyboard controls ("Space or ↑ to Jump")
- Adds `aria-live="polite"` to the `#score` element for screen readers
- Cleans up `venv` from `requirements.txt` to fix pip install failures
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 a visible instructional overlay to the Mario game indicating the required keyboard controls, and added ARIA live attributes to the score display. Also fixed a dependency issue in
requirements.txt.🎯 Why
Custom keyboard bindings (like Space/Up Arrow to jump) have no native semantics or visual affordances, leaving users to guess the controls. Additionally, dynamic updates to text (like the score) are not announced by default by screen readers unless specifically marked. Finally, the inclusion of
venvinrequirements.txtwas causing automated build and local setup steps to fail.📸 Before/After
Before: The user is greeted by a blank screen and a stationary character with no indication of how to start playing.
After: A semi-transparent overlay explicitly states "Press Space or ↑ to Jump" in the upper right.
♿ Accessibility
aria-live="polite"andaria-atomic="true"to the#scorediv. This ensures that screen reader users are notified when the score updates dynamically.PR created automatically by Jules for task 6344300784578598795 started by @EiJackGH