Skip to content
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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

feat(Table): rows grouping #1003

wants to merge 5 commits into from

Conversation

savutsang
Copy link
Contributor

@savutsang savutsang commented Oct 11, 2024

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.

@savutsang savutsang requested a review from a team as a code owner October 11, 2024 13:59
@savutsang savutsang force-pushed the dev/DS-915 branch 3 times, most recently from 613185c to 12e6764 Compare October 11, 2024 14:25
Copy link

Storybook for this build: https://ds.equisoft.io/pr-1003/

Copy link

Webapp for this build: https://ds.equisoft.io/pr-1003/webapp/

@savutsang savutsang force-pushed the dev/DS-915 branch 2 times, most recently from 090a905 to 5c2120d Compare October 11, 2024 14:48
Copy link
Contributor Author

@savutsang savutsang Oct 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai des erreurs de Typescript que j'arrive pas a resoudre, plus j'essaie de fixer ca et plus je recule. Le probleme est essentiellement l'incompatibilite des types generics, l'erreur est visbile dans data-table.stories.tsx. J'aurais besoin de cerveau frais qui pourrais m'aider a resooudre ca.

@@ -136,7 +136,7 @@ export const WithFooter: Story = () => {

// Footer function for the 'numbers' column
const footerSum = (): ReactElement => {
const total = data.reduce((sum, row) => row.numbers + sum, 0);
const total = (data as FooterData[]).reduce((sum, row) => row.numbers + sum, 0);
Copy link
Contributor Author

@savutsang savutsang Oct 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'aimerais mieux qu'on ai pas a caster ici, ni les autres casting plus bas non plus

Copy link

@maboilard maboilard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Étrangement, les rows de section sont en état selected lorsque je clique sur le checkbox du column header. Les rows de section devraient cependant être indépendantes.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants