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

Migration vers Bootstrap4 ? #230

Closed
lpofredc opened this issue Mar 16, 2020 · 14 comments
Closed

Migration vers Bootstrap4 ? #230

lpofredc opened this issue Mar 16, 2020 · 14 comments
Projects

Comments

@lpofredc
Copy link
Contributor

GeoNature-atlas est développé avec Bootstrap 3.3. Je me propose pour migrer GeoNature-atlas vers Bootstrap 4. Cela nécessiterait aussi de modifier la source des icônes (remplacement de Glyphicon par font-awesome?).

Des remarques? contraintes ?

Merci.

@camillemonchicourt
Copy link
Member

Merci !

@lpofredc
Copy link
Contributor Author

Oui ;) Je ne suis pas webdesigner mais si certains ont des propositions à faire pour améliorer l'interface de l'appli, n'hésitez pas. C'est le moment ;)

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Mar 17, 2020

Globalement c'est simple et fonctionnel, donc pas de changement particulier en tête.

@lpofredc
Copy link
Contributor Author

Ok, merci. J'en profite pour restructurer les templates avec une trame principale (core/layout.html contenant les entêtes, les barres de navigation, le pied de page) et pour chaque page en séparant les cards dans des fichiers distincts, plus facile de s'y retrouver car les pages sont longues. Flask a également des possibilités intéressantes d'organisation des contenus avec les balises jinja {% block monbloc %}{% endblock %}.
cf. le travail en cours ici avec les dossiers:

  • core (éléments partagés sur tous les templates)
  • home (page d'accueil) > _main.html est le squelette de la page qui fait appel aux différentes briques.
  • specieSheet (fiche espèce) > idem home

cf. https://github.com/lpofredc/GeoNature-atlas/tree/bootstrap4/templates

@camillemonchicourt
Copy link
Member

Voir la PR : #232

@lpofredc
Copy link
Contributor Author

lpofredc commented Apr 1, 2020

Nouvelle proposition de rendu de la fiche commune, remplacement de datatables par les listes avec medias de bootstrap, egalement filtrable sur les taxons
image

Screenshot_2020-04-02 Commune de Clermont-Ferrand cbiodiv org - Atlas - LPO Auvergne-rhône-Alpes(1)

@lpofredc
Copy link
Contributor Author

Un aperçu des modifications apportées par cette migration vers BootStrap4

https://atlas.cbiodiv.org/

@camillemonchicourt
Copy link
Member

Merci, beaucoup de choses intéressantes.

@Splendens
Copy link
Contributor

Bonjour,

Je voudrais me baser sur l'atlas en version BS4 avant de continuer à faire mes propres modifications, du coup j'ai récupéré directement l'atlas ici : https://github.com/lpofredc/GeoNature-atlas/tree/bootstrap4 et je l'utilise sur mon instance de test.

Déjà merci @lpofredc ! Les fonctionnalités pour la recherche d'autres types de territoires, les fiches communes avec les territoires associés, la visualisation des répartition des taxons sur chaque fiche de territoires… tout ça est vraiment génial !! :-)

Je commence tout juste avec cette version de l'atlas, et à chaud, j'ai 2 petites questions :

  • le block "A voir en ce moment" reste vide : je ne récupère rien dans mostViewTaxon. Tu as remarqué ce problème aussi ?
  • est-ce tu aurais un exemple pour le template "présentation" ? C'était vraiment confortable d'avoir un exemple prémâché jusque là. Il n'y avait plus qu'à modifier les images du carousel, changer les titres du menu et insérer son propre contenu. S'il y a moyen d'avoir un exemple, je suis preneuse !

Merci

@camillemonchicourt
Copy link
Member

Ouais on espère pouvoir bosser sur cette PR (#233) un de ces 4, mais il y a beaucoup de choses mélangés dedans, pas que le passage à BS4 :-)
Concernant la page de présentation, en effet, on l'a nettoyé mais c'est à revoir, comme tu dis c'est pratique d'avoir une base à adapter. On va la remettre, c'est noté dans les commentaires de la PR : #233 (review)

@Splendens
Copy link
Contributor

Oui, c'est justement toutes les fonctionnalités (en dehors de BS4) qui m'intéressent !

Est-ce ça veut dire que la PR #233 (review) va être amenée à avoir encore d'autres commit ?
En fait, j'aimerais bien me baser sur cette version de l'atlas (avec les fonctionnalités ajoutées par Fred) pour refaire mes PR. Mais ça m'a l'air un peu compliqué de me baser sur une branche de qqun d'autre, surtout si elle doit encore bouger !

ps : Pour les mostViewTaxon : c'est bon j'ai trouvé le soucis, ça vient de mes données.

@camillemonchicourt
Copy link
Member

Oui de ce qu.on avait vu en regardant rapidement il y avait quelques éléments à reprendre, quelques trucs cassés aussi.

@Adrien-Pajot Adrien-Pajot added this to Backlog in V2 via automation Jun 25, 2021
@Adrien-Pajot
Copy link
Contributor

Dans le cadre de COBIODIV SI on va faire en sorte que les PR #232 et #233 soient étudiées et intégrées parce que beaucoup de fonctionnalités et une migration importante sont effectuées. Même si cela nécessite un gros travail de relecture avec le nombre de commits.

@corentinlange corentinlange moved this from Backlog to Done in V2 Jul 8, 2021
@camillemonchicourt camillemonchicourt moved this from Done to To validate in V2 Jul 8, 2021
@camillemonchicourt
Copy link
Member

Intégré dans la 1.5.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To validate
V2
To validate
Development

No branches or pull requests

4 participants