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

Accessibilité - Deuxième vague #807

Merged
merged 48 commits into from
Sep 17, 2024
Merged

Accessibilité - Deuxième vague #807

merged 48 commits into from
Sep 17, 2024

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Aug 21, 2024

Description succincte du problème résolu

Modification d'accessibilité

Cartes traitées :

Type de changement :

  • Bug fix
  • Nouvelle fonctionnalité
  • Refactoring de code (explication à retrouver dans la description)

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

Comment tester

La procédure de test est détaillée en commentaire de chaque carte Notion

⚠️ Régressions à traiter avant le merge de la PR

  • Transitions du panel de filtres avancés - J'ai supprimé quelques overflow-hidden qui avaient des effets de bord indésirable sur les formulaires, je vois plus tard comment rétablir ça
  • La carte est cachée par un qfdmo-invisible
  • Résultats en ligne pas scrollables
  • Les filtres avancés s'affichent lorsqu'on passe de la vue en ligne / à proximité
  • Recherche > retour à la recherche > problème d'affichage

Déploiement

Après la MEP, créer des issues au DSFR concernant

  • Contrôle segmenté pas accessible (role="tab" et tablist)
  • Rôles facultatifs sur main / header / footer

@fabienheureux fabienheureux force-pushed the a11y-feedbacks-lucas-batch-2 branch 10 times, most recently from 31777df to e4c392a Compare August 22, 2024 14:16
@fabienheureux fabienheureux changed the title A11y feedbacks lucas batch 2 Accessibilité - Deuxième vague Aug 22, 2024
@fabienheureux fabienheureux marked this pull request as ready for review August 22, 2024 14:48
@fabienheureux fabienheureux requested a review from a team as a code owner August 22, 2024 14:48
@fabienheureux fabienheureux requested review from kolok and Hazelmat and removed request for a team August 22, 2024 14:48
@fabienheureux fabienheureux force-pushed the a11y-feedbacks-lucas-batch-2 branch 6 times, most recently from 26307e4 to b3aff83 Compare August 22, 2024 15:59
@fabienheureux
Copy link
Member Author

@kolok c'est de nouveau dispo en review.
J'ai testé pas mal de comportement de ça me semble ok mais je veux bien ton regard ! (sur le fonctionnel et pourquoi pas une dernière relecture du code)

@kolok
Copy link
Contributor

kolok commented Sep 16, 2024

J'ai du mal à retrouver comment ça marchait avant car on avait la scrollbar sur le formulaire en prod et ça marchait correctement

Maintenant, on n'a plus la scrollbar ni en prod ni dans cette PR. cela pose un problème aussi quand la hauteur par défaut est utilisé dans la nouvelle version car le en cas d'erreur sur les champs, le bonton rechercher n'est plus visible :

CleanShot 2024-09-16 at 08 38 59@2x

@kolok
Copy link
Contributor

kolok commented Sep 16, 2024

Les tests e2e passe en local sur mon PC, ici il merdouille, je ne sais pas pourquoi

@fabienheureux
Copy link
Member Author

C'est good ils passent !

Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

👍

@kolok kolok merged commit e0912b7 into main Sep 17, 2024
12 checks passed
@kolok kolok deleted the a11y-feedbacks-lucas-batch-2 branch September 17, 2024 06:25
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.

2 participants