Projet

Général

Profil

Development #26641

empêcher de casser le partage de template

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
23 septembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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.


Fichiers

Révisions associées

Révision 0c854a9e (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

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

Historique

#1

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

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

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

#3

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

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

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

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

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

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

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

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

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

  • Statut changé de Solution validée à 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

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

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

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

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

#11

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

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

Go.

#12

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

  • Statut changé de Solution validée à 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

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

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

Formats disponibles : Atom PDF