Project

General

Profile

Development #26641

empêcher de casser le partage de template

Added by Frédéric Péters over 1 year ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
23 Sep 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Sur une page redirection/modèle, occuper l'espace "contenu" par une explication ("cette page est une redirection vers ..., elle peut également lui servir de modèle") et ne pas inclure le <select> d'ajout de cellule.

0001-pages-always-display-placeholder-blocks-in-skeleton-.patch View (3.08 KB) Lauréline Guerin, 21 Oct 2019 05:17 PM

0001-pages-always-display-placeholder-blocks-in-skeleton-.patch View (3.06 KB) Lauréline Guerin, 22 Oct 2019 03:05 PM

0001-pages-always-display-placeholder-blocks-in-skeleton-.patch View (3.98 KB) Lauréline Guerin, 28 Oct 2019 09:02 AM

Associated revisions

Revision 0c854a9e (diff)
Added by Lauréline Guérin 4 months ago

pages: always display placeholder blocks in skeleton (#26641)

History

#1 Updated by Lauréline Guerin 4 months ago

  • Assignee set to Lauréline Guerin

#2 Updated by Lauréline Guerin 4 months ago

#3 Updated by Thomas Noël 4 months ago

Je pense que l'idée est plutôt de jouer sur le cas object.redirect_url and placeholder.key == "content" dans la boucle {% for placeholder in placeholders %} de combo/manager/templates/combo/page_view.html ... a priori rien de plus. Dans ce cas, afficher le message et ne pas poser le select qui permet d'ajouter une cellule. Et sans doute afficher un petit warning si placeholder.cells n'est pas vide.

#4 Updated by Frédéric Péters 4 months ago

Oui c'était l'idée mais en en discutant je suis venu avec l'idée qu'on pouvait très bien ne pas empêcher l'ajout de cellules, en reprenant de manière systématique les templatetags placeholder; ça permettrait ainsi de définir dans combo du contenu qui s'affichera systématiquement sous le contenu posé par l'application. (dessus ou dessous, choix à faire, ma préférence va à dessous, dans l'idée de permettre d'écrire là les infos "vie privée etc." qu'on veut parfois voir sans les démarches, et qui se trouvent aujourd'hui mochement dans le pied de page).

#5 Updated by Thomas Noël 4 months ago

Ok, dans cette optique le patch me semble bel et bon. On a gagne donc une nouvelle fonctionnalité.

Dans le message

This page is configured as a redirection to the URL "{{ redirect_url }}", it can also be used as a template for the page behind the URL "{{ redirect_url }}"

on pourrait éviter la répétition de l'URL (qui sera moche) et dire que ça peut concerner plusieurs pages :

... it can also be used as a template for the pages behind this URL.

#7 Updated by Thomas Noël 4 months ago

  • Status changed from Solution proposée to Solution validée

#8 Updated by Frédéric Péters 4 months ago

  • Status changed from Solution validée to En cours

Ça fait apparaitre la partie de la page modèle au-dessus du contenu mis par authentic/w.c.s.; je ne sais pas si c'est délibéré, ma préférence allait à faire apparaitre ça sous le contenu.

#9 Updated by Frédéric Péters 4 months ago

A priori il s'agit juste de déplacer le bloc

{% if render_skeleton %}
{{ skeleton }}
{% endif %}

avant la boucle {% for cell in cells %}.)

#10 Updated by Lauréline Guerin 4 months ago

#11 Updated by Frédéric Péters 4 months ago

  • Status changed from Solution proposée to Solution validée

Go.

#12 Updated by Lauréline Guerin 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 0c854a9e50df58ca4f18eef2ae0350ca5aa2e3de (HEAD -> master, origin/wip/26641-redirect, origin/master, origin/HEAD, wip/26641-redirect)
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Oct 21 17:15:50 2019 +0200

    pages: always display placeholder blocks in skeleton (#26641)

#13 Updated by Frédéric Péters 4 months ago

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

Also available in: Atom PDF