Projet

Général

Profil

Development #37135

placeholders optionnels dans l'édition d'une page

Ajouté par Frédéric Péters il y a plus de 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
22 octobre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Aujourd'hui on affiche une section par placeholder, ça donne par exemple barre latérale, contenu, pied de page.

Sur la plupart des pages, avoir une section "pied de page" ne sert à rien, parce que son contenu restera inchangé. Il y a parfois également eu l'idée d'ajouter d'autres zones, par exemple pour pouvoir éditer des liens en haut de page, et ça n'a pas été fait notamment parce que ça alourdirait toutes les pages d'édition.

Je serais ainsi pour pouvoir marquer un placeholder avec optional=True, ce qui ferait que le placeholder ne serait pas repris sur la page d'édition, sauf après manipulation explicite, sans doute encore à réfléchir un peu ici, je pensais à quelque chose comme une action "éditer une section optionnelle", qui ouvre une popup où l'utilisateur peut choisir une section puis ça redirige vers la page ?include-section=pre-header, l'usager peut alors y ajouter une cellule. Ça serait ensuite persistant parce que les sections contenant des cellules seraient systématiquement affichées (sauf si la seule cellule est ParentContentCell).

Je note optional=True, c'est après avoir hésité à réutiliser acquired=True pour cet usage, puis m'être dit qu'on sera mieux à ne pas mêler les deux comportements.


Fichiers

Révisions associées

Révision e40a3e8b (diff)
Ajouté par Lauréline Guérin il y a environ 4 ans

placeholder: optional placeholders display and edition (#37135)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 4 ans

Je note sur :

les sections contenant des cellules seraient systématiquement affichées (sauf si la seule cellule est ParentContentCell).

que ça voudrait dire qu'on n'affiche pas les placeholders optionnels s'ils sont vide ou s'ils contiennent uniquement une ParentContentCell, et ça me semble un petit peu compliqué quand il faudra comprendre/expliquer ces deux cas (et déboguer le manque d'une ParentContentCell, parce que ça ne sera pas visible). Tout ça pour dire que je trouverais quand même pas mal que quelque part on affiche que le placeholder existe et qu'il est "comme son parent" (mais je n'ai pas d'idée pour rendre cela clair mais discret)

#2

Mis à jour par Frédéric Péters il y a plus de 4 ans

Alors peut-être que plutôt que l'idée "éditer une section optionnelle" avoir en barre latérale un titre "Sections optionnelles" sous la section "Paramètres", qui serait une simple liste, genre :

  • Pre header (empty) [edit]
  • Top of page (empty) [edit]
  • Footer (like parent) [edit]

?

#3

Mis à jour par Thomas Noël il y a plus de 4 ans

Frédéric Péters a écrit :

Alors peut-être que plutôt que l'idée "éditer une section optionnelle" avoir en barre latérale un titre "Sections optionnelles" sous la section "Paramètres", qui serait une simple liste, genre :

  • Pre header (empty) [edit]
  • Top of page (empty) [edit]
  • Footer (like parent) [edit]

Ca suffirait très bien.

#4

Mis à jour par Lauréline Guérin il y a environ 4 ans

  • Assigné à mis à Lauréline Guérin
#5

Mis à jour par Lauréline Guérin il y a environ 4 ans

#7

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

  • Statut changé de Solution proposée à Solution validée

Par mimétisme avec les options plus haut dans la colonne, je remplacerais bien "edit" par "change", entre parenthèses :

-    <a href="{% url 'combo-manager-page-view' pk=object.id %}?include-section={{ placeholder.key }}">{% trans 'edit' %}</a>
+    (<a href="{% url 'combo-manager-page-view' pk=object.id %}?include-section={{ placeholder.key }}">{% trans 'change' %}</a>)

mais ok avec ou sans.

#8

Mis à jour par Lauréline Guérin il y a environ 4 ans

je fais la modif et je merge, merci :)

#9

Mis à jour par Lauréline Guérin il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit e40a3e8b670b1529ba8666b269eb19eea874257c
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Jan 23 16:18:51 2020 +0100

    placeholder: optional placeholders display and edition (#37135)
#10

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF