Skip to content
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

Add tag difficulty #246

Merged
merged 4 commits into from
Nov 18, 2021
Merged

Add tag difficulty #246

merged 4 commits into from
Nov 18, 2021

Conversation

SimonF30
Copy link
Contributor

@Aniort @PigeardSylvain

Pr pour l'ajout des tags

  • modification sur la gestion des titres h1 pour les articles qui possèdes des tags
  • ajout des tags difficultés
  • modification des classes badges qui ne prennent pas le focus

A vérifier :

  • Voir si la vocalisation des tags est bonne ( masquage accessible ? )
  • Vérifier les différents niveaux de difficultés que j'ai mis sur les articles

@SimonF30 SimonF30 self-assigned this Oct 22, 2021
@netlify
Copy link

netlify bot commented Oct 22, 2021

✔️ Deploy Preview for a11y-guidelines-orange ready!

🔨 Explore the source changes: 08ac876

🔍 Inspect the deploy log: https://app.netlify.com/sites/a11y-guidelines-orange/deploys/6184f0a2fb563100075197a4

😎 Browse the preview: https://deploy-preview-246--a11y-guidelines-orange.netlify.app

@PigeardSylvain
Copy link
Member

Dans la liste des articles, pour chaque article, on a le titre, la description et les tags.
A cet endroit, les tags sont vocalisés "filtre" + nom du tag. Je verrai bien "thématique associée" (en masquage accessible) + liste des tag comme dans l'en-tête d'un article.

@SimonF30
Copy link
Contributor Author

SimonF30 commented Nov 4, 2021

@PigeardSylvain @Aniort

j'ai essayé d'améliorer la vocalisation, plusieurs choses

Dans la liste des articles :

  • J'ai mis les badges juste en dessous des titres
  • Renommer "Filtres" en "Filtrer par thématiques"
  • Enlever le titre de la navigation sinon le aria-labelledby n'est pas prononcé à la tabulation Landmarks with aria-labelledby tested positive but actually does not work Access4all/adg#287
  • Rajouter un aria-labelledby sur la liste des tags pour que "Thématiques associés" soit prononcé à la tabulation (en train de me dire si c'est pas mieux de mettre aria-describedby)
  • J'ai enlevé tout les masquages accessibles (""filtre" + nom du tag") ça polluaient énormément la vocalisation

voilà c'est à peu près tout, j’attends vos retours

@PigeardSylvain
Copy link
Member

Dans les articles je mettrais bien le texte "thématique associée" en masquage accessible et j'ajouterai la date pour faire un semblant d'en-tête :
image

Copy link
Collaborator

@Aniort Aniort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok avec les remarques de @PigeardSylvain
sinon le reste c bien impec !

@SimonF30
Copy link
Contributor Author

SimonF30 commented Nov 5, 2021

j'ai fait l'update

@SimonF30 SimonF30 requested a review from Aniort November 10, 2021 09:29
@SimonF30
Copy link
Contributor Author

@Aniort @PigeardSylvain vous pouvez check?
j'ai un nouvel article à écrire, ça permettra de partir sur les mêmes bases

@SimonF30 SimonF30 merged commit 1c042c7 into master Nov 18, 2021
@SimonF30 SimonF30 deleted the add-tag-difficulty branch November 18, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants