Accessibility Bug - Duplicate Ids Created When More Than One Instance of the Tabs Is Used on a Page
Une pull request liée a déjà été fusionnée.
- #458 par @danez — fusionnée
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript, react
- Domaine
- accessibility, frontend
Piste de recherche
Commencez par reproduire le problème avec deux instances du contrôle react-tabs sur une page et inspectez les IDs des onglets et des panneaux d’onglet générés dans le DOM. Comparez les IDs en double et leurs associations, puis examinez les approches proposées dans l’issue ; le travail est terminé lorsque plusieurs instances ont des IDs uniques et restent correctement liées pour l’accessibilité.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
This is an accessibility bug caused by the same id being used multiple times in the DOM. What react-tabs does to cause this bug is it autogenerates ids for the tabs it renders to the page. The ids start at 0, and are incremented for each tab and tab-panel rendered. This wouldn’t be a problem if the control is only used once, but when it's used more than once in your project, the separate instances of the control do not know about each other and so the ids for each instance of the control start at 0 and get incremented in the same way, causing id overlap. Ways this issue could be resolved: use random ids, require an instance of the tabs control to be given an id and use that id in the ids of the autogenerated ids for the tabs and tab panels.
By the way, I have made sure that I am using the most up to date version of react-tabs as of the time of writing this, 3.1.0.
- Langage dominant
- JavaScript
- Étoiles
- 3.1k
- Forks
- 455
- Merge moyen
- 5 h 9 min
- PR mergées (30 j)
- 2
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de reactjs/react-tabs
-
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
reactjs/react-tabs#584 ·
-
className default is lost when props.className is undefined in UncontrolledTabsPeut-être pris @Shiva903-hue l’a pris il y a 267 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
reactjs/react-tabs#581 · 3 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
reactjs/react-tabs#571 ·
-
Support hiding of all TabPanelsOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
reactjs/react-tabs#525 · 1 commentaire · 1 réaction ·
-
Disabled tabs should remain keyboard focusable as per W3C WAI ARIA guidelinesPeut-être à nouveau libre @Jastagar l’a pris il y a 1048 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
reactjs/react-tabs#521 · 3 commentaires ·
Toutes les issues de reactjs/react-tabs
Issues similaires
-
documentation good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
bug release:v5.56
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Jason-Vaughan/TangleClaw#2270 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
style-dictionary/style-dictionary#1773 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
SignalK/signalk-server#3143 ·
Les mainteneurs répondent en général sous 1 jour