Development #40156
avoir une classe pour marquer un formulaire où les astérisques ne doivent pas être ajoutées aux champs obligatoires
0%
Description
Genre un formulaire de connexion, qui pourrait être marqué façon
<form method="post" class="pk-mark-optional-fields">
(avec l'utilisation du gadjo |with_template en front également, le style devra également être présent côté publik-base-theme).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a environ 4 ans
Ici l'objectif est plutôt de poser la classe pk-mark-optional-fields sur le formulaire quand il y a des champs optionnels à marquer, il me semble.
Mis à jour par Frédéric Péters il y a environ 4 ans
L'objectif est de ne pas avoir d'astérisques ajoutés au formulaire de connexion, par exemple. Là-dessus si ton commentaire est que "pk-mark-optional-fields" donne plutôt l'idée que ça s'applique à un formulaire où il y aurait des champs optionnels, ne donne pas l'impression d'être applicable au formulaire de connexion, où il y a juste deux champs obligatoires, oui je peux être assez d'accord et dire qu'un autre nom de classe serait plus opportun.
(mais peut-être que j'interprète mal et que ton souhait n'est pas une discussion sur le nom de la classe)
Mis à jour par Serghei Mihai il y a environ 4 ans
Je m'exprime mal: effectivement ce n'est pas du nom de la classe dont je parle. D'abord avoir un mécanisme pour poser cette classe sur le formulaire. Ensuite définir les styles nécessaires.
Mis à jour par Frédéric Péters il y a environ 4 ans
Je ne comprends pas du tout, tu appelles quoi "mécanisme" ? (versus ouvrir le fichier de gabarit et ajouter class="whatever" sur la balise <form>).
Mis à jour par Serghei Mihai il y a environ 4 ans
Ma réflexion doit être influencée par #40159 car je ne pense pas uniquement au formulaire de connexion ou on poserait simplement une classe css, mais aussi à ceux de création, modification du compte.
Et par "mécanisme" j'entends un template tag qui vérifierait s'il y a des champs optionnels dans le formulaire et poserait cette classe.
Mis à jour par Serghei Mihai il y a environ 4 ans
La nuit portant conseil je me rends compte que mon idée est inutile. On pose la classe de toute manière et ensuite les styles font le rendu des champs optionnels.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Bloque Development #40159: templates: utilise le template tag with_template pour le rendu des formulaires ajouté
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Fichier 0001-add-class-to-mark-forms-optional-fields-40156.patch 0001-add-class-to-mark-forms-optional-fields-40156.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Serghei Mihai
- Patch proposed changé de Non à Oui
astérisques cachés pour les champs obligatoires et les titres des champs optionnels en italique. Une autre idée serait d'afficher à côté du libellé des champs optionnels un texte genre "(optionnel)".
Mis à jour par Frédéric Péters il y a plus de 3 ans
optional, un seul n. (ça pointe aussi que ça n'a pas vraiment été testé, tu pourras taper une capture ?).
Sur le côté libellé en italique, c'est quelque chose qui est compris par les gens ou tu comptes ajouter du texte pour expliciter ?
Mis à jour par Serghei Mihai il y a plus de 3 ans
Effectivement faute de typo.
Concernant l'italique, je pense qu'il serait plus clair d'avoir le texte "(optionnel)" en italique à côté du libellé du champ, et donc modifier le template tag with_template.
Mis à jour par Frédéric Péters il y a plus de 3 ans
et donc modifier le template tag with_template
Je ne vois pas comment tu imagines la chose; la classe étant posée sur le <form> indépendamment, tu n'y a pas accès dans |with_template.
- modifier |with_template pour prendre un paramètre supplémentaire pour donner l'info ?
- tout le temps ajouter (optionnel) mais par défaut cacher le texte ? (mais c'est plutôt gadjo/widget.html que le templatetag qu'il s'agit de modifier alors)
- autre chose ?
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Fichier 0001-add-class-to-mark-forms-optional-fields-40156.patch 0001-add-class-to-mark-forms-optional-fields-40156.patch ajouté
- Fichier Screenshot_2020-08-11 Authentic2 - 1.png Screenshot_2020-08-11 Authentic2 - 1.png ajouté
Frédéric Péters a écrit :
et donc modifier le template tag with_template
- tout le temps ajouter (optionnel) mais par défaut cacher le texte ? (mais c'est plutôt gadjo/widget.html que le templatetag qu'il s'agit de modifier alors)
Par "modifier le template tag with_template" j'entendais la modification du rendu du template tag, qui se réduit à modifier le template:
- {% if field.field.required %}<span title="{% trans "This field is required." %}" class="required">*</span>{% endif %}
+ {% if field.field.required %}
+ <span title="{% trans "This field is required." %}" class="required">*</span>
+ {% else %}
+ <span class="optional">{% trans "(optional)" %}</span>
+ {% endif %}
et puis appliquer l'italique au texte
(optional) avec le rendu comme dans la capture.Mis à jour par Frédéric Péters il y a plus de 3 ans
Mais ça va afficher optionnel même quand on n'a pas mis pk-mark-optional-fields.
Ou bien tu as remis l'ancien patch par erreur ?
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Fichier 0001-add-class-to-mark-forms-optional-fields-40156.patch 0001-add-class-to-mark-forms-optional-fields-40156.patch ajouté
En effet, collé un vieux patch.
Mis à jour par Frédéric Péters il y a plus de 3 ans
qui se réduit à modifier le template
Le patch ne contient toujours pas ça. (?)
Mis à jour par Serghei Mihai il y a plus de 3 ans
Mis à jour par Frédéric Péters il y a plus de 3 ans
Sur un <span> préciser display: inline; ne me semble pas nécessaire.
Mis à jour par Serghei Mihai il y a plus de 3 ans
Sans le inline le texte optional ne sera pas affiché quand la classe pk-mark-optional-fields est positionnée sur form.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Ah capté, c'est un display: inline pour dire display: initial, annuler la valeur modifiée, ok.
Mis à jour par Serghei Mihai il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3de06b197049c331eb6b2c89903cc5d3538773d7 (HEAD -> master, origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Aug 11 16:59:09 2020 +0200
misc: explicit forms optional fields and add class to mark them (#40156)
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
misc: explicit forms optional fields and add class to mark them (#40156)