Skip to content

A11Y: Ajout des attibuts aria sur tous les <i> remixicon#5601

Merged
hellodeloo merged 1 commit intomasterfrom
deloo/fix-attribute-aria-on-icons
Feb 12, 2025
Merged

A11Y: Ajout des attibuts aria sur tous les <i> remixicon#5601
hellodeloo merged 1 commit intomasterfrom
deloo/fix-attribute-aria-on-icons

Conversation

@hellodeloo
Copy link
Copy Markdown
Contributor

@hellodeloo hellodeloo commented Feb 12, 2025

🤔 Pourquoi ?

Pour corriger les erreurs d'accessibilité potentielles

🍰 Comment ?

En harmonisant les règles. A savoir :

Comme les <i class="ri-*></i> ne contiennent généralement pas de contenu texte et ne sont là que comme une aide visuelle, elles doivent être ignorées par les lecteurs d'écran. Pour cela, le plus simple est d'ajouter un aria-hidden="true" sur ces <i class="ri-*></i>

Cependant, parfois, les <i class="ri-*></i> se trouvent dans un <parent> qui ne contient ni texte, ni aria-label pour informer de son rôle/utilité/. Dans ce cas, il ne faut pas mettre d'aria-hidden=true" sur l'icone <i class="ri-*></i> mais un aria-label="xxx".

@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Feb 12, 2025
@hellodeloo hellodeloo self-assigned this Feb 12, 2025
@hellodeloo hellodeloo changed the title fix: attibute aria on icons A11Y: Ajout des aria attibuts sur toutes les <i> remixicon Feb 12, 2025
@hellodeloo hellodeloo changed the title A11Y: Ajout des aria attibuts sur toutes les <i> remixicon A11Y: Ajout des attibuts aria sur tous les <i> remixicon Feb 12, 2025
@notion-workspace
Copy link
Copy Markdown

🧑‍🦯 A11Y:

@hellodeloo hellodeloo added this pull request to the merge queue Feb 12, 2025
Merged via the queue into master with commit 90ae789 Feb 12, 2025
@hellodeloo hellodeloo deleted the deloo/fix-attribute-aria-on-icons branch February 12, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants