-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(Table): rows grouping #1003
Conversation
613185c
to
12e6764
Compare
Storybook for this build: https://ds.equisoft.io/pr-1003/ |
Webapp for this build: https://ds.equisoft.io/pr-1003/webapp/ |
090a905
to
5c2120d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques petits problèmes avec l'état des checkboxes.
- Si le parent est checked, ça sélectionne les enfants. Si tu un-check un enfant, le parent a les deux icons "-" & "✓" des states.
- Si tu un-check tous les enfants, le parent garde l'icon/état checked "✓"
- Si tu select tous les enfants, le parent n'a l'icon/état checked "✓"
DS-915_checkboxes-states.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je vais te laisser adresser les autres reviews mais LGTM!! 👍
C'est corrige. Je l'ai refact, il y avait plein de bugs avec l'ancienne facon de faire. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Attends l'approval de Max ou PY mais sinon la vie est belle pour moi.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans Figma, le children a une indentation sur la colonne de checkboxes et la première colonne de texte. Peut-être que c'est hors scope et qu'on peut arranger ça dans un autre ticket, mais je voulais quand même le mentionner.
Si c'est quelque chose qui peut être fixé rapidement, faisons là dans cette PR mais sinon, on le fera dans une autre PR étant donné que Oli en a besoin pour ce sur quoi il travaille présentement. |
Notre maquette pour le tableau de doublons ne comporte pas cette indentation, donc la situation présente me va parfaitement. |
C'est un oublis de ma part mais c'est pas si simple de faire ce modificaton. Presentement le checkbox est dans une colonne a part, si on veut faire l'indentation, il faudrait que le checkbox et le texte soit dans la meme colonne. Bref un autre PR. |
On a un souci de temps pour l'implémentation côté CRM. Je peux juste parler pour ma team, mais ça me va eeeeeen masse comme c'est là :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On pourra régler l'indentation dans un autre ticket. Beau boulot!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, on pourra itérer à partir de ça au besoin!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai migre au nouveau format storybook pour qu'on puisse jouer avec les controls, mais en realite ya juste le story Grouping qui a ete ajoute.
* feat(Table): rows grouping * fix: minor * fix: remove temporary patch * fix: remove unecessary fields * fix: remove temporary patch for build * fix: new version of grouping * fix: comment test * fix: proper checking * fix: selection state * fix: test
Permettre d'afficher des rows sous des groups
Note: Les screenshots dans JIRA ont des background gris, mais c'est plutot blanche/transparent tel que dans Figma.