adresse.data.gouv.fr icon indicating copy to clipboard operation
adresse.data.gouv.fr copied to clipboard

Preview Quick-Fix/Refonte du site de l'adresse

Open nkokla opened this issue 3 years ago • 3 comments

Cette PR permet d'avoir un aperçu de la refonte sur un environnement en ligne. Elle sera probablement redécouvre par la suite, en de multiple PR, selon les responsabilités technique.

nkokla avatar Jan 30 '23 15:01 nkokla

Pourquoi importer les icônes et les styles ? 10 000 lignes ajoutées pour un quick-fix c'est énorme. Faudrait revoir le titre de la PR aussi, et la description, pour les 144 personnes qui ont le dépôt dans leurs stars ou le watchent. Surtout pour une PR de cette ampleur.

jdesboeufs avatar Feb 02 '23 10:02 jdesboeufs

En effet, je vais mettre à jour le nom et la description. 👍 Cette PR permet d'avoir un aperçu sur un environnement en ligne. Elle sera probablement redécoupe par la suite, en de multiple PR, selon les responsabilités. Pour les styles, l'objectif est de faire une migration progressive du style template-data-gouv vers celui du DSFR. Il a fallu pour ça encapsuler l'ensemble des déclarations CSS sous une classe 'parent'. Au besoin, il est surement possible de scripter cette étape, mais je ne sais pas encore si l'investissement à un intérêt. L'idée étant de supprimer ce style lorsque la migration aura été effectuée. (C'est, entre autres, pour ça que cette PR est en 'Draft' 😝)

nkokla avatar Feb 02 '23 15:02 nkokla

C'est plus clair comme ça ;) Il faut peut-être jeter un oeil à ce que la taille des bundles n'explose pas, sinon ça peut faire chuter les performances Webperf et donc le ranking SEO.

jdesboeufs avatar Feb 02 '23 22:02 jdesboeufs

Cette PR est devenu obsolote avec la refonte du site avec le DSFR.

vinsag avatar Nov 26 '24 11:11 vinsag