Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A11Y: Amélioration de l'accessibilité des tooltip #5380

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented Jan 13, 2025

🤔 Pourquoi ?

Les tooltips ne sont pas toujours activables à la tabulation

🍰 Comment ?

  • En remontant le data-bs-toggle="tooltip" sur les <a> qui enroberaient un <i data-bs-toggle="tooltip">
  • En ajoutant sur un role="button" et tabindex="0" sur les <span>, <i> et <div> possédant un data-bs-toggle="tooltip"
  • En remplacant les title="blabla..." par des data-bs-title="blabla..." afin que le js de BS puisse correctement associer (avec une aria-labelledby="") les data-bs-toggle="tooltip" avec le <div> contenant le texte du data-bs-title lors de l'activation du tooltip

💻 Captures d'écran

Avant

capture 2025-01-13 à 10 48 41
capture 2025-01-09 à 17 12 33

Apres

capture 2025-01-13 à 10 48 50
capture 2025-01-13 à 15 20 52

@francoisfreitag
Copy link
Contributor

francoisfreitag commented Jan 13, 2025

En ajoutant sur un role="button" et tabindex="0" sur les <span>, <i> et <div> possédant un data-bs-toggle="tooltip"

La doc de bootstrap dit:

Keep tooltips accessible to keyboard and assistive technology users by only adding them to HTML elements that are traditionally keyboard-focusable and interactive (such as links or form controls). While other HTML elements can be made focusable by adding tabindex="0", this can create annoying and confusing tab stops on non-interactive elements for keyboard users, and most assistive technologies currently do not announce tooltips in this situation.

Est-ce qu’on ne devrait pas changer le markup pour suivre ces recommendations ?

@hellodeloo hellodeloo force-pushed the deloo/fix-tooltip-a11y branch from 26100be to e20826a Compare January 13, 2025 15:29
@hellodeloo
Copy link
Contributor Author

Est-ce qu’on ne devrait pas changer le markup pour suivre ces recommendations ?

Pour les <span> et les <div>, oui, tu as raison, je vais remplacer par des <button> 👍
Pour les <i>, je vais voir si je peux en enrober certains dans un <button> mais ça ne sera surement pas possible pour tous

@hellodeloo hellodeloo force-pushed the deloo/fix-tooltip-a11y branch from 29b2a62 to 3d03510 Compare January 14, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants