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

fix(homepage): wording and content styles #640

Merged
merged 6 commits into from
Jan 15, 2025
Merged

Conversation

narduin
Copy link
Contributor

@narduin narduin commented Dec 18, 2024

Manque les virgules en fin de ligne dans la liste de FAQ

J'ai remplacé par des points virgules pour être vraiment conforme 😁

Harmoniser le lien explicite à ecologie.gouv.fr dans FAQ

Je n'ai pas compris… @streino c'était ton retour ?

fix ecolabdata/ecospheres#484
fix ecolabdata/ecospheres#517

@narduin narduin requested review from abulte and streino December 18, 2024 14:17
Copy link

netlify bot commented Dec 18, 2024

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit edd53bb
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/678689055b54b70008d1a1df
😎 Deploy Preview https://deploy-preview-640--ecospheres.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 18, 2024

Deploy Preview for meteo-france ready!

Name Link
🔨 Latest commit edd53bb
🔍 Latest deploy log https://app.netlify.com/sites/meteo-france/deploys/67868905ad03c20008d5ea6d
😎 Deploy Preview https://deploy-preview-640--meteo-france.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@streino
Copy link
Contributor

streino commented Dec 18, 2024

Harmoniser le lien explicite à ecologie.gouv.fr dans FAQ

Je n'ai pas compris… @streino c'était ton retour ?

C'était tous mes retours ;)

Le ministère est mentionné de manière variable et potentiellement incohérente à différents endroits de la FAQ (et du About) :

Il me semblait avoir trouvé encore une autre formulation dans la FAQ mais je ne retrouve plus.

Edit: Trouvé ! C'est "ministère en charge des territoires, de l’écologie et du logement" dans la carte "Explorer" de la home.

@narduin
Copy link
Contributor Author

narduin commented Dec 18, 2024

Le ministère est mentionné de manière variable et potentiellement incohérente à différents endroits de la FAQ (et du About)

Alors qu'il s'appelle "Ministère de la Transition écologique, de l’Énergie, du Climat et de la Prévention des risques" maintenant ? 😅

@streino
Copy link
Contributor

streino commented Dec 18, 2024

Le ministère est mentionné de manière variable et potentiellement incohérente à différents endroits de la FAQ (et du About)

Alors qu'il s'appelle "Ministère de la Transition écologique, de l’Énergie, du Climat et de la Prévention des risques" maintenant ? 😅

C'est chaud de se tenir à jour, et ça va peut-être encore bientôt changer. 🙈
À cela s'ajoute la distinction entre le ministère et le pôle ministériel. Par exemple, on parle de logement sur la home, qui est encore (il me semble) dans le pôle ministériel, mais pas dans le ministère (bien qu'on ait la DGALN 🤷), ce qui orienterait vers l'utilisation de "pôle ministériel". Mais à l'inverse, je sais pas si le CGDD a encore des multi-tutelles ou si on est plus que sur le périmètre "ministère".
@Thesauruv on utilise quoi ?

Copy link
Contributor

@streino streino left a comment

Choose a reason for hiding this comment

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

Reste #640 (comment) sinon LGTM

src/custom/ecospheres/views/HomeView.vue Outdated Show resolved Hide resolved
src/custom/ecospheres/views/HomeView.vue Show resolved Hide resolved
@narduin
Copy link
Contributor Author

narduin commented Dec 19, 2024

Je vais modifier ça.

Edit: Trouvé ! C'est "ministère en charge des territoires, de l’écologie et du logement" dans la carte "Explorer" de la home.

Si j'ai bien suivi ça devrait être "des ministères" (c.f. ecologie.gouv.fr) non ?

@DanFluture275
Copy link

Pour le nom du ministère, mettre "ministère en charge de l'environnement"
On peut garder "pôle ministériel ecologie.gouv.fr (faq)"

@narduin narduin requested a review from abulte December 23, 2024 13:31
@DanFluture275
Copy link

@narduin , il manque un point entre data.gouv.fr. et "Si vous ne trouvez pas..." dans la FAQ.
image

Et si on veut être cohérent partout pour le formatage, faudrait aussi mettre data.gouv.fr dans le bloc contribuer.

image

@narduin
Copy link
Contributor Author

narduin commented Jan 14, 2025

Et si on veut être cohérent partout pour le formatage, faudrait aussi mettre data.gouv.fr dans le bloc contribuer.

Impossible de formatter ce contenu, il est fourni en tant que texte à un composant datagouv.

@narduin narduin requested a review from DanFluture275 January 14, 2025 15:56
Copy link

@DanFluture275 DanFluture275 left a comment

Choose a reason for hiding this comment

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

testé PO OK

@narduin narduin merged commit d49f684 into main Jan 15, 2025
8 checks passed
@narduin narduin deleted the fix/homepage-content-styles branch January 15, 2025 15:20
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.

Paragraphes page About Small fixes Homepage
4 participants