Development #30371
Affichage dans les listings : rendre décoché par défaut
0%
Description
Le comportement actuel de ce paramètre de champ de formulaire est d'être activé (coché) par défaut (https://doc-publik.entrouvert.com/admin-fonctionnel/fabrique-formulaires/form-champs/parametres-champ-generiques/).
Le résultat est de se retrouver avec un tableau de traitement avec de trop nombreuses colonnes.
En tant qu'administrateur fonctionnel, il est préférable de choisir explicitement les qq colonnes à afficher plutôt que soustraire les nombreuses non souhaitées.
Fichiers
Révisions associées
Historique
Mis à jour par Brice Mallet il y a environ 5 ans
A été discuté et validé lors de l'EOcamp Prissé (oct18)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0001-misc-change-default-to-not-include-fields-in-tables-.patch 0001-misc-change-default-to-not-include-fields-in-tables-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ça a surtout demandé des adaptations aux tests…
Mis à jour par Thomas Noël il y a environ 5 ans
Plutôt supprimer la ligne "in_listing = True" qui sera pris de class Field, non ?
Qui de virer les "in_listing = False # no way to represent data in a single cell" dans TableField et TableRowsField et de poser le commentaire "# no way to represent data in a single cell" à côte des form.remove(in_listing) et t.remove(in_listing)
Mais bon, ton patch est ok aussi, pousse tel quel si tu préfères laisser de l'explicite.
A noter que c'est en "advanced=True" donc le choix n'apparaitra pas par défaut, je trouve ça presque un poil dommage pour le coup... (mais on peut se dire que c'est comme ça et que le travail serait plutôt de revoir la présentation des champs "advanced" pour moins les oublier, que sais-je).
Mis à jour par Frédéric Péters il y a environ 5 ans
Mais bon, ton patch est ok aussi, pousse tel quel si tu préfères laisser de l'explicite.
Yep, je préfère.
A noter que c'est en "advanced=True" donc le choix n'apparaitra pas par défaut, je trouve ça presque un poil dommage pour le coup... (mais on peut se dire que c'est comme ça et que le travail serait plutôt de revoir la présentation des champs "advanced" pour moins les oublier, que sais-je).
Il y a sans doute à longuement réfléchir sur ce qu'on veut là-dedans, les options rares, les options compliquées, etc. ?
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Mais bon, ton patch est ok aussi, pousse tel quel si tu préfères laisser de l'explicite.
Yep, je préfère.
Push it -- désolé oublié de changer le statut, c'était validé bien sur.
A noter que c'est en "advanced=True" donc le choix n'apparaitra pas par défaut, je trouve ça presque un poil dommage pour le coup... (mais on peut se dire que c'est comme ça et que le travail serait plutôt de revoir la présentation des champs "advanced" pour moins les oublier, que sais-je).
Il y a sans doute à longuement réfléchir sur ce qu'on veut là-dedans, les options rares, les options compliquées, etc. ?
Ouaip... on va demander aux collègues utilisateurs ? (ailleurs que dans ce ticket, of course)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9193f2c936d7e6c727aeadae4419883b3258cf42 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 5 16:40:29 2019 +0100 misc: change default to not include fields in tables (#30371)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: change default to not include fields in tables (#30371)