Projet

Général

Profil

Development #44616

déplacer blocks footer-top & -bottom à l'intérieur de #footer

Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
30 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Les {% block %} `footer-top` et `footer-bottom` se trouve en dehors de la `div#footer`

Mais si on veut être cohérent avec les autres blocs du template 'nav, header, content` ils devraient se retrouver juste au-dessus et en dessous du contenu du footer en sein de #footer.

L'ajout de contenu avant et après #footer-wrapper devrait être dédié à des {% block %} `footer-pre` et `footer-post`.


Fichiers

Révisions associées

Révision e2a92943 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

move blocks footer-top & -bottom inside #footer; add footer-pre & -post (#44616)

Révision 5fbed090 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

montlouis-sur-loire: use footer-pre instead of footer-top (#44616)

Révision 85a8069d (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

rochefort: use footer-post instead of footer-bottom (#44616)

Révision c7e7d698 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

toodego: use footer-pre instead of footer-top block (#44616)

Historique

#2

Mis à jour par Nicolas Roche il y a presque 4 ans

J'ai l'impression qu'il faudrait aussi traiter le thème de "Mon Strasbourg" : templates/variants/strasbourg-2018/theme.html

#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

Strasbourg a son propre fichier theme.html.
Ce Patch ne l'affecte donc pas, voilà pourquoi je n'y ai pas touché.
Je ne sais pas s'il est préférable de le mettre à jour à chaque fois qu'on met à jour le theme du core ou de le laisser en état.

#4

Mis à jour par Nicolas Roche il y a presque 4 ans

Strasbourg a son propre fichier theme.html. Ce Patch ne l'affecte donc pas,

Ok, pigé (il ne dérive pas du thème général, comme Rochefort par exemple).

Pour gagner en cohérence dans les thèmes que tu modifies, peut-être aussi mettre à jour les identifiants des div imbriquées, et donc également les css relatifs ?

  • Montlouis-sur-Loire

templates/variants/montlouis-sur-loire/combo/page_template.html

{% block footer-pre %}
<div id="footer-top">
...

static/montlouis-sur-loire/_custom.scss

div#footer-top {
    background: white;

  • Rochefort

templates/variants/rochefort/theme.html

{% block footer-post %}
<div id="footer-bottom-wrapper">
  <div id="footer-bottom">

static/rochefort/_custom.scss

div#footer-bottom-wrapper {
...
    div#footer-bottom {
        max-width: $width;

#5

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

Pour gagner en cohérence dans les thèmes …

Bien vu.

#7

Mis à jour par Nicolas Roche il y a presque 4 ans

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

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit c7e7d69894e201af206d1497e936ce45a691dd23 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Jun 30 11:20:36 2020 +0200

    toodego: use footer-pre instead of footer-top block (#44616)

commit 85a8069d6a2eac2273a4414f282c6db9341e99c4
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Jun 30 11:19:43 2020 +0200

    rochefort: use footer-post instead of footer-bottom (#44616)

commit 5fbed09030c35318f7a29968178a0c4fed219125
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Jun 30 11:19:11 2020 +0200

    montlouis-sur-loire: use footer-pre instead of footer-top (#44616)

commit e2a92943d6f188d6bf7935276810c9a62e42eea5
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Jun 30 11:16:34 2020 +0200

    move blocks footer-top & -bottom inside #footer; add footer-pre & -post (#44616)

#9

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF