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

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

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

nkokla
Copy link
Contributor

@nkokla nkokla commented Jan 30, 2023

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 nkokla force-pushed the feat/quick-fix-home-page-and-navigation branch 7 times, most recently from a6c7893 to b6e2f40 Compare February 1, 2023 22:07
@jdesboeufs
Copy link
Member

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.

@nkokla
Copy link
Contributor Author

nkokla commented Feb 2, 2023

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' 😝)

@jdesboeufs
Copy link
Member

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.

@nkokla nkokla changed the title Feat/quick fix home page and navigation Preview Quick-Fix/Refonte du site de l'adresse Feb 4, 2023
@vinsag vinsag requested review from vinsag, antoineludeau and tmerlier and removed request for vinsag February 7, 2023 12:51
@nkokla nkokla force-pushed the feat/quick-fix-home-page-and-navigation branch from b6e2f40 to 72ad4db Compare February 7, 2023 14:52
@nkokla nkokla force-pushed the feat/quick-fix-home-page-and-navigation branch from 72ad4db to ce7a3fe Compare February 7, 2023 14:53
@tmerlier tmerlier removed their request for review November 6, 2023 08:18
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.

3 participants