incubateur-ademe / nosgestesclimat-site-nextjs

https://nosgestesclimat.vercel.app

MIT License

2 stars 6 forks source link

:sparkles: Add loading skeletons #693

Closed bjlaa closed 2 months ago

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
nosgestesclimat	✅ Ready (Inspect)	Visit Preview	Sep 12, 2024 7:57am

notion-workspace[bot] commented 2 months ago

[Layout/UX/SEO] Ajoute des skeleton partout sur le site lors des chargements

github-actions[bot] commented 2 months ago

Report for the pull request #693

--- ### :globe_with_meridians: Translation status #### UI's texts | Language | Nb. missing translations | Status | |:--------:|:-------------------------|:------:| | _en_ | Ø | :heavy_check_mark: | | _es_ | Ø | :heavy_check_mark: | #### FAQ's questions | Language | Nb. missing translations | Status | |:--------:|:-------------------------|:------:| | _en_ | Ø | :heavy_check_mark: | | _es_ | Ø | :heavy_check_mark: | > _You will find more information about the translation in the dedicated file._

bjlaa commented 2 months ago

@florianpanchout Je modifie le mode d'affichage pour éviter d'afficher un loader le temps que le modèle se charge, curieux d'avoir ton avis sur ces modifs et les effets de bords potentiels qui pourraient en découler (ceux que je n'aurais pas vus)

bjlaa commented 2 months ago

Encore une carte qui montre qu'il faut que l'on fasse plus de "kickoff" dev. (je ne sous entends pas que ma solution est meilleure, mais ça aurait probablement été constructif d'en débattre et de s'aligner avant) (je sais que tu as commencé pendant que j'étais en vacance, c'est juste à garder en tête à l'avenir)

Oui pourquoi pas ! Il faudrait sans doute faire ça pendant le sprint planning