-
Notifications
You must be signed in to change notification settings - Fork 91
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
amelioration(champs.erreurs): ETQ usager, je retrouve les erreurs sous les champs + les champs de type text/number sont au format dsfr 🫖🥖 #9004
Conversation
PR Summary
|
j'ai plusieurs remarques/retours avec des propositions de styles et de comportements :
sur le fichier Figma, page Styles et comportements j'ai orthographié l'ensemble des messages d'erreurs pour tous les types de champs : https://www.figma.com/proto/YKcafywdYhT6ABFlmy5d56/Am%C3%A9lioration-vue-usagers?page-id=1%3A2&node-id=2681-125753&viewport=-19420%2C-41992%2C0.91&scaling=min-zoom&starting-point-node-id=2681%3A125753&show-proto-sidebar=1 |
merci @Olivier-Marcellin pour ces retours ; en l'état ça se voulait plus être un point d'architecture du code pour solliciter les collègues voir si on est assez mature sur les bases du code actuel pour tenter l'aventure. mais tes retours sont interessants, mes réponses :
je comprends ton point mais une nuance reste. C'est un go ou un no-go ? A mon sens c'est un go car aujourd'hui il y a du support qui est chronophage (tant coté bizdev, que dev.. et de maniere pragmatique : cote usager, ils sont bloqués, on les debloque) lié aux erreurs sur le formulaire. En substance sur 1 semaine glissante : 4 retours :
Bref, aujourd'hui pour solutionner ça ETQ dev faisant du support : de manière très expérimentale, ± a chaque fois qu'un user revient avec "c'est bogué, j'ai remplis tous les champs mais ça marche pas", j'envoie un lien vers l'element
nous avons des formulaires avec +- 100 champs, dans ce cas scroller pour identifier le champs en erreur ne me semble pas si évident, qu'en penses tu? j'imagine qu'on peut remonter ça au DSFR, dans l'intermediaire je propose cette rustine qui pointe vers l'irritant afin de permettre a l'usager d'être autonome.
La locution des messages d'err est top ! point d'attention cependant : https://secure.helpscout.net/conversation/2202674693/2025725/. il arrive que des admins formulent mal le libellé (ici, demandant un RIB plutot qu'un IBAN... resulat, j'ai appelé l'usager, 30min au tel, pour identifier avec elle le problème en plus du lien de correction). bref, c'est une belle 1ere itération, mais on peut je pense anticiper plusieurs aller/retour en fonction du support.
same c'est top! on tend vers ça (sur tout le site, sauf le formulaire usager. mais ça arrive !). isoler l'erreur et proposer une locution qui oriente sur la correction. PS: pour les remarques hors scope je propose de se cantonner au périmêtre de ce sujet afin d'eviter de la frustration |
cc06e07
to
6bd5be0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trop bien déjà ! Je tique juste un peu sur le rouge en gras mais à ce stade c'est un détail ;)
app/components/editable_champ/editable_champ_component/editable_champ_component.html.haml
Outdated
Show resolved
Hide resolved
6bd5be0
to
8047e0e
Compare
4071afc
to
8938ada
Compare
je compile ici toutes mes remarques suite aux éléments d'intégration postés à ce jour de haut en bas sur les pages :
n'hésite pas à consulter les maquettes que je mets à jour dès qu'il y a une nouvelle conception de manière à avoir une vision d'ensemble et prospective : sinon c'est top ça commence à prendre forme (form) :) |
pour information voici un exemple de mise en page de formulaire au dsfr intégré par les développeurs https://template.incubateur.net/formulaire |
910b312
to
8a3e10a
Compare
d66bb7b
to
e17a133
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
retours en cours
attributes: | ||
champs/datetime_champ: | ||
hints: | ||
value: "Expected format : dd/mm/yyyy hh:mm" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a priori on peut pas hardcoder les formats qui dépendent de la locale du navigateur, mais c'est peut-être plus à refactorer dans une autre PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeap, tchak a fait la mm remarque. la j'ai juste deplacé ça ds la trad du champs pr avoir les erreurs "automatique" au bon endroit. a voir pr un controller js qui internationalise le format
bf86d03
to
1b33203
Compare
c'est beau en tout cas, bravo ! |
6a6bd1a
to
cf6fc2e
Compare
…u taille actuelle] avec le composant DSFR
…r l'asterisk en fonction de l'etat du champ
…mi-DSFR, mi-DS n'est pas trop moche amelioration(design): ETQ usager la cohabitation du design d'un form mi-DSFR, mi-DS n'est pas trop moche
…mi-DSFR, mi-DS n'est pas trop moche amelioration(design): ETQ usager la cohabitation du design d'un form mi-DSFR, mi-DS n'est pas trop moche
cf6fc2e
to
9a0ee85
Compare
sumup :
preview complète :
Homogénéisation du compteur de caractère sur le champ de type textarea :
Homogénéisation des feedbacks sur le champ siret :
sur un form existant :