Refacto du template user-info.html permettant les surcharges (#72598) #85
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/72598-user-info-refacto"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
WIP: Refacto du template user-info.html permettant les surcharges (#72598)to Refacto du template user-info.html permettant les surcharges (#72598)Beaucoup de commit, mais tous petits.
J'en ai profité pour supprimer les templates de label qui deviennent inutiles.
(mais ça peut être dans une autre PR).
Possible aussi de tout stasher.
@Serghei, pour la MEL, on perd la condition sur l'image. Dis moi si c'est problématique.
Attention juste retirer le WIP ne fait pas évoluer le statut côté redmine.
(sur les intitulés de commits, il y aurait tpl → template).
Je trouve qu'on perd à ne plus avoir ces fichiers séparés, on passe d'un fichier où juste écrire "Je me connecte" à un fichier où écrire
Non, ce n'est pas un souci.
@ -152,2 +152,4 @@
{% skeleton_extra_placeholder top-links %}
{% include 'includes/user-info.html' %}
{% end_skeleton_extra_placeholder %}
{% endblock %}
Indentation ratée; je pensais que djhtml gérerait.
C'est djhtml qui m'inpose cette indentation :/
voir ligne 161 où djhtml a déjà indenté de la sorte :
Attention il y a aussi à gérer dans imio-publik-themes.
Oui j'ai regardé. J'attendais de voir si le code était ok de ce côté.
C'est pour pas dépasser parfois les 80 caractères :)
Pas faux. C'est dans l'idée de revenir à une cohérence avec les autres templates, où jamais on n'utilise d'include pour juste un label, mais des blocks.
1 seul fichier reprend l'avantage lorsqu'il y a plusieurs labels a changer et je trouve ça mieux en lecture.
bd7ff62c6b
to77fa01266d
Pour avancer sur ce qui m'intéresse, j'ai supprimé les commits relatifs aux labels. Les includes labels sont restaurés.