Development #48003
class "form-with-error" sur body pour pages de validation d'étape
0%
Description
En cherchant des solutions pour #39239…
Pour le moment il n'est possible de mettre en exergue que les champs directement en erreur.
Il serait pourtant graphiquement intéressant de pouvoir parfois modifier d'autres composant de la page, par exemple.
- l'étape en cours,
- les champs déjà validés,
- titre de l'étape.
J'expose l'idée, (si techniquement possible) d'ajouter une class spécifique sur le body pour les pages qui possèdent un formulaire qui ne passe pas la validation.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Description mis à jour (diff)
Mis à jour par Frédéric Péters il y a plus de 3 ans
On peut imaginer que du js regarde s'il y a un élément avec la classe widget-with-error et garde synchro le nécessaire ? (c'est dans l'idée qu'on pourrait imaginer qu'un champ soit modifié pour corriger une erreur et que ça en retire la classe widget-with-error (ce que fait static/grosboule-les-bains/extra.js) et que voilà on se trouve alors à devoir dupliquer une logique pour taper ou non une classe sur le body.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
On peut imaginer que du js regarde s'il y a un élément avec la classe widget-with-error et garde synchro le nécessaire ?
Oui, JS peut faire le job.
ce que fait static/grosboule-les-bains/extra.js
Le code va supprimer la class .widget-with-error au moindre remplacement de value même si le champs ne passe pas la validation.
Je n'ai pas compris
on se trouve alors à devoir dupliquer une logique pour taper ou non une classe sur le body
Mis à jour par Frédéric Péters il y a plus de 3 ans
Le code va supprimer (...)
Oui, on n'a pas de retranscription js ou interrogation du serveur pour faire quelque chose d'avancé.
on se trouve alors à devoir dupliquer une logique pour taper ou non une classe sur le body
Je voulais dire qu'on se retrouverait, en prenant l'existant grosboule-les-bains + la modification suggérée ici, à avoir à reproduire en javascript, là où on a actuellement le .removeClass('widget-with-error'), une logique supplémentaire de retirer, ou pas, une classe sur le <body>, en fonction de la présence d'autres champs en erreur.
Et de là, quant à donc avoir à faire le code en javascript, qu'il y soit totalement.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Assigné à mis à Thomas Jund (congés, retour le 29/04)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Nouveau à En cours
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Fichier 0001-JS-add-form-with-error-class-to-body-if-form-contain.patch 0001-JS-add-form-with-error-class-to-body-if-form-contain.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Proposition de modification du JS pour ajouter une class `form-with-error` au body
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Allons-y ainsi.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit bc361012d986e58a74690cc841477f59a47c1e80 (HEAD -> master, origin/master, origin/HEAD) Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Nov 4 15:29:37 2020 +0100 JS: add form-with-error class to body if form contain error (#48003)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
JS: add form-with-error class to body if form contain error (#48003)