Projet

Général

Profil

Development #8081

possibilité de modifier le contenu du footer

Ajouté par Thomas Noël il y a plus de 8 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
21 août 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Actuellement le footer est en dur dans le index.html.

Il serait intéressant de pouvoir le modifier globalement depuis le index.json (injection de code HTML, tout simplement).


Fichiers

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Assigné à changé de Ghislain Loaec à Frédéric Péters
#2

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Fichier 0001-add-support-for-placing-a-custom-footer-in-the-manif.patch ajouté
  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui

Voilà, il y a un peu de gymnastique pour continuer à utiliser par défaut un template, utilisant les variables se trouvant dans le manifest.

#3

Mis à jour par Frédéric Péters il y a plus de 8 ans

#4

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Fichier 0001-add-support-for-placing-a-custom-footer-in-the-manif.patch supprimé
#5

Mis à jour par Thomas Noël il y a plus de 8 ans

J'ai pas envie de me plonger dnas le code, mais pourquoi ne pas avoir fait un "simple" if(o.footer) o.footer else le-code-par-default ... ?

#6

Mis à jour par Frédéric Péters il y a plus de 8 ans

Parce que je me disais qu'on pouvait vouloir continuer à profiter des fonctionnalités de template pour le footer.

#7

Mis à jour par Thomas Noël il y a plus de 8 ans

Je pense que mon cerveau refuse de comprendre. Je vais dire Ack juste parce que ça marche.

(moi j'avais bêtement pensé :

--- a/www/index.html
+++ b/www/index.html
@@ -307,6 +307,9 @@
                 <footer class="navbar navbar-inverse momo-footer">
                     <div class="container-fluid clearfix">
                         <div class="row">
+                            {% if(o.footer){ %}
+                            {%# o.footer %}
+                            {% } else { %}
                             <div class="col-xs-6 navbar-btn text-left">
                                 <a href="https://dev.entrouvert.org/projects/momo/" 
                                     class="btn btn-success pull-left momo-repo" 
@@ -317,6 +320,7 @@
                                 <a href="mailto:{%= o.meta.contact %}" 
                                     class="btn btn-success pull-right momo-contact">{%= o.meta.contact %}</a>
                             </div>
+                            {% } %}
                         </div>
                     </div>
                 </footer>
)

#8

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)

(tu ne comprends pas qu'on veuille vouloir un footer personnalisé qui soit malgré tout un template ?)

commit 1ef0a5b2d24764c4d40d050e21f7235dc92ae711
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 22 10:00:26 2015 +0200

    add support for placing a custom footer in the manifest (#8081)
#9

Mis à jour par Frédéric Péters il y a environ 5 ans

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

Formats disponibles : Atom PDF