Projet

Général

Profil

Development #52275

Autre #43311: Plan de refactorisation du block `user-info`

Faciliter le déplacement du block user-info dans le thème

Ajouté par Thomas Jund il y a environ 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
22 mars 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision b9cdb20a (diff)
Ajouté par Thomas Jund il y a presque 3 ans

themes: easily move the user-info block (#52275)

  • add block 'header-content'
  • add block 'top-links'
  • move user-info code in own file

Révision 8fc249ab (diff)
Ajouté par Thomas Jund il y a presque 3 ans

grand-chambery: move user-info block code into specific template file (#52275)

Révision 24e1ea0a (diff)
Ajouté par Thomas Jund il y a presque 3 ans

lille: isolate user-info template (#52275)

Révision 5edf8e00 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

entrouvert: isolate user-info template (#52275)

Révision 292d4d80 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

haute-garonne-cd31: isolate user-info template (#52275)

Révision b1016924 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

metz-metropole-2019: isolate user-info template (#52275)

Révision bccbb154 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

sables-d-olonne: isolate user-info template (#52275)

Révision 2bcd87de (diff)
Ajouté par Thomas Jund il y a presque 3 ans

toulouse-metropole: erase top-links instead user-info (#52275)

Révision c210f0bc (diff)
Ajouté par Thomas Jund il y a presque 3 ans

villeneuve-dascq: isolate user-info template (#52275)

Révision 9aa53c70 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

grenoble-metropole-2019: replace block user-info by top-links (#52275)

Révision 946e92c5 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

montpellier: replace block user-info by top-links (#52275)

Révision f4c3dffc (diff)
Ajouté par Thomas Jund il y a presque 3 ans

moselle: replace block user-info by top-links (#52275)

Révision 0259bd09 (diff)
Ajouté par Thomas Jund il y a presque 3 ans

hellemmes: remove "include_top_links: false" (#52275)

Révision 6f2e367a (diff)
Ajouté par Thomas Jund il y a presque 3 ans

lomme: remove "include_top_links: false" (#52275)

Révision d765b03d (diff)
Ajouté par Thomas Jund il y a presque 3 ans

gpseo: isolate user-info template (#52275)

Historique

#1

Mis à jour par Thomas Jund il y a environ 3 ans

  • Tracker changé de Autre à Development
  • Assigné à mis à Thomas Jund
#2

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
#3

Mis à jour par Thomas Jund il y a environ 3 ans

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).

#4

Mis à jour par Serghei Mihai il y a environ 3 ans

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

Cela me semble bon.

#5

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.

#6

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)

#7

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 %}.

#9

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

#10

Mis à jour par Thomas Jund il y a presque 3 ans

avec {% load combo %}

#11

Mis à jour par Serghei Mihai il y a presque 3 ans

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

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
#13

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

Formats disponibles : Atom PDF