-
Notifications
You must be signed in to change notification settings - Fork 0
feat(webapp): adds users data table #935
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
Conversation
Storybook for this build: https://ds.equisoft.io/pr-935/ |
Webapp for this build: https://ds.equisoft.io/pr-935/webapp/ |
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! J'ai identifié quelque chose qui serait bien à corriger. En fait, la ligne de Column header (32px) n'a pas la même hauteur que les autres lignes du tableaux (40px) ce qui look un peu weird à mon avis et qui n'est pas conforme avec les maquettes Figma. @LarryMatte Il faudrait créer une tâche afin de faire l'ajustement.
ça a été fixé dans une PR: #954 |
DS-1136
Description
Ici on vient ajouter la data table d'utilisateurs
*Problème détecté: le focus sur les headers des columns de la table ne sont pas visibles. Ils respectent l'ordre désiré dans le ticket, mais l'élément visuel n'apparait pas
Tests fonctionnels