Project

General

Profile

Development #30371

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

Added by Brice Mallet 6 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
04 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-misc-change-default-to-not-include-fields-in-tables-.patch View (3.7 KB) Frédéric Péters, 05 Feb 2019 05:14 PM

Associated revisions

Revision 9193f2c9 (diff)
Added by Frédéric Péters 6 months ago

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

History

#1 Updated by Brice Mallet 6 months ago

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

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

  • Assignee set to Frédéric Péters

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

Ça a surtout demandé des adaptations aux tests…

#4 Updated by Thomas Noël 6 months ago

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 Updated by Frédéric Péters 6 months ago

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 Updated by Thomas Noël 6 months ago

  • Status changed from Solution proposée to 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 Updated by Frédéric Péters 6 months ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 5 months ago

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

Also available in: Atom PDF