Development #52275
Autre #43311: Plan de refactorisation du block `user-info`
Faciliter le déplacement du block user-info dans le thème
0%
Description
Plusieurs thèmes déplacent le bloc user-info en header-pre (et j'en ai besoin pour l'update du theme de CD06)
Et ce de manière assez complexe :
- poser la variable `include_top_links = False` pour supprimer les liens présents dans le header (ce qui sémantiquement est très confusant : je demande à supprimer mais en fait je déplace).
- surcharger theme.html pour poser le bloc {% block user-info %} au sein d'un autre bloc
- entourer manuellement le nouveau bloc par un wrapper comme celui d'origine `<div id="toplinks">` car déplacer le bloc ne déplace par son wrapper, et tous les styles CSS des user-links dépendent de ce wrapper.
Je pense qu'il y a moyen de simplifier/clarifier un peu les choses.
Fichiers
Révisions associées
grand-chambery: move user-info block code into specific template file (#52275)
lille: isolate user-info template (#52275)
entrouvert: isolate user-info template (#52275)
haute-garonne-cd31: isolate user-info template (#52275)
metz-metropole-2019: isolate user-info template (#52275)
sables-d-olonne: isolate user-info template (#52275)
toulouse-metropole: erase top-links instead user-info (#52275)
villeneuve-dascq: isolate user-info template (#52275)
grenoble-metropole-2019: replace block user-info by top-links (#52275)
montpellier: replace block user-info by top-links (#52275)
moselle: replace block user-info by top-links (#52275)
hellemmes: remove "include_top_links: false" (#52275)
lomme: remove "include_top_links: false" (#52275)
gpseo: isolate user-info template (#52275)
Historique
Mis à jour par Thomas Jund il y a environ 3 ans
- Tracker changé de Autre à Development
- Assigné à mis à Thomas Jund
Mis à jour par Thomas Jund il y a environ 3 ans
- Sujet changé de Faciliter le déplacement du block user-info dasn le thème à Faciliter le déplacement du block user-info dans le thème
Mis à jour par Thomas Jund il y a environ 3 ans
- Fichier 0001-themes-easily-move-the-user-info-block-52275.patch 0001-themes-easily-move-the-user-info-block-52275.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Demande d'avis sur cette proposition avant de modifier les themes :
Je reprends le système que j'avais introduit avec le block 'back-to-top'
- ajout du {% block top-links %} pour avoir un block conteneur facile à déplacer.
- ajout du {% block header-content %} (pour supprimer le block 'top-links' et intéressant également pour injecter du code dans le header, comme la nav par exemple)
- déplacement du code des user-links au sein de son propre ficher `user-info.html` (pour mieux identifier les thèmes qui surcharge le composant et éventuellement pour l'appeler plusieurs fois)
- la condition sur la variable `include_top_links` est déplacé au sein du fichier `user-info.html` pour supprimer le composant quel que soit son/ses emplacement(s).
Je n'ai pas réglé cette histoire de sémantique confuse entre 'top-links' et 'user-info', le patch serait trop imposant (mais je suis partisan de n'utiliser que 'user-info')
une mise à jour des templates d'authentic-cut sera nécessaire, avec idéalement l'import du fichier en {% include %} pour éviter une redondance de code du composant (Serghei OK pour la modif).
Mis à jour par Serghei Mihai il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Cela me semble bon.
Mis à jour par Thomas Jund il y a environ 3 ans
- Statut changé de Solution validée à En cours
OK, je patch les thèmes.
Mis à jour par Thomas Jund il y a presque 3 ans
- Statut changé de En cours à Solution proposée
Thèmes patchés dans la branche.
wip/52275-isolate-user-info/
Idem avec 2 themes patchés dans dépot imio (branch du même nom)
Mis à jour par Thomas Jund il y a presque 3 ans
2e passe sur les thèmes de mon côté je pense n'avoir rien oublié.
Je n'ai pas 'refactoré' Rouen pour simplifier son code (trop spécifique), mais rien de cassé puisqu'il redéfini entièrement `theme.html`.
Pour les thèmes qui n'utilisent pas combo, il faut maintenant importer le code via un tag {% include %}.
Mis à jour par Serghei Mihai il y a presque 3 ans
- Statut changé de Solution proposée à En cours
Dans user-info.html
il manque un {% load combo %}
pour avoir accès au template tag skeleton_extra_placeholder
Mis à jour par Thomas Jund il y a presque 3 ans
- Fichier 0001-themes-easily-move-the-user-info-block-52275.patch 0001-themes-easily-move-the-user-info-block-52275.patch ajouté
- Statut changé de En cours à Solution proposée
avec {% load combo %}
Mis à jour par Serghei Mihai il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Jund il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commits publik-base-theme
commit d765b03d874fea69ff11fec5b6f179894591bf22 (HEAD -> main, origin/wip/52275-isolate-user-info, origin/main, wip/52275-isolate-user-info) Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 13:00:00 2021 +0200 gpseo: isolate user-info template (#52275) commit 6f2e367a2c2bc35f7edf1342941dfccb522ff018 Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 12:51:14 2021 +0200 lomme: remove "include_top_links: false" (#52275) commit 0259bd09cedd5c234dc95a6f0fd29dde4a1c0ace Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 12:50:28 2021 +0200 hellemmes: remove "include_top_links: false" (#52275) commit f4c3dffc644f7af4c97a5139d64871c92c3d9876 Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 12:18:15 2021 +0200 moselle: replace block user-info by top-links (#52275) commit 946e92c55aa715ed193ae0349e2b7d30bf38ddab Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 12:11:38 2021 +0200 montpellier: replace block user-info by top-links (#52275) commit 9aa53c70634d2587f817f832160ecfd6bf297eab Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Mar 30 12:08:29 2021 +0200 grenoble-metropole-2019: replace block user-info by top-links (#52275) commit c210f0bce3ee397caf529839d7a9c852b746da54 Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Mar 29 17:11:44 2021 +0200 villeneuve-dascq: isolate user-info template (#52275) commit 2bcd87decf55baf2bd77711aa1f4af9d4ef08b9e Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 17:44:14 2021 +0100 toulouse-metropole: erase top-links instead user-info (#52275) commit bccbb1540040b1df67659682d32f458b52a2f2c2 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 17:34:31 2021 +0100 sables-d-olonne: isolate user-info template (#52275) commit b10169244356170be589e2504c2256749f988c19 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 17:26:38 2021 +0100 metz-metropole-2019: isolate user-info template (#52275) commit 292d4d80ce94700bf4302d43d822995b611ee652 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 17:08:44 2021 +0100 haute-garonne-cd31: isolate user-info template (#52275) commit 5edf8e00d754bce4139f9d139abd4a18f9e27363 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 17:08:04 2021 +0100 entrouvert: isolate user-info template (#52275) commit 24e1ea0a7e2f78316fce053fb81395ec46943fa2 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 16:30:11 2021 +0100 lille: isolate user-info template (#52275) commit 8fc249aba7877b5f64c362c6f1e5e4490caf1c03 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Mar 26 16:34:19 2021 +0100 grand-chambery: move user-info block code into specific template file (#52275) commit b9cdb20a1dbe16d21567685cd6a7d68d0580e219 Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Mar 22 14:54:10 2021 +0100 themes: easily move the user-info block (#52275) * add block 'header-content' * add block 'top-links' * move user-info code in own file
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
themes: easily move the user-info block (#52275)