airbnb/rheostat

Aria-labels can't be added to slider buttons

Open

#273 ouverte le 3 janv. 2020

Voir sur GitHub
 (3 commentaires) (9 réactions) (0 assignés)JavaScript (1 688 stars) (215 forks)batch import
enhancementhelp wanted

Description

For compliance, I need to add aria-labels to the slider buttons in an implementation like this one: http://localhost:9001/?path=/story/slider--rtl-multiple-handle. There doesn't seem to be a way to do this currently.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontendaccessibility
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic React knowledgeunderstanding of ARIA attributes
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The issue is to add aria labels to slider buttons in the rheostat component. Start by examining the slider component source code in the repository to identify where buttons are rendered. Look for existing prop handling for accessibility attributes. Check comments on the issue for any suggested approach. Consider adding a prop like `buttonAriaLabel` that can be passed to each handle. Ensure to update the component's TypeScript definitions if present. Test the change using the existing test suite and manual verification.