249 improve mesoigner - Githubissues

CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose

https://vitemadose.covidtracker.fr

Other

80 stars 51 forks source link

249 improve mesoigner #253

Closed nhumblot closed 3 years ago

nhumblot commented 3 years ago

Cette Pull Request est

[x] Un correctif
[ ] Une nouvelle fonctionnalité

Checklist

Si cette PR résoud une issue, j'indique son numéro dans les description
Cette PR vise la branche dev
Elle n'est pas en conflit avec la branche dev

Description

Change le logo de MeSoigner en SVG. Refactorise la gestion des logos pour un emplacement indépendant du format.

Question soulevée, faut-il attendre avant de supprimer l'ancien emplacement des logos qui ont déjà été exposés pour se prémunir contre des problèmes de cache et/ou de vieux liens ?

Avant:

Après:

fcamblor commented 3 years ago

@nhumblot bonne remarque vis à vis du cache, en effet je conserverais les vieux assets, au moins jusqu'à la prochaine MEP :)

nhumblot commented 3 years ago

@nhumblot bonne remarque vis à vis du cache, en effet je conserverais les vieux assets, au moins jusqu'à la prochaine MEP :)

Ma PR ne les supprime pas donc c'est bon. 🙂

Je pourrai ouvrir une PR côté serveur pour mettre en place une redirection au niveau de Caddy.

nhumblot commented 3 years ago

Tu voies toujours un souci sur ce changement @fcamblor ? N'hésite pas si c'est le cas, je pense avoir traité les demandes. 🙂

fcamblor commented 3 years ago

@nhumblot c'est bien mieux

Juste une question ... il n'y avait pas un mic mac où les gens de mesoigner voulaient finalement conserver le logo avec le double origami vert/gris ? (moi je trouve que ta version sans cet origami rend le logo plus lisible, mais il faudrait ptet checker pour en être sûr)

nhumblot commented 3 years ago

La version actuelle convient semble-t-il.

https://mattermost.covidtracker.fr/covidtracker/pl/w6853airw7nzfch48nscguexsa

Je ferme cette PR.