Development #64681
Les paramètres pour les titres de cellule
0%
Description
Dans #42968 sur l'ajout d'une option "titre" sur les cellules texte il y a "Toutes les cellules ont un champ titre qui correspond au titre de la cellule." mais ça n'est pas totalement exact, on a surtout une variété de gestions différentes.
Avant de pouvoir avancer sur #42968 je serais donc pour mettre un peu d'ordre et pour moi, comme on a souvent un titre par défaut qui est tout à fait correct, ce qui est illustré pa la forme des optins qui sont "Titre personnalisé" (et laissé vide c'est le titre par défaut) (pour la cellule "demandes de l'usager" par exemple), ou "titre" sous forme d'option <select> avec "Titre par défaut" sélectionné par défaut et une option "Titre personnalisé" dedans (pour la cellule "contenu d'une fiche").
Paramètre accessoire donc, je serais pour le déplacer vers l'onglet "Apparence".
(ça permet ensuite de boucler sur #42968 et y ajouter l'option et garder une cellule qui dans son paramétrage immédiat reste totalement simple avec un seul champ texte).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 2 ans
- Lié à Development #42968: Un champ titre pour la cellule texte ajouté
Mis à jour par Corentin Séchet il y a presque 2 ans
Fred, tu parles bien uniquement des titres personnalisés à déplacer dans l'onglet Apparence ? Concernant les cellules qui ont un champs "Titre" (SearchCell, GalleryCell...) je le laisse dans "Général" ?
Mis à jour par Frédéric Péters il y a presque 2 ans
Ils sont partout optionnels donc je pensais bien tous les déplacer.
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-misc-move-title-fields-to-appearance-tab-64681.patch 0001-misc-move-title-fields-to-appearance-tab-64681.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pour les tests, je ne suis pas sûr de ce qui est le mieux, entre tester le HTML final (ce que j'ai fait) ou juste vérifier que les champs titre soient bien dans cell.get_appearance_form_class().base_fields. Je n'ai pas fait les tests pour les cellules BookingCalendar, ChartCell et Gauge qui sont, il me semble, dépréciées.
Mis à jour par Frédéric Péters il y a presque 2 ans
Il y a un peu trop de diversité je trouvé, à la fois yield 'custom_title' et return ['custom_title'].
Mais à passer sur tout ça je me dis que je verrais ça davantage systématique, quelque chose de cet ordre, dans CellBase,
def get_appearance_fields(self):
return ['title', 'custom_title']
def get_appearance_form_class(self):
fields = [x for x in self.get_appearance_fields() if x est bien un champ du modèle] + ['slug', 'extra_css_class']
[...]
et ainsi tu peux juste redéfinir get_appearance_fields à l'endroit où ça change un peu,
def get_appearance_fields(self):
return ['title_type', 'custom_title']
Éventuellement ça se discute d'inclure slug et extra_css_class dans cette liste, plutôt que les avoir ajoutés systématiquement, mais tant qu'on n'a pas de cellule où il y auait à ne pas avoir slug ou extra_css_class, on s'en passe pour le mieux.
Mis à jour par Corentin Séchet il y a presque 2 ans
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
(ok, prochain cycle)
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2b68a863ed7c2394fb8accc3fc9629f4223ecb42
Author: Corentin Séchet <csechet@entrouvert.com>
Date: Mon May 2 14:13:38 2022 +0200
misc: move title fields to appearance tab (#64681)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: move title fields to appearance tab (#64681)