ETQ usager je ne veux pas pouvoir saisir un nombre invalide sans feedback #9516
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
colinux
force-pushed
the
fix-invalid-facultative-evaporation
branch
2 times, most recently
from
a4a4639 to
255967d
Compare
colinux
changed the title
colinux
force-pushed
the
fix-invalid-facultative-evaporation
branch
from
255967d to
416db5a
Compare
colinux
changed the title
colinux
force-pushed
the
fix-invalid-facultative-evaporation
branch
2 times, most recently
from
d86a375 to
b0c316b
Compare
colinux
changed the title
LeSim
approved these changes
Sep 28, 2023
| @@ -1 +1 @@ | |||
| = @form.number_field(:value, input_opts(id: @champ.input_id, aria: { describedby: @champ.describedby_id }, step: :any, required: @champ.required?)) | |||
| = @form.text_field(:value, input_opts(id: @champ.input_id, aria: { describedby: @champ.describedby_id }, required: @champ.required?, pattern: "-?[0-9]+([\.,][0-9]{1,3})?", inputmode: :decimal, data: { controller: 'format decimal-number-input', format: :decimal })) | |||
There was a problem hiding this comment.
autoriser les espaces ? (mm chose pour les nombres entiers)
There was a problem hiding this comment.
Yes je ferai ça dans une autre PR, ça touche à d'autres aspects: le formatteur JS et le backend qui doit les supprimer pour parser le string en nombre.
Au passage j'avais aussi passé du temps à essayer de cabler un truc plus général en guise de séparateur de milliers (pour accepter aussi les , et .) mais ça mène très trop loin car ces caractères sont sont aussi possibles comme séparateur de décimales, et un usager peut saisir dans un format autre que la locale de son navigateur…
Les input=number n'ont pas de value lorsque la valeur saisie n'est pas un nombre. Par conséquent dans ces cas là, nous ne pouvions faire remonter au backend pour validation / enregistrement, et il n'y avait aucun feedback signalement l'erreur à l'usager. On les convertit en inputs texte, avec les adaptations nécessaires pour montrer le pavé numérique sur mobile, et un style correct.
colinux
force-pushed
the
fix-invalid-facultative-evaporation
branch
from
b0c316b to
e492c8a
Compare
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Quand un usager remplissait une donnée invalide d'un champ nombre, l'autosave ne transmettrait pas la valeur au backend car le navigateur détecte la valeur comme invalide et on a aucun moyen de connaître la valeur saisie. On faisait un autosave d'un champ vide.
Cette PR :
type=numberpar untype=textavec unpatternetinputmodeadaptés.et,(on continue d'ignorer les séparateurs de milliers)