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

[FEATURE] Mise à jour du wording lors du défocus pour candidats avec besoins d'ajustements (PIX-14428). #10258

Merged
merged 12 commits into from
Oct 9, 2024

Conversation

alexandrecoin
Copy link
Contributor

@alexandrecoin alexandrecoin commented Oct 7, 2024

🦄 Problème

Dans le cadre de l’epix a11y , une version “accessible” des focus va être implémentée.

Si on permet à un candidat ayant un test aménagé a11y de valider une question focus bien répondue, même en cas de défocus, le message en cas de défocus actuel est trompeur.

🤖 Proposition

Mise à à jour du wording des warning des questions focus pour un test aménagé a11y : pour un test de certification v3, pour lequel la case “aménagement a11y a été coché” :


Lorsque le candidat sort de la zone de question

image

Nouveau wording :
🇫🇷 “Répondez à cette question sans vous aider d’internet ou d’applications.”
🇬🇧 “Answer this question without using the Internet or any applications.”


Lorsque le candidat défocus de l'épreuve

image

Nouveau wording :
🇫🇷 “Nous avons détecté un changement de page. Si vous avez été contraint de changer de page pour utiliser un outil d’accessibilité numérique (tel qu'un lecteur d'écran ou un clavier virtuel), répondez tout de même à la question.”
🇬🇧 “We have detected a page switch. If you have been forced to change pages in order to use a digital accessibility tool (such as a screen reader or virtual keyboard), please answer the question anyways.”

🌈 Remarques

On décide de renvoyer undefined dans le payload du serializer pour le mois à venir avant la généralisation.

💯 Pour tester

  • Créer une session de certification V3 dans un centre featurisé a11y ([email protected])

  • Ajouter deux candidats à cette session

  • Modifier les infos d'un des candidats afin d'y ajouter un besoin d'ajustement

  • Passer les sessions des deux candidats et vérifier que les messages de défocus lors des épreuves focus correspondent aux différents besoins d'ajustements des candidats

  • Créer une session V2 et y ajouter un candidat

  • Passer le test de certification et vérifier que les messages de défocus lors des épreuves focus n'ont pas changé

@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 :

@alexandrecoin alexandrecoin force-pushed the pix-14428-update-unfocused-messages-in-certif branch 7 times, most recently from 3c69d45 to a666f5e Compare October 8, 2024 04:05
@alexandrecoin alexandrecoin marked this pull request as ready for review October 8, 2024 06:55
@alexandrecoin alexandrecoin requested a review from a team as a code owner October 8, 2024 06:55
@alexandrecoin alexandrecoin force-pushed the pix-14428-update-unfocused-messages-in-certif branch from a666f5e to 643a484 Compare October 8, 2024 06:56
@mcampourcy
Copy link
Contributor

Test func OK ✅

Sans a11y Avec a11y
Capture d’écran 2024-10-08 à 10 06 50 Capture d’écran 2024-10-08 à 10 09 08

@mcampourcy mcampourcy added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Oct 8, 2024
@alexandrecoin
Copy link
Contributor Author

Candidat V2
Capture d’écran 2024-10-09 à 09 16 43
Capture d’écran 2024-10-09 à 09 16 51
Capture d’écran 2024-10-09 à 09 17 03

@alexandrecoin
Copy link
Contributor Author

Candidat V3 SANS besoin d'ajustement
Capture d’écran 2024-10-09 à 09 20 47
Capture d’écran 2024-10-09 à 09 20 52
Capture d’écran 2024-10-09 à 09 21 02

@alexandrecoin
Copy link
Contributor Author

Candidat V3 AVEC besoin d'ajustement
Capture d’écran 2024-10-09 à 09 19 52
Capture d’écran 2024-10-09 à 09 19 58
Capture d’écran 2024-10-09 à 09 20 05

@alexandrecoin alexandrecoin force-pushed the pix-14428-update-unfocused-messages-in-certif branch from 643a484 to ad2f3b4 Compare October 9, 2024 09:08
@alexandrecoin alexandrecoin enabled auto-merge (squash) October 9, 2024 09:09
@alexandrecoin alexandrecoin force-pushed the pix-14428-update-unfocused-messages-in-certif branch from ad2f3b4 to c09e026 Compare October 9, 2024 09:33
@alexandrecoin alexandrecoin force-pushed the pix-14428-update-unfocused-messages-in-certif branch from c09e026 to eafbce3 Compare October 9, 2024 09:36
@alexandrecoin alexandrecoin merged commit 58c6592 into dev Oct 9, 2024
7 of 8 checks passed
@alexandrecoin alexandrecoin deleted the pix-14428-update-unfocused-messages-in-certif branch October 9, 2024 09:41
dianeCdrPix pushed a commit that referenced this pull request Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants