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

[TECH] Rendre accessible les écrans d'instruction (PIX-11847) #9465

Merged

Conversation

P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Jul 8, 2024

🦄 Problème

Des nouveaux écrans d’instruction ont été implémentés pour les candidats. Il s’agit ici d'en améliorer l’accessibilité.

🤖 Proposition

  • Vocaliser les étapes dans les titres
  • Rendre visuelle la position dans les écran autrement que par la couleur

💯 Pour tester

  • Se connecter à mon-pix en tant que candidat v3
  • Rejoindre une session de certification
  • Activer le lecteur d'écran et constater que la position "Page n sur 5" est vocalisée après le titre de la page
  • Constater que le focus est remis sur le titre après avoir cliqué sur "continuer"
  • Constater que le point actif sur le caroussel est plus petit que les autres

image

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@P-Jeremy P-Jeremy marked this pull request as ready for review July 8, 2024 13:27
@P-Jeremy P-Jeremy force-pushed the pix-11847-improve-a11y-certification-instructions branch from 9161072 to f418a0e Compare July 8, 2024 14:45
@P-Jeremy P-Jeremy force-pushed the pix-11847-improve-a11y-certification-instructions branch 3 times, most recently from 21a689c to b2daf31 Compare July 12, 2024 12:16
@P-Jeremy P-Jeremy requested review from theotime2005 and a team July 12, 2024 12:17
Copy link
Contributor

@mcampourcy mcampourcy left a comment

Choose a reason for hiding this comment

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

👌

@P-Jeremy P-Jeremy force-pushed the pix-11847-improve-a11y-certification-instructions branch from b2daf31 to b6f912b Compare July 12, 2024 13:04
Copy link
Contributor

@theotime2005 theotime2005 left a comment

Choose a reason for hiding this comment

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

Review fonctionnelle!!!

@P-Jeremy P-Jeremy added the Func Review OK PO validated functionally the PR label Jul 12, 2024
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-11847-improve-a11y-certification-instructions branch from 9610ea2 to b3ee8f6 Compare July 12, 2024 14:02
@pix-service-auto-merge pix-service-auto-merge merged commit af4850a into dev Jul 12, 2024
6 of 7 checks passed
@@ -1,4 +1,8 @@
<div class="instructions-content">
<h2 {{did-insert @focus}} tabindex="0" class="instructions-step__title">{{@title}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh ! Bien trouvé le did-insert 👍 j'ai découvert quelque chose :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants