empreinte-digitale / assistant-rgaa

Extension de navigateur permettant d'auditer des pages web en suivant le référentiel RGAA.
https://assistant-rgaa.empreintedigitale.fr/
MIT License
31 stars 0 forks source link

Gestion des titres #18

Closed luce-carevic closed 3 years ago

luce-carevic commented 3 years ago

Titre h1 manquant

Comportement actuel :
L'assistant remonte en erreur une arborescence qui ne commence pas par un titre <h1>.

Comportement attendu : Il n'est pas obligatoire de commencer une hiérarchie de titres par un titre <h1>. Cela ne doit donc pas remonter en erreur.

Role heading et aria-level

Sur les titres, l'assistant ne gère pas les cas suivants :

2021-07-22-13 23 05@2x

2021-07-22-13 24 58@2x

Comportement actuel :

Comportement attendu

felixgirault commented 3 years ago

Merci pour ce retour détaillé ! Un dernier cas me pose question pour bien couvrir toutes les possibilités :

<h3 role="heading">Titre</h3>

Ce serait considéré comme un niveau 3 ? Ou plutôt 2 en supposant que l'attribut rôle écrase la sémantique initiale du h3 ? Auquel cas ce serait équivalent à div[role="heading"], donc 2 par défaut.

felixgirault commented 3 years ago

Il semble aussi qu'actuellement des balises comme <span aria-level="1"> soient considérées comme des titres :thinking:

luce-carevic commented 3 years ago

Ça reste bien un titre de niveau 3, car le h3 a déjà une sémantique de titre et donc le role="heading" seul n'écrase pas le niveau.

Je peux pas vérifier sur toute la base de réf actuellement la restitution, mais ça devrait logiquement être bon.

felixgirault commented 3 years ago

Ça marche merci.

Concernant mon commentaire https://github.com/empreinte-digitale/assistant-rgaa/issues/18#issuecomment-884882919, je me suis fait avoir par la présentation. Lorsqu'on active le test 9.1.1 on affiche pas les attributs role. Il serait peut-être judicieux de les afficher ?

Capture d’écran_2021-07-22_14-53-25

luce-carevic commented 3 years ago

Ça marche merci.

Concernant mon commentaire #18 (comment), je me suis fait avoir par la présentation. Lorsqu'on active le test 9.1.1 on affiche pas les attributs role. Il serait peut-être judicieux de les afficher ?

Capture d’écran_2021-07-22_14-53-25

Oui, je pense !