fix: skip filler rows in data-table when pagination is disabled

[Copilot]

Grouped list view renders excessive empty rows because each group's data-table inherits pageSize: 10 while setting pagination: false. The filler row logic pads every group to 10 rows regardless.

Change

In data-table.tsx, gate filler row rendering on pagination being enabled:

- {paginatedData.length > 0 && Array.from({ length: Math.max(0, pageSize - paginatedData.length) }).map((_, i) => (
+ {pagination && paginatedData.length > 0 && Array.from({ length: Math.max(0, pageSize - paginatedData.length) }).map((_, i) => (

Filler rows exist to maintain consistent page height during pagination — they have no purpose when pagination is off (grouped views, embedded tables).

Tests

• Added test: no filler rows when pagination: false with pageSize: 10
• Added test: filler rows still render when pagination: true

Original prompt

列表视图分组模式有很多空行

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
objectui	Ready	Preview, Comment	Feb 26, 2026 9:50am
objectui-demo	Ready	Preview, Comment	Feb 26, 2026 9:50am
objectui-storybook	Ready	Preview, Comment	Feb 26, 2026 9:50am

[Copilot]

Pull request overview

Fixes excessive empty rows in grouped list views by preventing DataTableRenderer from rendering filler rows when pagination is disabled (even if pageSize is set).

Changes:

• Gate filler row rendering on pagination being enabled in DataTableRenderer.
• Add tests covering filler-row behavior for pagination: false vs pagination: true.
• Document the root cause, fix, and test coverage update in ROADMAP.md.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
packages/components/src/renderers/complex/data-table.tsx	Skip rendering filler rows unless pagination is enabled.
packages/components/src/renderers/complex/tests/data-table-airtable-ux.test.tsx	Adds assertions for filler row presence/absence depending on pagination.
ROADMAP.md	Records the bug’s root cause and the implemented fix + tests.

Comments suppressed due to low confidence (1)

packages/components/src/renderers/complex/data-table.tsx:927

• Filler row colSpan doesn’t include the optional row-numbers column. When showRowNumbers is true, data rows include an extra cell but filler rows span one fewer column, which can misalign the table layout. Consider matching the other colSpan calculations in this component by adding (showRowNumbers ? 1 : 0) here as well (and any other optional leading columns).

                {pagination && paginatedData.length > 0 && Array.from({ length: Math.max(0, pageSize - paginatedData.length) }).map((_, i) => (
                  <TableRow key={`empty-${i}`} className="hover:bg-transparent">
                    <TableCell colSpan={columns.length + (selectable ? 1 : 0) + (rowActions ? 1 : 0)} className="h-[52px] p-0" />
                  </TableRow>