-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Recherche #748
Comments
ProblèmeURLDescriptionQu'est-ce qui ne va pas ? TODO
ImpactDans quelle situation ceci est-il gênant ? En vocal ? Au clavier ? Quel handicap ? TODO Nature
Code problématiquePour les résultats : <li class="pagefind-ui__result">
<div class="pagefind-ui__result-thumb"><img class="pagefind-ui__result-image"
src="https://osuny-1b4da.kxcdn.com/mta2x08r53kehr77lxxjipl0kans?width=244&amp;height=0&&amp;fit=inside"
alt="Les pages"></div>
<div class="pagefind-ui__result-inner">
<p class="pagefind-ui__result-title"><a class="pagefind-ui__result-link"
href="/fr/blocks/blocs-de-liste/pages/">Les pages</a></p>
<p class="pagefind-ui__result-excerpt">Les pages. Exemple de <mark>block</mark> pages. Agrandir
l'image. Table des matières. Table</p>
</div>
</li> SolutionDescription
Code ciblePour le titre de la recherche : <div id="search" class="search__modal" aria-hidden="false" aria-modal="true" role="dialog" aria-label="Recherche">
<button class="search__close" aria-label="Fermer la recherche">
Fermer la recherche
</button>
<p role="heading" aria-level="1" class="sr-only">Rechercher</p>
<div class="pagefind-ui svelte-e9gkc3 pagefind-ui--reset">
[ ... ]
</div>
<div aria-live="polite" aria-atomic="true" class="sr-only pagefind-ui__accessible-message"></div>
</div> Pour les résultats : <li class="pagefind-ui__result">
<div class="pagefind-ui__result-thumb"><img class="pagefind-ui__result-image"
src="https://osuny-1b4da.kxcdn.com/mta2x08r53kehr77lxxjipl0kans?width=244&amp;height=0&amp;&amp;fit=inside"
alt=""></div>
<div class="pagefind-ui__result-inner">
<p class="pagefind-ui__result-title" role="heading" aria-level="2"><a
class="pagefind-ui__result-link" href="/fr/blocks/blocs-de-liste/pages/">Les pages</a></p>
<p class="pagefind-ui__result-excerpt">Les pages. Exemple de <mark>block</mark> pages. Agrandir
l'image Table des matières Table</p>
</div>
</li> Effet attenduEn quoi est-ce une amélioration pour les personnes impactées ? TODO SourceD'où vient cette recommandation ? TODO Exemple d'implémentationOù peut-on observer cette solution en production ? TODO |
@goetsu les points liés à la recherche ont déjà été remontés côté Pagefind. Ils l'ont pris en compte, et on a bon espoir que ça soit corrigé. |
Point 2 : ajouter un Il me semble que cela ajouterait un second H1 dans l'arborescence des titres ce qui semble être en contradiction avec les recommandations de la structuration pertinente des titres : https://accessibilite.numerique.gouv.fr/methode/criteres-et-tests/#9.1.1 |
Point 2 : cela créerait 2 h1, ce qui est déconseillé par MDN |
Point 1 : @goetsu quel est le critère qui demande ça ? |
Les points 4 et 5 dépendent de Pagefind |
@goetsu je ne sais pas si tu as vu nos commentaires ici ? |
point 1 critère 12.8 |
Merci pour le critère ! Pour le point 1 et 3 c'est corrigé. Pour le point 2 et 4 je comprends tout à fait le point de vue en mode lecteur d'écran. Mais je suis embêté car ça met le titre principal de la page au même niveau que "Rechercher" qui est présent sur toutes les pages, sémantiquement ça semble problématique ? Pour le point 5 je ne comprends pas pourquoi il faut des |
la modale est en display none dans le code donc encore une fois il n'y a pas deux h1 après si c'est pour des raisons SEO tu peux utiliser dans la modale pour le point 5 peut tu me dire comment les images sont récupérés ? c'est un contribution spécifique ? c'est la première image dans l'article ? |
Ok je crois que je comprends, le Pour le point 5 ce n'est pas une contribution spécifique à la recherche. C'est la même image que l'image de partage |
oui c'est ça pour le role et le aria-level pour les images exemple sur https://example.osuny.org/fr/actualites/2023-03-04-referentiel-general-decoconception-de-services-numeriques-rgesn/ du fait que l'image ouvre dans la visionneuse on va recommander au contributeur qu'elle dispose d'un alt décrivant l'image car dans le contexte visionneuse c'est nécessaire cf discussion ticket block image sauf qu'ici dans le contexte résultat recherche cela n'a aucune intéret de décrire l'image elle n'ouvre pas la visionneuse. Après si sur certains article on a une image réellement informative utilisée en hero d'un article sans visionneuse, le alt dans le résultat de recherche pourrait etre utile Je ne vois pas de solution réellement tout terrain mais vu l'usage ici en mode vignette réduite + le fait que le lien permet de toute façon d'accéder à l'article ou si besoin l'utilisateur va trouver l'image et son alt complet il est préférable de partir sur un alt="" en dur dans le template |
Ok je comprends mieux merci. Pour le |
On est bon pour ces 5 points sur example ! |
ok pour moi |
<p role="heading" aria-level="1" class="sr-only">Recherche</p>
après le bouton de fermeture de la modale (amélioration)<div aria-live="polite" aria-atomic="true" class="sr-only pagefind-ui__accessible-message">
(conformité)<h2>
(conformité)The text was updated successfully, but these errors were encountered: