Bug #7206
sekizai is impossible to deal with theming-wise
100%
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 %}
Files
Associated revisions
templates: replace use of sekizai by our own template tags
fixes #7206
templates: replace use of sekizai by our own template tags
fixes #7206
History
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
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.
Updated by Frédéric Péters about 10 years ago
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.
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
- File 0001-templates-remove-sekizai-tags-and-rename-base.html.patch added
- Assignee set to Benjamin Dauvergne (retour le 1er août)
- Target version set to 2.1.20
- Patch proposed changed from No to Yes
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
- File deleted (
0001-templates-remove-sekizai-tags-and-rename-base.html.patch)
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
- Status changed from Nouveau to Résolu (à déployer)
- % Done changed from 0 to 100
Appliqué par commit authentic2|f145e12706fab1b31f5fa7fe87d680b056994640.
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
Appliqué par commit authentic2|66815866144e08fd2a9c9f8e5b8cec2c7f2a0baf.
Updated by Benjamin Dauvergne (retour le 1er août) about 10 years ago
- Status changed from Résolu (à déployer) to Fermé
templates: replace use of sekizai by our own template tags
fixes #7206