[Accessibilité - formulaires] Contrôle et aide à la saisie

[hmeneuvrier]

Ticket

658

659

622

674

621

Description

658 : Ajout de signalement dans le BO, préciser les champs obligatoires 659 : Ajout de signalement dans le BO, préciser les champs obligatoires et les formats 622 : Dépôt de signalement front : préciser les champs obligatoires et les formats 674 : contrôle sur les formats des fichiers à uploader sur les différents formulaires (dépôt de signalement, ajout de signalement historique et ajout d'information) 621 : Tous les formulaires : ajouter un id au champ d'erreur et le lier au aria-describedby du champ

Changements apportés

• Modifications de js
• Modifications de twig
• Modifications de FormType

Pré-requis

Tests

• [ ] Tester tous les formulaires, et vérifier qu'il y a une vérification de format pour les fichiers, qu'il y a des indications de champs obligatoires/facultatifs et de format

[numew]

Le soucis que j'ai remarqué sur ce que j'ai voulu tester :

• Signalement logement : Le message d'erreur sur les images n'est pas bloquant (>10mo) et en fin de parcours la soumission plante (time out) + j'aurais préféré que les deux erreur possible soit cumulable (si mauvaise taille et mauvais format)
• Envoyer une estimation : Le champ montant accepte le texte et l'erreur ne s'affiche qu’après la soumission.

J'ai du mal à saisir l'utilisation de la macro :

• Ca me semble sans effet sur les modales ajout entreprise/employée
• ~~Je pensais que le "form_errors" ne servait pas et après avoir trouvé j'ai un doute sur l'affichage (de + il faudrait dans ce cas supprimer les message flash répétant les erreurs en haut de page qui ne sont plus nécessaire) ~~

Je pense que ce que j'ai relevé peu être fait dans des tickets séparés (et je replace l'idée de l'utilisation des form theme plutot que celle de macro même si j'arrive un peu tard https://symfony.com/doc/current/form/form_themes.html)

[numew]

Ok pour moi

[emilschn]

Retours de tests @hmeneuvrier @numew

1/ est-ce que l'ajout des hint faisait partie de cette PR ? je ne sais plus... parce que le format attendu (10 chiffres, ou email) est bien là pour le formulaire de signalement ; mais par exemple pour l'ajout d'entreprise / employé, il n'y est pas. je ne sais plus si ça faisait partie des critères corrigés (je dirais que oui), mais ce serait dommage que ça saute à cause de ces formulaires secondaires. d'ailleurs, les champs sont pas trop espacés dans ces formulaires

2/ dans le formulaire BO de création de signalement, il y a encore un "please fill out this field", au moins pour la date de l'intervention, et "please select an item from the list" pour entreprise et nom de l'agent (mais ça c'est peut-être moins gênant)

3/ La correction de @numew sur l'affichage des erreurs n'est pas prise en compte pour le formulaire de signalement BO sur le prix facturé (un texte par exemple) => la page se recharge et la liste de puces est bien présente

[hmeneuvrier]

@emilschn @numew

1/ traité 2/ je ne reproduis pas pour la date d'intervention ni le nom de l'agent (mais ça ne m'étonne pas car je crois qu'il risque de trainer des textes en anglais car ça dépend des navigateurs/types d'input)

3/ J'ai un comportement encore plus bizarre maintenant. Si je met du texte pour le prix facturé, la page se recharge, aucune erreur n'est affichée, mais le signalement n'est pas créé car j'ai bien une erreur. Je pense qu'il ne faut pas enlever les messages flash dans le cas de signalements créés dans le BO

[emilschn]

Oui je suis d'accord, je crois que c'est nécessaire pour celle-là.

[numew]

J'ai remis les messages flash, pas vu de différence d'affichage sur l'erreur du montant :