search

leonarf / VCA4D

Code source du site web crée par le Basic pour mettre en valeur les résultats des études de VCA4D
1 stars 2 forks source link

refactor: Generify the rows of the comparison to create an expandable row #85

Closed maximepiard closed 4 weeks ago

maximepiard commented 4 weeks ago

Ticket: #67

Contexte

On veut pouvoir afficher et cacher des sous-row dans la page de comparaison Cette première PR genérifie les rows de comparaison pour les utiliser par les 3 sections (eco/social/environnement)

Les 2 derniers commits crééent un composant ComparisonExpandableRow qu'on utilise (pour demo) dans la branche main-67-test

Screencast from 2024-08-20 18-03-26.webm

Commits

maximepiard commented 4 weeks ago

@leonarf Il me reste du styling a faire pour les rows "expandables" dans la page de comparaison, mais le layout des composant est déjà là.

Si tu as le temps d'y jeter un oeil avant de partir ce soir ce serait top, sinon je continue mon bonhomme de chemin :innocent: Normalement la lecture commit par commit se passe bien, j'ai fait attention a ne pas aller trop vite dans les refactors

Tu peux tester sur main-67-test. Voila ce que ça donne dans les composants qui définissent les indicateurs a montrer, ça me parait assez élégant.

:eyes: EDIT: Tu peux meme tester meme sur main-67-style avec le style final et des fausses données que j'ai mises en random Je n'ai pas reussi a faire de transition du dropdown, comme prévu, mais le style est plutot cool je trouve. Il faudra juste faire une passe par Gaspard je pense

Screencast from 2024-08-20 19-42-00.webm