Projet

Général

Profil

Development #30371

Affichage dans les listings : rendre décoché par défaut

Ajouté par Brice Mallet il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 février 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 9193f2c9 (diff)
Ajouté par Frédéric Péters il y a environ 5 ans

misc: change default to not include fields in tables (#30371)

Historique

#1

Mis à jour par Brice Mallet il y a environ 5 ans

A été discuté et validé lors de l'EOcamp Prissé (oct18)

#2

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Assigné à mis à Frédéric Péters
#3

Mis à jour par Frédéric Péters il y a environ 5 ans

Ça a surtout demandé des adaptations aux tests…

#4

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

#5

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

#6

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)

#7

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)
#8

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

Formats disponibles : Atom PDF