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] Ajouter un composant local de gestion des règles de mot de passe (PIX-14149) #10259

Merged

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Oct 7, 2024

🦄 Problème

Le component input avec mot de passe, utilisé sur les pages d'inscription, n'est pas complètement accessible. On s'est aperçu que:

  • la progression n'est pas clairement énoncée lorsque l'utilisateur saisit un mot de passe
  • les règles ne sont présentes uniquement dans le label.

🤖 Proposition

  • Ajouter un component PasswordCheckListInput permettant de gérer ces cas en s'inspirant de ce qui a été fait sur le codepen fourni par Tanaguru.

🌈 Remarques

  • Le message d'erreur, lors de la sortie d'input, n'est pas vocalisé par le lecteur d'écran. Il manque sans doute un aria-live=polite sur le message d'erreur sur le composant pix-ui PixInputPassword (A faire dans un autre ticket).
  • Le dernier commit est à retirer lors du merge de la PR !

💯 Pour tester

Cas nominal

  • Aller sur la page /connexion
  • Tester chaque règle du composant (saisir un chiffre, une minuscule, une majuscule, 8 caractères)
  • Vérifier que le picto correspondant à la règle en question se met bien à jour lors de la saisie dans le champ

Cas d'erreur

  • Enlever des caractères dans votre saisie
  • Vérifier que le picto correspondant à la règle en question se met bien à jour
  • Sortir du focus du champ en cliquant par exemple à côté.
  • Vérifier que le message d'erreur Le mot de passe saisi est erroné. Votre mot de passe doit être supérieur à 8 caractères et contenir au minimum un chiffre, une minuscule et une majuscule. s'affiche bien.

Accessibilité

  • Ouvrir un logiciel de lecteur d'écran
  • Aller sur le nouveau composant Input (par clic ou tabulation)
  • Vérifier que les règles sont bien énoncées par le lecteur d'écran
  • Saisir un mot de passe correspondant aux règles mentionées
  • Vérifier que la progression est vocalisée (X sur Y règles complétées)

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

@er-lim er-lim force-pushed the pix-14149-add-password-checklist-component branch 2 times, most recently from f4a9c13 to 7aaeaab Compare October 7, 2024 07:37
@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch from 7aaeaab to b034ac6 Compare October 7, 2024 09:49
Copy link

gitguardian bot commented Oct 7, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch from b034ac6 to 3102849 Compare October 7, 2024 09:52
@bpetetot bpetetot marked this pull request as ready for review October 7, 2024 09:53
@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch 2 times, most recently from 32f702d to 1716c12 Compare October 8, 2024 08:52
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

✅ Testé et validé. A11y testé avec Voice over.

@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch from 1716c12 to 21f4c35 Compare October 10, 2024 07:41
@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch from 21f4c35 to b836b9e Compare October 10, 2024 08:04
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Lu et testé fonctionnellement avec succès avec Firefox

@bpetetot bpetetot force-pushed the pix-14149-add-password-checklist-component branch from 29b19f1 to 47d0371 Compare October 10, 2024 09:04
@pix-service-auto-merge pix-service-auto-merge merged commit f5a91da into dev Oct 10, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14149-add-password-checklist-component branch October 10, 2024 09:10
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.

5 participants