🎨 Palette: Rating reset and accessibility improvements#55
Conversation
- Added 'clearRating' translation key for EN and FR - Implemented a 'Clear Rating' button in GameDetailHeader to reset personal ratings - Improved accessibility by linking labels to platform and rating inputs - Added aria-hidden to decorative range labels - Fixed build-blocking unused variable in GameScreenshotsCarousel Co-authored-by: Gamepulse <8333979+Gamepulse@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
This PR introduces a small but meaningful UX improvement by allowing users to reset their personal rating for a game, which was previously impossible once a value was set on the range input.
Additionally, it enhances the accessibility of the
GameDetailHeadercomponent by:idandhtmlFor.aria-hidden="true"to decorative "0" and "100" text labels next to the rating slider to reduce screen reader noise.aria-labelandtitleattributes for the new 'Clear Rating' button.A minor build-blocking TypeScript error in
GameScreenshotsCarousel.tsx(unusedhasIgdbIdstate) was also resolved to ensure a clean production build.PR created automatically by Jules for task 9371840659939791655 started by @Gamepulse