Bug #53558
villeneuve-d'asq: 2 icônes Accueil dans le header
0%
Description
Retirer la seconde icône qui semble liée à un défaut de compatibilité descendante
entre {{ block.super }} et le filtre skeleton_extra_placeholder.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a presque 3 ans
- Fichier 0001-villeneuve-dascq-remove-dupplicated-icon-on-toplinks.patch 0001-villeneuve-dascq-remove-dupplicated-icon-on-toplinks.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Comme expliqué dans le commentaire,
j'ai recopié remplacé {{ block.super }} dans le template /villeneuve-dascq/includes/user-info.html
par le bloc user-info de templates/includes/user-info.html,
parce que sinon ce premier template semble être chargé 2 fois en django 1.11.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
Il n'y pas d'autre solution ?
J'aime pas voir un template cloné, cette pratique complique beaucoup l'évolution de publik-base-theme. L'objectif de déplacer le code du bloc user-info dans son fichier était justement
- de proposer une solution pour éviter ce genre de pratique en permettant de déplacer le code sans le cloner
- d'identifier plus simplement les themes qui surcharges le bloc en analysant uniquement les fichiers template `user-info.html`
Mis à jour par Frédéric Péters il y a presque 3 ans
On est toujours sur un diagnostic incertain vu que Nicolas n'a pas pu faire tourner son environnement local en 1.11 mais si c'est bien l'origine, on peut juste accepter ça et faire un ticket pour retirer ça dans quelques semaines et voilà.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
(ok, je n'avais pas compris que c'était un patch provisoire en attendant la montée de version)
Mis à jour par Nicolas Roche il y a presque 3 ans
C'est juste un petit peu moins incertain : Paul reproduit le bug sur son instance pas à jour (en django 1.11).
Par contre moi je n'ai pas poussé plus loin : il me faudrait tourner en python3.7 (qui n'est pas dispo en testing) et ensuite bidouiller l'ansible de publik-devinst.
Il n'y pas d'autre solution ?
Se serait de débuger le filtre combo skeleton_extra_placeholder pour qu'il fonctionne dans ce cas de figure (avec un "super" dans le template qu'il traite) en django 1.11, mais comme on va prochainement passer en django 2.2 ça m'irait bien un ack de complaisance pour passer à autre chose.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit efcb24ace75ae76ef87c858c1a4a58d276d02e64
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Thu Apr 29 14:25:25 2021 +0200
villeneuve-dascq: remove dupplicated icon on toplinks (#53558)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a presque 3 ans
- Lié à Bug #53699: Villeneuve d'Ascq, 'block' tag with name 'placeholder-user-info' appears more than once ajouté
villeneuve-dascq: remove dupplicated icon on toplinks (#53558)