Refacto feuille de paie

Pour discussions et suites : un début de refactorisation du WaterfallView.

requested review from @Dorine

assigned to @dsmadja

added 1 commit

• 1b75aed1 - Ajoute fond heropattern sur le composant LinkedVariables

Compare with previous version

added 1 commit

• 32edc182 - Aligne flèche ouvrir/fermer variables enfants

Compare with previous version

@dsmadja je me demande si la place de ce composant est vraiment dans le dossier variables.

Dans ce dossier, il y a uniquement des composants liés à la partie gauche de l'interface, dédiée au code en vigueur.

Je me demande si nous ne devrions pas créer un dossier "Waterfall" (Ou décomposition d'ailleurs), dans lequel on met ce composant LinkedVariables, le nouveau composant "StandardOfLiving et les autres composants du Waterfall.

changed this line in version 6 of the diff

ici, je vois que tu as déplacé les fonctions dans un fichier dédié. Partante d'en discuter lundi prochain pour comprendre l'intention :)

Salut @dsmadja j'ai testé, je n'ai pas rencontré de problème, mis à part le fond qui ne s'affichait pas sur le composant LinkedVariables (j'ai corrigé dans un commit).

Je me demande si on ne peut pas créer un dossier pour les composants du Waterfall (cf commentaire plus haut) mais rien de bloquant.

approved this merge request

added 1 commit

• 278b9c2d - Fix taille montant niveau de vie

Compare with previous version

added 1 commit

• 98b2b12f - Change style des montants y compris pour les intermédiaires

Compare with previous version

added 52 commits

• 98b2b12f...f474077a - 44 commits from branch master
• 609df5e5 - Move Linked variables code into its own component
• 17795a9f - Move StandardOfLiving into its own component
• ebdeec27 - Simplify getCalculatedVariableNumberValue
• 02612f12 - Ajoute fond heropattern sur le composant LinkedVariables
• 49bda24e - Aligne flèche ouvrir/fermer variables enfants
• 320c5aac - Fix taille montant niveau de vie
• acdf7bcc - Change style des montants y compris pour les intermédiaires
• 5e20f206 - Rebase

Compare with previous version

marked this merge request as ready

merged

mentioned in commit f137cd63