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

[FO - Formulaire - Accessibilité - 11.13] Faciliter remplissage automatique des champs #2575

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented May 17, 2024

Ticket

#2236

Description

Critère 11.13 [AA] : La finalité d'un champ de saisie peut-elle être déduite pour faciliter le remplissage automatique des champs avec les données de l'utilisateur ?

Changements apportés

  • Ajout de données d'autocomplete dans les json du formulaire
  • Prise en charge de ces attributs d'autocomplete pour les composants SignalementFormTextfield, SignalementFormPhonefield, SignalementFormAdress et SignalementFormEmailfield
  • Ajout d'une balise form et d'un bouton submit caché dans l'app pour faire marcher l'autocomplete

Pré-requis

Tests

  • Faire des tests avec différents profils et tester l'autocompletion sur les adresses, les mails, les téléphones, les noms et prénoms et noms d'organisation (plus facile sous chromium que sous firefox car on peut pré-enregistrer des adresses et noms d'organisation chrome://settings/addresses)
  • Vérifier que les autres champs textes sans données d'autocomplétion sont valides et que le formulaire fonctionne bien

@hmeneuvrier hmeneuvrier force-pushed the feature/2236-fo-formulaire-accessibilite-1113-faciliter-remplissage-automatique-des-champs branch from d746831 to ac83b33 Compare May 21, 2024 10:14
@hmeneuvrier hmeneuvrier changed the title [WIP][FO - Formulaire - Accessibilité - 11.13] Faciliter remplissage automatique des champs [FO - Formulaire - Accessibilité - 11.13] Faciliter remplissage automatique des champs May 21, 2024
@hmeneuvrier hmeneuvrier marked this pull request as ready for review May 21, 2024 13:40
Copy link
Collaborator

@numew numew 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 relecture OK

@emilschn
Copy link
Collaborator

Après test :
C'est normal qu'on garde un autocomplete sur les champs de complément d'adreses (étage, etc.) ?

Sinon le reste me semble ok :)

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

Une question :)

@hmeneuvrier
Copy link
Collaborator Author

C'est normal qu'on garde un autocomplete sur les champs de complément d'adreses (étage, etc.) ?

@emilschn non effectivement, j'avais oublié de gérer les subscreens, c'est corrigé

Copy link

sonarcloud bot commented May 23, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

Vu ensemble : certains champs ont encore un comportement étrange, mais on laisse tel quel.

@emilschn emilschn merged commit 532a193 into develop May 29, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2236-fo-formulaire-accessibilite-1113-faciliter-remplissage-automatique-des-champs branch June 11, 2024 10:27
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.

None yet

3 participants