Projet

Général

Profil

Bug #7206

sekizai is impossible to deal with theming-wise

Ajouté par Frédéric Péters il y a presque 9 ans. Mis à jour il y a presque 9 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
-
Version cible:
Début:
12 mai 2015
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

The insistance of sekizai templatetags to be declared in the top template makes authentic theming quite a pain; e.g. you cannot do:

{% extends 'theme/common.html' %}
{% block extra-head %}
{% render_block "css" %}
{% render_block "js" %}
{% endblock %}

Fichiers

Révisions associées

Révision f145e127 (diff)
Ajouté par Benjamin Dauvergne il y a presque 9 ans

templates: replace use of sekizai by our own template tags

fixes #7206

Révision 66815866 (diff)
Ajouté par Benjamin Dauvergne il y a presque 9 ans

templates: replace use of sekizai by our own template tags

fixes #7206

Révision 4c1b2b0f (diff)
Ajouté par Benjamin Dauvergne il y a presque 9 ans

templates: replace use of sekizai by our own template tags

fixes #7206

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

Are you sure it does not work or are you quoting the documentation ? It seems to me that the way authentic uses sekizai it should work.

The principle of sekizai is to set a dictionary in a context variable before template rendering and during rendering the dictionnary is filled by subtemplates.

I think that this behaviour is broken if you put the render_block and addtoblock tag in different blocks of the same template as I think you cannot be sure of the ordering of the rendering. But that is not how authentic use sekizai, we begin by rendering subtemplates for the different authentication frontend class, the we pass the rendered blocks to the final template of the login view. Maybe in this case it still works.

#2

Mis à jour par Frédéric Péters il y a presque 9 ans

I tried and it didn't work, for sure. But I set this ticket as low priority, so this can be discussed when we have more time on our hands.

#3

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

  • Fichier 0001-templates-remove-sekizai-tags-and-rename-base.html.patch ajouté
  • Assigné à mis à Benjamin Dauvergne
  • Version cible mis à 2.1.20
  • Patch proposed changé de Non à Oui
#4

Mis à jour par Frédéric Péters il y a presque 9 ans

The attached patch is about fargo, not authentic.

#5

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

  • Fichier 0001-templates-remove-sekizai-tags-and-rename-base.html.patch supprimé
#7

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#9

Mis à jour par Benjamin Dauvergne il y a presque 9 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF