osunyorg / theme

Le thème Hugo d'Osuny
https://osuny.org
MIT License
9 stars 5 forks source link

Bloc tableau : remplacer <caption> par <summary> #578

Open Clararigaud opened 3 weeks ago

Clararigaud commented 3 weeks ago

5.5 - Majeur : Présence d’un titre de tableau peu pertinent, le contenu de la balise caption reprend l'intégralité des informations données dans le tableau

Rendre le titre du tableau plus pertinent en identifiant le contenu du tableau de façon claire et concise

Clararigaud commented 3 weeks ago

En fait @Olivia206 @arnaudlevy je crois que c'est un probleme de contenu et non de balisage > https://www.w3.org/WAI/tutorials/tables/caption-summary/

Dans le commentaire , le problème évoqué est que la description repete le contenu du tableau. Peut etre juste supprimer les détails ou en faire un paragraphe (chapitre) en dessous

Clararigaud commented 3 weeks ago

Image

Par contre ce qui va dans caption est considéré comme transcription dans l'admin. Ca c'est un probleme

Olivia206 commented 3 weeks ago

On avait fait ça en suivant le RGAA :

Capture d’écran 2024-08-21 à 16 21 12 Capture d’écran 2024-08-21 à 16 21 27

(ici ça dit bien que caption est ok, c'est à n'y rien comprendre)

Clararigaud commented 3 weeks ago

De ce que je comprends il y a deux choses différentes: la caption qui est un descriptif du tableau genre "les colonnes indiquent X, les lignes indiquent Y"

Et la transcription qui est un paragraphe complet qui explique ce que le tableau nous apprend.

Il faut ajouter un champs caption dans l'admin qui arra dans la balise caption. Et que ce qui est actullement transcription textuelle aille dans un paragraphe distinct

Clararigaud commented 3 weeks ago

On peut peut etre mettre ca dans la liste des chose à discutter demain :)

Olivia206 commented 3 weeks ago

J'ai cru comprendre du rapport d'audit que <caption> doit identifier le titre, mais on pourra voir demain, oui

Clararigaud commented 2 weeks ago

@Olivia206 on laisse tomber ça du coup ?

Olivia206 commented 2 weeks ago

En partie ! Si on part du principe qu'on n'aura pas de tableau de données complexes, il faut que <caption> correspondre au titre du tableau (donc changements à prévoir en admin (pour enlever le "transcription" et rajouter un court champ de texte) et ajustement dans le thème comme on ne reçoit pas la même donnée (à vérifier)) Tu en penses quoi ?

Clararigaud commented 2 weeks ago

Je pensais à ca aussi. Dans le theme il n'y a rien de sémantique ( juste une balise ) donc changer dans l'admin "transcription" en "titre" ou "caption" devrait suffire pour l'a11y

Clararigaud commented 2 weeks ago

Ticket correspondant en admin > https://github.com/osunyorg/admin/issues/2154