Bug #20670
Remplacer les [] par des {{}} dans le listing des champs d'un formulaire
0%
Description
Aujourd'hui les variables qui apparaissent dans le listing des champs sont entre simples crochets, [form_var_nom]. Il faut les passer entre double accolades pour être cohérent avec l'utilisation des gabarit Django que l'on préconise désormais, {{form_var_nom}}.
J'ai déjà rédigé la doc dans ce sens.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-backoffice-use-when-displaying-varnames-20670.patch 0001-backoffice-use-when-displaying-varnames-20670.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Pas tout à fait fan du résultat; ça alourdit la lecture de la liste des champs (mais le côté intrusif de ces variables était déjà noté au moment de l'ajout, dans #11255). (ma préférence irait au retrait des []).
Mis à jour par Thomas Noël il y a plus de 6 ans
Pareil, retrait des []. Peut-être ajout d'un title="{{ form_var_machin }}" histoire de... ou pas.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier Screenshot-2017-12-14.png Screenshot-2017-12-14.png ajouté
En discutant et jouant là-dessus, je suis arrivé à la capture en pièce jointe; qui augmente la taille de l'affichage du type de champ + l'info optionnel ou pas, et réduit la taille de la police monospace contenant le form_var_whatever, et alors je trouve que ça passe, même avec les {{ }}.
Mis à jour par Thomas Noël il y a plus de 6 ans
C'est correct effectivement, à mon goût. Pierre ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Validé via jabber avec Pierre; j'attache le patch dans trois minutes.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit c45b8364bb36cada7ddb02ca6ed3d8959d55ccfe Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Dec 14 08:02:55 2017 +0100 backoffice: use {{ }} when displaying varnames (#20670)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: use {{ }} when displaying varnames (#20670)