Problème de sémantique HTML dans le composant Header

[humchaop]

Il y a une erreur de sémantique dans le composant Header.
Le texte du logo "Intitulé officiel" est dans une balise <p> inclus dans une balise <a>.
Les balises <p> n'étant pas autorisées dans les balises <a>, nous avons une anomalie d'accessibilité.

Il faudrait supprimer la balise <p> ou la remplacer par un <span>.

Exemple de code :
Composant header simple

Composant Header simple avec nom du service

[garronej]

Pourrais tu regarder si c'est le cas dans le template officiel?
https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/en-tete

Si c'est le cas, il faudrais traitée cette issue upstream avant

[humchaop]

C'est bien le cas. Quelle est la marche à suivre dans ce cas ?

[garronej]

Rejoindre le slack du DSFR et reporter le problème sur le canal #aide_dev.
Je vais le faire.

[garronej]

J'implémenterais ce qu'ils suggère

[enguerranws]

Ah, je n'avais pas vu la discussion et ai répondu sur l'issue du footer : #217 (comment)

[enguerranws]

Pour suivre plus simplement la discussion, je copie mon retour ici :

Inclure des balises block dans un élément était une erreur de validation en HTML / XHTML 4 et moins, ce n'est plus le cas en HTML5.
Ça ne pose (à ma connaissance), pas de souci d'accessibilité également : l'arbre d'accessibilité me semble clean (cf. screenshot), l'élément est bien accessible au clavier et la lecteur de texte fonctionne bien également.

@humchaop Je reste à dispo si tu constates un impact négatif, pour éventuellement faire remonter ça à l'équipe du DSFR (ici, nous ne faisons que suivre l'implem HTML / CSS qu'ils mettent en place).

[garronej]

@humchaop

[humchaop]

J'ai crée le ticket suite à un audit sur l'un des sites que l'on développe. Je vais voir avec l'auditeur ce qu'il en est.

[enguerranws]

@humchaop aussi, n'hésite pas à partager l'url de ton projet (si tu peux).

[humchaop]

@enguerranws je ne peux pas ce sont des projets internes du MEAE

[enguerranws]

@humchaop des nouvelles au sujet de cette demande ?

[enguerranws]

@humchaop s'il n'y a pas de nouvelles, je propose de clore le ticket.