Skip to content

feat: add a custom 'curve-data-table' component#89

Open
loofpc wants to merge 16 commits into
mainfrom
feat/custom-data-table-wrapper
Open

feat: add a custom 'curve-data-table' component#89
loofpc wants to merge 16 commits into
mainfrom
feat/custom-data-table-wrapper

Conversation

@loofpc

@loofpc loofpc commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Adds a curve-data-table component that wraps the data-table component.

Features in this PR:

  • The curve-data-table component itself, with various components and models for cell rendering etc. neatly split into separate files.
  • Mostly same functionality and similar look to the regular data-table. Configurablility is limited, for example no column with image ánd a text, no CSS passable for making text upper or lower case. Additionally, this component allows showing or hiding the search input and column filter dropdown, and the search input can be put on different columns. Current functionality is just how far we got in the POC project, extending it with more options and flexibility would of course be possible.
  • a /table page that shows the same table content as the /demo page but using the new wrapper component.
  • a CurveDataTable Storybook page that shows the same table as the DataTable page. This has just one interactive story for now. Since this component differs in options from the regular DataTable, it might not be feasible or desireable to create the same stories.

@loofpc loofpc changed the title feat: add a custom 'curve-data-table' component that wraps the comple… feat: add a custom 'curve-data-table' component Jul 7, 2026

@sjoerdbeentjes sjoerdbeentjes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Paar kleine comments, maar ziet er goed uit!

Comment thread packages/angular/src/lib/ui/curve-data-table/src/lib/curve-data-table.ts Outdated
Comment thread packages/angular/src/lib/ui/curve-data-table/src/lib/curve-data-table.ts Outdated
Comment thread packages/angular/src/lib/ui/curve-data-table/src/lib/curve-data-table.ts Outdated
Comment thread packages/contracts/src/curve-data-table.ts Outdated
Pieter Loof and others added 13 commits July 13, 2026 12:06
The link variant now uses the dedicated --link design token instead of --primary in both the React and Angular buttons, exposing a matching text-link utility via a --color-link theme mapping so link text picks up the theme- and mode-specific colour.
rm -rf/cp -RL require a POSIX shell and fail on Windows. Replace them
with a Node script using fs.rmSync/fs.cpSync so the font-file copy
step works the same on every OS.
Internal build tooling change, no consumer-facing effect on the
published package output.
Neither published package (curve-react, curve-angular per package.json
publishConfig) had a README, so npm/GitHub package pages had nothing
to show. Covers install, peer deps, usage, theming, and a Storybook
link for the full component set.
…e-wrapper

# Conflicts:
#	apps/angular-app/CHANGELOG.md
#	apps/angular-app/package.json
#	apps/react-app/CHANGELOG.md
#	apps/react-app/package.json
#	packages/angular/CHANGELOG.md
#	packages/angular/package.json
#	packages/contracts/CHANGELOG.md
#	packages/contracts/package.json
#	packages/react/CHANGELOG.md
#	packages/react/package.json
#	packages/storybook-config/CHANGELOG.md
#	packages/storybook-config/package.json
#	packages/tokens/CHANGELOG.md
#	packages/tokens/package.json
#	packages/typescript-config/CHANGELOG.md
#	packages/typescript-config/package.json
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