Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Texte et page de présentation #65

Closed
camillemonchicourt opened this issue May 17, 2019 · 19 comments
Closed

Texte et page de présentation #65

camillemonchicourt opened this issue May 17, 2019 · 19 comments
Labels
enhancement New feature or request

Comments

@camillemonchicourt
Copy link
Member

Il semblerait qu'aucune zone ou page ne soit prévue pour présenter le portail, la démarche, le contexte. Assez brut et déconcertant du coup. Et peu explicite.

Il faudrait prévoir à minima :

  • Un bloc HTML sur la home surcouchable et désactivable permettant de présenter le portail, le projet
  • Une page de présentation HTML surcouchable permettant d'aller plus loin dans la présentation.

Sur le modèle de ce qui a été fait dans GeoNature-atlas par exemple

@camillemonchicourt camillemonchicourt added the enhancement New feature or request label May 17, 2019
@camillemonchicourt
Copy link
Member Author

Bloc HTML sous le bloc de photo, où les gens mettent ce qu'ils veulent en HTML avec les classes Bootstrap et la surcouche CSS si besoin.
Un template HTML par langue ou dans les fichiers de traductions ?
Bien récupérer cette customisation à chaque mise à jour.

@20cents
Copy link
Contributor

20cents commented Sep 1, 2020

Demo du texte d'intro sur la home :
http://vps587786.ovh.net/
Je l'ai stocké en bdd dans la table de conf, il me semble que c'est plus simple pour l'administrer.
On pourrait un jour avoir un éditeur dans l'admin.
La table de conf est key:value
D'abord on regarde s'il y a home_intro_<lang> sinon juste home_intro sinon rien

@camillemonchicourt si t'es ok sur le principe, je fais pareil avec la page de présentation, non ?

@camillemonchicourt
Copy link
Member Author

OK, merci pour cette solution technique. Pas le plus simple à éditer si on met un long texte avec de la mise en forme, voire des images etc, notamment pour la page de présentation. Mais ça permet d'être plus générique, multilangue et d'envisager de gérer le contenu dans l'Admin plus tard.
OK donc pour moi comme fonctionnement. Je me demande juste pourquoi le code fait référence en dur à home_intro_fr si le système de langue est générique ? On n'est pas forcément en français.

J'aurai plutôt mis le texte d'introduction avant la mosaïque des images. Mais pas forcément évident pour tous ?
En faire un paramètre pour le avant ou après ?
Ou sinon on le met plutôt avant et on verra à l'usage ?

Pour la page de présentation, je me demande si on la nomme "Présentation" (presentation_page) ou plutôt "A propos" (about_page) ?

@geobrun
Copy link

geobrun commented Sep 1, 2020 via email

@camillemonchicourt
Copy link
Member Author

Ah super, merci pour le lien et ravi que vous ayez pu déployer l'outil. Une version 1.1.0 est en préparation comme tu as du le comprendre (https://github.com/PnX-SI/GeoPaysages/projects/1).

L'idée de le mettre dans la mosaïque est intéressante.
Mais comme on vise un truc générique, pas sur que ça s'adapte bien à tous les cas.
Donc je resterai sur un bloc avant ou après la mosaïque pour commencer, et à voir si on peut faire plus complexe ensuite ?

@geobrun
Copy link

geobrun commented Sep 2, 2020

Effectivement, la solution que je propose peut être problématique car si quelqu'un veut mettre un long pavé dans le bloc de texte, cela va vite devenir illisible (trop petit) !

@camillemonchicourt
Copy link
Member Author

Oui et en l'état cela ne fonctionne pas sur mobile.
Mais ça me semble une piste intéressante à creuser pour une évolution prochaine.

@geobrun
Copy link

geobrun commented Sep 2, 2020

Effectivement, je n'ai pas encore programmer la partie mobile. Je fera ça prochainement, ça ne devrait pas être compliqué à faire. Je suis parti du principe que la partie mobile n'était pas prioritaire pour un observatoire du paysage : des photos se regardent mieux sur un véritable ordinateur que sur un smartphone ! :)

@xavyeah39
Copy link
Collaborator

Salut,

De mon côté j'ai aussi opté pour l'ajout d'un paramètre en BDD dans la table conf pour activer (true) ou non (false) un bloc html que j'ai appelé home_welcome_section.

Pour ça, j'ai ajouté un bloc avec condition sur ce paramètre dans le template home.html, mis les blocs dans des sections et ajouté du css dans home.css qui s'applique sur cette nouvelle section nommée welcome (surcouchable dans custom.css du coup).

Pour les contenu de textes tout est dans le fichier d'internationalisation messages.po avec des variables pour le titre de bienvenue (welcome_title), le texte chapeau (welcome_text) et les autres éléments (welcome_button et welcome_scroll_text).

Le tout placé au dessus de la mosaïque avec une image de fond.

Aperçu de ce que ça donne ici (idem, site en construction --> ne pas diffuser)

Par contre, faute de temps, je ne me suis pas attaqué à :

  • l'affichage responsive sur mobile
  • essayer de le gérer avec des templates customisables (comme dans geonature-atlas)
  • mettre au propre et proposer une PR ;)

@20cents
Copy link
Contributor

20cents commented Sep 2, 2020

@camillemonchicourt

Je me demande juste pourquoi le code fait référence en dur à home_intro_fr si le système de langue est générique ?

Il me semble que le code n'a pas de ref en dur à home_intro_fr
En revanche j'ai modifié le nom de la variable intro_fr pour intro_locale dans le template home.html

Sinon, about_page me paraît bien.

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Sep 2, 2020

@xavyeah39, la solution retenue pour le moment est pas uniquement un paramètre activant ou non le bloc dans la BDD, mais bien que tout le contenu du bloc soit dans la BDD, pour gérer aussi le multilangue, et envisager à terme de l'éditer dans l'ADMIN. Donc pas de fichier HTML custom, mais des champs dans la BDD dans ce qui a été proposé.
Le bloc est plus simple et générique, même si on peut en surcoucher le CSS dans le fichier de surcouche CSS.

@20cents, oui je parlai de la variable intro_fr, merci pour le renommage.
OK pour faire pareil pour la page de présentation, nommée "A propos", donc about_page. Merci.

@20cents
Copy link
Contributor

20cents commented Sep 2, 2020

J'ai mis un param en table de conf pour la position du du texte d'intro.
Si ce param n'est pas renseigné ou que ça valeur n'est ni top ni bottom alors le texte n'apparaît pas

@xavyeah39
Copy link
Collaborator

@camillemonchicourt,

@xavyeah39, la solution retenue pour le moment est pas uniquement un paramètre activant ou non le bloc dans la BDD, mais bien que tout le contenu du bloc soit dans la BDD, pour gérer aussi le multilangue, et envisager à terme de l'éditer dans l'ADMIN. Donc pas de fichier HTML custom, mais des champs dans la BDD dans ce qui a été proposé.
Le bloc est plus simple et générique, même si on peut en surcoucher le CSS dans le fichier de surcouche CSS.

OK.
Si je comprend bien, on pourra donc directement injecter/personnaliser le HTML du bloc d'intro dans le paramètre ad-hoc de la table conf (plutôt que le HTML soit imposé dans le template) ?
Si oui, top !

@camillemonchicourt
Copy link
Member Author

Oui c'est ça, avec possibilité de le définir par langue.

OK pour le paramètre de position, impeccable.

Il faudrait maintenant documenter tous les paramètres disponibles et leurs valeurs.

@20cents
Copy link
Contributor

20cents commented Sep 2, 2020

http://vps587786.ovh.net/about
Désolé pour le libellé de commit pas top

@camillemonchicourt
Copy link
Member Author

OK merci.
Dans ce cas, tu as mis quoi comme contenu de test dans le champs de la BDD ?

@20cents
Copy link
Contributor

20cents commented Sep 2, 2020

@camillemonchicourt
Copy link
Member Author

OK pour moi, merci.

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Sep 2, 2020

Dans cet exemple, le contenu du paramètre page_about_content dans la BDD est :

<div class="container">
    <div class="row">
      <div class="col">
        <img src="/static/images/sample.png">
      </div>
      <div class="col">
        <p>
          Lorem ipsum dolor sit amet, consectetur adipiscing elit. Phasellus magna lectus, auctor nec dapibus a, sagittis
          sit amet diam. In vehicula est quam, id volutpat arcu molestie sed. Nam vestibulum tempor urna. Nulla in lectus
          et augue tempus pellentesque quis non justo. Orci varius natoque penatibus et magnis dis parturient montes,
          nascetur ridiculus mus. Etiam pharetra nisi ligula, at volutpat leo condimentum eu. Vivamus pretium blandit
          interdum.
        </p>
        <p>
          Quisque maximus vitae tortor eu dignissim. Sed sed lectus consequat, molestie dolor a, auctor nisl. Praesent
          aliquam id lacus ac congue. Nulla quis ultrices eros. Nullam efficitur nulla quis magna sollicitudin, sit amet
          placerat felis dictum. Ut eu accumsan risus, scelerisque varius augue. Curabitur eleifend in orci at
          vestibulum.
        </p>
        <p>
          Sed convallis ornare venenatis. Donec vitae molestie felis. Morbi tortor erat, ultrices sed faucibus sit amet,
          viverra at lorem. Ut a eros tempus enim euismod tempor eget ut nunc. Donec nec urna ac ipsum dapibus tempor eu
          a nibh. Cras nisi ante, congue quis pulvinar ac, feugiat nec justo. Aliquam et pharetra sem.
        </p>
        <p>
          Sed convallis ornare venenatis. Donec vitae molestie felis. Morbi tortor erat, ultrices sed faucibus sit amet,
          viverra at lorem. Ut a eros tempus enim euismod tempor eget ut nunc. Donec nec urna ac ipsum dapibus tempor eu
          a nibh. Cras nisi ante, congue quis pulvinar ac, feugiat nec justo. Aliquam et pharetra sem.
        </p>
      </div>
    </div>
  </div>

Le titre page_about_title est un texte qui est affiché en <h1> centré au début de la page.
Le nom de la page ABOUT dans le MENU est défini dans les fichiers de langue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

4 participants