Development #18925
Découvrir automatiquement les placeholders
0%
Description
Pour le moment on passe par de la config pour lister les placeholders possibles dans un template, ça serait bien de pouvoir zapper ça, que ça soit défini au plus près, dans le template même. Ça permettrait par exemple pour une intégration particulière de simplement ajouter la déclaration dans le template, sans devoir passer par une rédéfinition intégrale des templates (ex hautes alpes avec {% placeholder "headerlogo" %}).
Syntaxe à réfléchir.
{% placeholder "footer" %} {% placeholder_label %}{% trans "Footer" %}{% endplaceholder_label %} {% placeholder_options acquired=True %} ... {% endplaceholder %}
Pour la compat le endplaceholder pourrait être optionnel, le traitement s'arrêtant alors au {% placeholder %} suivant.
(pas du tout réfléchi à comment techniquement mettre ça en place)
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-general-extract-placeholder-details-from-templates-i.patch 0001-general-extract-placeholder-details-from-templates-i.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Bien beau; non seulement ça a l'avantage de ne plus avoir à gérer ces settings manuellement, ça permet par exemple aussi au portail agent de ne plus présenter une zone "footer" qui n'existe pas dans le template.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Et plutôt qu'introduire des nouveaux tags, j'ai juste étendu l'existant :
{% trans "Footer" as name %} {% placeholder "footer" name=name acquired=True %}
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
J'aurai préféré un parcours de l'AST du template, j'aime pas trop le fait qu'ils puissent y avoir des effets de bord, mais bon c'est le pedantic man en moi qui parle.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Aussi je vois bien que l'interaction avec trans serait difficile à obtenir sans ça.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
T'as pas viré placeholder_option, ça fait double emploi.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Fermé
Ça n'a pas été noté mais ça a été poussé.
general: extract placeholder details from templates instead of settings (#18925)