Project

General

Profile

Development #40156

avoir une classe pour marquer un formulaire où les astérisques ne doivent pas être ajoutées aux champs obligatoires

Added by Frédéric Péters 8 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
24 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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).

0001-add-class-to-mark-forms-optional-fields-40156.patch View (753 Bytes) Serghei Mihai, 11 Aug 2020 03:59 PM

Screenshot_2020-08-11 Authentic2.png View (29 KB) Serghei Mihai, 11 Aug 2020 04:33 PM

0001-add-class-to-mark-forms-optional-fields-40156.patch View (779 Bytes) Serghei Mihai, 11 Aug 2020 05:10 PM

Screenshot_2020-08-11 Authentic2 - 1.png View (30.1 KB) Serghei Mihai, 11 Aug 2020 05:11 PM

0001-add-class-to-mark-forms-optional-fields-40156.patch View (867 Bytes) Serghei Mihai, 11 Aug 2020 06:24 PM

0001-misc-explicit-forms-optional-fields-and-add-class-to.patch View (1.6 KB) Serghei Mihai, 11 Aug 2020 06:33 PM

47014
47021

Related issues

Blocks Authentic 2 - Development #40159: templates: utilise le template tag with_template pour le rendu des formulaires Solution déployée 25 Feb 2020

Associated revisions

Revision 3de06b19 (diff)
Added by Serghei Mihai 2 months ago

misc: explicit forms optional fields and add class to mark them (#40156)

History

#1 Updated by Serghei Mihai 8 months ago

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.

#2 Updated by Frédéric Péters 8 months ago

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)

#3 Updated by Serghei Mihai 8 months ago

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.

#4 Updated by Frédéric Péters 8 months ago

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>).

#5 Updated by Serghei Mihai 8 months ago

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.

#6 Updated by Serghei Mihai 8 months ago

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.

#7 Updated by Benjamin Dauvergne 3 months ago

  • Blocks Development #40159: templates: utilise le template tag with_template pour le rendu des formulaires added

#8 Updated by Serghei Mihai 3 months ago

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)".

#9 Updated by Frédéric Péters 3 months ago

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 ?

#10 Updated by Serghei Mihai 3 months ago

47014

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.

#11 Updated by Frédéric Péters 3 months ago

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 ?

#12 Updated by Serghei Mihai 3 months ago

47021

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.

#13 Updated by Frédéric Péters 3 months ago

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 ?

#14 Updated by Serghei Mihai 3 months ago

En effet, collé un vieux patch.

#15 Updated by Frédéric Péters 3 months ago

qui se réduit à modifier le template

Le patch ne contient toujours pas ça. (?)

#17 Updated by Frédéric Péters 2 months ago

Sur un <span> préciser display: inline; ne me semble pas nécessaire.

#18 Updated by Serghei Mihai 2 months ago

Sans le inline le texte optional ne sera pas affiché quand la classe pk-mark-optional-fields est positionnée sur form.

#19 Updated by Frédéric Péters 2 months ago

  • Status changed from Solution proposée to Solution validée

Ah capté, c'est un display: inline pour dire display: initial, annuler la valeur modifiée, ok.

#20 Updated by Serghei Mihai 2 months ago

  • Status changed from Solution validée to 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)

#21 Updated by Frédéric Péters 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF