Scribouilli / mimoza

Le thème par défaut pour Scribouilli
MIT License
1 stars 7 forks source link

[Accessibilité] Lien vide dans le menu #14

Open juliemoynatPro opened 10 months ago

juliemoynatPro commented 10 months ago

Bonjour,

Il y a un lien vide dans le menu :

<li>
          <a href="/images/"></a>
</li>

Les personnes qui naviguent au clavier peuvent l'atteindre et les personnes aveugles auront l'information (via lecteur d'écran ou plage Braille) que ce lien est vide (ou bien l'URL sera restituée) et ne sauront pas à quoi il sert.

Cela ressemble à une erreur étant donné la page où ce lien mène mais qui a des conséquences sur l'accessibilité. Si c'est bien une erreur, il faudrait retirer ce lien. Sinon, il faudrait lui donner un intitulé visible (et probablement le placer ailleurs, à un endroit plus pertinent ?).

maiwann commented 10 months ago

Merci pour ce retour, on a conscience de ce bug mais on ne l'avait pas priorisé car on pensait qu'il était seulement esthétique. Que tu ai indiqué qu'il posait problème pour l'accessibilité nous fait le re-prioriser. Merci à toi.

(Note : j'ai déplacé l'issue dans le dépôt dédié au thème de Scribouilli car c'est ici que le souci va se régler)

DavidBruant commented 8 months ago

On a trouvé le problème et travaillé sur la correction ici : https://github.com/Scribouilli/mimoza/commit/ff6cbee067dff03a1947755f6261604bae6ecafc

En gros, la variable site.pages de Jekyll contient tous les fichiers (et pas seulement les "pages" comme son nom l'indique). Et donc, on trouve notamment robots.txt ou redirects.json et images/, donc tout un tas de choses qui n'ont pas de vraie url ou de .title à afficher

Par contre, la variable site.html_pages semble ne lister que les "vraies pages web" (générées à partir de .md ou de .html) et donc, ça fait disparaître les liens qu'on n'avait pas envie de voir (et les espaces entre ces liens)

Dans https://github.com/Scribouilli/mimoza/pull/15 , on a corrigé d'autres trucs liés au header (notamment un menu burger pour les écrans de petite largeur). Ce bug sera corrigé quand on aura fini cette PR