Resolve "Appliquer la nouvelle feuille de paie au mode comparaison"
Closes #364 (closed)
Merge request reports
Activity
- changed milestone to %
changed milestone to %
added En cours label
assigned to @Dorine
added 1 commit
- 255c3270 - Fix les bordures qui permettent de suivre les variables enfants
added 1 commit
- 9ca5e6f7 - Fix le layout qui accueille le waterfall comparé, et harmonise avec le code...
added 25 commits
-
2a80b5e6...f2a19a01 - 19 commits from branch
master - 420d5f8b - Copie ancien composant pour les avoir en vue le temps de la refacto
- 909175eb - Modifie la div générale
- feab1d78 - Applique la structure de la nouvelle PaySlip au mode comparé, en gardant le...
- 979d6bba - Fix les bordures qui permettent de suivre les variables enfants
- b7d7ca65 - Fix le layout qui accueille le waterfall comparé, et harmonise avec le code...
- ab89bf2c - ajout margin bouton affichage des variables
Toggle commit list-
2a80b5e6...f2a19a01 - 19 commits from branch
added 1 commit
- 70dfaca8 - Centre le squelette du loader au milieu de la payslip
added 1 commit
- 3a27613f - Ajoute baisse de l'opacité sur les montants doubles
added 1 commit
- 369e74fe - Essai (sans succès) d'ajouter les variables liées dans la feuille de paie comparée
@dsmadja dans le cas où tu aurais le temps de te pencher sur ce sujet les deux prochaines semaines :
Dans cette MR, j'ai réussi à restructurer la feuille de paie mode comparaison, qui a maintenant le même design que la version 1 cas type. (Normalement c'est bon, mais une revue est bienvenue)
Par contre, dans mon dernier commit, j'ai essayé d'ajouter les variables liées, mais sans succès, pour l'instant la variable liée est toujours à 0. Et en plus elle n'est visible que pour le cas type de gauche, or je souhaite que le panneau soit présent sur les deux cas types, à gauche et à droite.
Je pense que ce que j'ai engagé dans ce dernier commit n'est pas terrible, j'ai recréé un composant variable liée, je ne suis pas sûre que c'était nécessaire, nous pourrions probablement utiliser le même que celui pour la feuille de paie simple.
Donc n'hésite pas à le revert !
J'ai ajouté un nouveau composant LinkedVariablesCompareView.
Ça m'a l'air de marcher comme on voudrait mais je te laisse jeter un oeil (surtout concernant les CSS et l'aspect responsive pour lequel je ne suis pass sûr du rendu que tu vises...) On pourra discuter de vive voix de ce qu'il reste à faire avant de merger.
added 31 commits
-
3d14821c...b2a5c0a1 - 21 commits from branch
master - cba0bc41 - Copie ancien composant pour les avoir en vue le temps de la refacto
- 30960972 - Modifie la div générale
- 0ea9f794 - Applique la structure de la nouvelle PaySlip au mode comparé, en gardant le...
- 164c1034 - Fix les bordures qui permettent de suivre les variables enfants
- e4ff84f9 - Fix le layout qui accueille le waterfall comparé, et harmonise avec le code...
- 4ca47a71 - ajout margin bouton affichage des variables
- 448796e9 - Centre le squelette du loader au milieu de la payslip
- 20ef490b - Ajoute baisse de l'opacité sur les montants doubles
- 2c332eb2 - Essai (sans succès) d'ajouter les variables liées dans la feuille de paie comparée
- e2191b5b - rework linkedvariables for compare view
Toggle commit list-
3d14821c...b2a5c0a1 - 21 commits from branch
added 1 commit
- 0f050f55 - Déplace hors du each le composant LinkedVariableCompareView afin qu'il ne...
added 1 commit
- 231b0a85 - Supprime ancien composant PaySlipCompareViewOld
@dsmadja ça m'a l'air bon après corrections précédentes. Je vais donc merger.
Il y a une évolution que nous pourrions essayer de faire dans une nouvelle MR : ajouter les unités à côté des valeurs.
Edited by Dorine Lambinetmentioned in commit 37a662eb
