Projet

Général

Profil

Development #27714

ajouter des blocs de personnalisation de l'avant/après navigation

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans la continuité de #26459, pour la navigation.


Fichiers

Révisions associées

Révision e03ea347 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

templates: add blocks for custom markup around navigation (#27714)

Historique

#1

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

(ça me servira à Toulouse pour poser le code de suivi sous la navigation).

#2

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

Comme c'est identique à #26459 il va suivre la même logique, mais par curiosité, pourquoi on n'a pas plutôt fait quelque chose comme :

  <div id="nav-wrapper">
  {% block "nav-wrapper" %}
    <div id="nav"> ... </div>
  {% endblock %}
  </div>

qui permettrait des surcharges avec :

{% block "nav-wrapper" %}
  ici le pre-blabla
  {{ block.super }}
  ici le post-blabla
{% endblock %}
#3

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

Comme c'est identique à #26459 il va suivre la même logique, mais par curiosité, pourquoi on n'a pas plutôt fait quelque chose comme :

Je n'ai pas de réponse historique, à vaguement réfléchir maintenant,

{% block "nav-wrapper" %}
  ici le pre-blabla
  {{ block.super }}
  ici le post-blabla
{% endblock %}

À imaginer ça dans un variants/xxx/theme.html, ça se trouverait peut-être éliminé par un template qui en ferait un extends, qui ferait {% block nav-wrapper %} et perdrait l'avant/après. Comme l'héritage de templates est quelque chose qui en plus change entre 1.8 et 1.11, peut-être que l'option était simplement de ne pas trop se poser de questions.

#4

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

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

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

(...) simplement de ne pas trop se poser de questions.

Parfaitement valable ; je ne suis pas toujours à l'aise avec les {{block.super}}.

#5

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit e03ea34753de9d2a21fc95ea8421e505eed66bbc
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Nov 1 12:08:53 2018 +0100

    templates: add blocks for custom markup around navigation (#27714)
#6

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

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

Formats disponibles : Atom PDF