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

Maj du theme itou vers la v0.7.1 et du footer #312

Merged
merged 3 commits into from
Jun 19, 2023
Merged

Conversation

hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented Jun 14, 2023

Carte Notion : https://www.notion.so/plateforme-inclusion/Am-liorer-le-footer-de-tous-les-chantiers-ba2dce09f9c545a2827d2a7707d8bd58?pvs=4

Quoi

Mise à jour du thème vers la v0.7.1
Mise a jour du Footer

Pourquoi

Pour modifier :

  • le html et le css du .s-footer
  • la largeur max du container (maintemant 1200px)

@hellodeloo hellodeloo self-assigned this Jun 14, 2023
@hellodeloo
Copy link
Contributor Author

hellodeloo commented Jun 14, 2023

@vincentporte Les liens et libellés sont ceux qui ont été validés avec Antoine mais si certains ne sont pas bons ou pas ceux souhaités, n'hésite pas à corriger/modifier 😘

@hellodeloo hellodeloo changed the title Mise a jour du thème et du footer Maj du theme itou vers la v0.7.1 et du footer Jun 14, 2023
@vincentporte
Copy link
Contributor

merci @hellodeloo , je verifie les liens.

concernant ces contenus statiques, j'ai constaté un défaut lors du déploiement du header. L'image n'était pas redimensionnée. Avec un CTRL+F5, le rendu était ok. Les utilisateurs qui ne connaissent pas l'astuce ont gardé un énorme logo très longtemps …
Meme sujet pour le footer, CTRL+F5 …

Comment pourrions nous éviter cet effet de cache ? voir forcer le refresh total pour tout le monde au premier passage après le déploiement ?

@vincentporte
Copy link
Contributor

@hellodeloo, peux tu modifier le footer_flatpages.html de la même manière que footer.html ?

@hellodeloo
Copy link
Contributor Author

concernant ces contenus statiques, j'ai constaté un défaut lors du déploiement du header. L'image n'était pas redimensionnée. Avec un CTRL+F5, le rendu était ok. Les utilisateurs qui ne connaissent pas l'astuce ont gardé un énorme logo très longtemps …

En général, selon les frameworks et langages, il y a une astuce/fonction pour forcer le refresh des assets en ajoutant un hash dynamique à l'url des assets (ex: /static/vendor/theme-inclusion/images/logo-emploi-inclusion.84487a726d32.svg) . Je vois que c'est le cas sur le C1 et le C4 mais je ne sais pas comment ça fonctionne précisément sur Django

@vincentporte
Copy link
Contributor

concernant ces contenus statiques, j'ai constaté un défaut lors du déploiement du header. L'image n'était pas redimensionnée. Avec un CTRL+F5, le rendu était ok. Les utilisateurs qui ne connaissent pas l'astuce ont gardé un énorme logo très longtemps …

En général, selon les frameworks et langages, il y a une astuce/fonction pour forcer le refresh des assets en ajoutant un hash dynamique à l'url des assets (ex: /static/vendor/theme-inclusion/images/logo-emploi-inclusion.84487a726d32.svg) . Je vois que c'est le cas sur le C1 et le C4 mais je ne sais pas comment ça fonctionne précisément sur Django

@hellodeloo , je viens d'ajouter la gestion des versions de fichiers statiques dans la PR #315, pour gérer ce cas. Test en cours en recette

@github-actions
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Copy link
Contributor

@vincentporte vincentporte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 LGTM 🎨

@vincentporte vincentporte removed the recette-jetable review_app label Jun 19, 2023
@hellodeloo hellodeloo merged commit 61ef541 into master Jun 19, 2023
@hellodeloo hellodeloo deleted the deloo/new-footer branch June 19, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants