@jielga/tmdatagrid 2.1.0 → 2.1.1

package/docs/column-layout.md

@@ -160,7 +160,7 @@ const { resetSettings } = useTMDataGrid({ data, columns });
 | Name | Kind | Type | Default | What it does |
 | --- | --- | --- | --- | --- |
 | `enableHiding` | Table option | `boolean` | `true` | Also a column option. `false` removes hiding entirely. |
-| `enableColumnPinning` | Table option | `boolean` | `true` | `false` removes the pin menu items. |
+| `enableColumnPinning` | Table option | `boolean` | `true` | `false` removes the pin menu items. The generated lanes stay pinned. |
 | `enablePinning` | Column option | `boolean` | `true` | `false` for one column. |
 | `enableColumnOrdering` | Option | `boolean` | `true` | Header dragging and the move menu items. Grid-defined. |
 | `meta.enableOrdering` | Column meta | `boolean` | `true` | `false` keeps one column where it is. |

package/docs/use-tm-data-grid.md

@@ -36,7 +36,7 @@ rather than forwarded to TanStack.
 | `enableSorting` | `boolean` | `true` | Enables sorting for the table. |
 | `enableColumnFilters` | `boolean` | `true` | Enables filtering for the table. |
 | `enableHiding` | `boolean` | `true` | Enables column visibility for the table. |
-| `enableColumnPinning` | `boolean` | `true` | Enables pinning for the table. |
+| `enableColumnPinning` | `boolean` | `true` | Enables pinning for the table. `false` removes the pin menu items; the generated lanes stay pinned. |
 | `enableColumnResizing` | `boolean` | `true` | Enables resizing for the table. |
 | `enableColumnOrdering` | `boolean` | `true` | Enables header dragging and the move menu items. Defined by the grid, see [Column layout](/docs/column-layout#ordering). |
 | `enablePagination` | `boolean` | `false` | Enables client-side paging and the `Footer` pager. Implied by `manualPagination`. Defined by the grid, see [Pagination](/docs/pagination). |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jielga/tmdatagrid",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "A React data grid built on TanStack Table v9 and Mantine - always virtualized, with resizable, reorderable, sortable, filterable, hideable and pinnable columns.",
   "license": "MIT",
   "type": "module",
@@ -76,31 +76,31 @@
   },
   "devDependencies": {
     "@babel/core": "^7.29.7",
-    "@mantine/core": "^9.4.0",
-    "@rolldown/plugin-babel": "^0.2.3",
+    "@mantine/core": "^9.4.2",
+    "@rolldown/plugin-babel": "^0.2.4",
     "@tabler/icons-react": "^3.46.0",
     "@tanstack/intent": "0.3.6",
-    "@tanstack/react-form": "^1.33.2",
-    "@tanstack/react-store": "^0.11.1",
+    "@tanstack/react-form": "^1.33.5",
+    "@tanstack/react-store": "^0.11.2",
     "@tanstack/react-table": "^9.2.4",
-    "@tanstack/react-virtual": "^3.14.4",
-    "@tanstack/store": "^0.11.1",
+    "@tanstack/react-virtual": "^3.14.14",
+    "@tanstack/store": "^0.11.2",
     "@tanstack/table-core": "^9.2.4",
     "@testing-library/jest-dom": "^7.0.0",
     "@testing-library/react": "^16.3.2",
-    "@testing-library/user-event": "^14.6.1",
+    "@testing-library/user-event": "^14.6.7",
     "@types/babel__core": "^7.20.5",
-    "@types/react": "^19.2.17",
-    "@types/react-dom": "^19.2.3",
-    "@vitejs/plugin-react": "^6.0.4",
+    "@types/react": "^19.2.18",
+    "@types/react-dom": "^19.2.7",
+    "@vitejs/plugin-react": "^6.0.5",
     "babel-plugin-react-compiler": "^1.0.0",
     "jsdom": "^29.1.1",
-    "react": "^19.2.7",
-    "react-dom": "^19.2.7",
+    "react": "^19.2.8",
+    "react-dom": "^19.2.8",
     "rolldown": "~1.1.5",
     "rolldown-plugin-dts": "^0.27.14",
     "typescript": "~7.0.2",
-    "vite": "^8.1.5",
+    "vite": "^8.1.6",
     "vitest": "^4.1.10",
     "zod": "^4.4.3"
   }

package/skills/appearance/SKILL.md

@@ -18,7 +18,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/styling.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/toolbar.md'

package/skills/cell-selection/SKILL.md

@@ -14,7 +14,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/cell-selection.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/src/core/cellNavigation.ts'

package/skills/columns/SKILL.md

@@ -16,7 +16,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/columns.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/column-layout.md'

package/skills/data/SKILL.md

@@ -18,7 +18,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/pagination.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/scrolling.md'

package/skills/editing/SKILL.md

@@ -18,7 +18,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/editing.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/draft-store.md'

package/skills/filtering/SKILL.md

@@ -18,7 +18,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/filtering.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/quick-search.md'

package/skills/getting-started/SKILL.md

@@ -13,7 +13,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/getting-started.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/anatomy.md'

package/skills/grouping/SKILL.md

@@ -14,7 +14,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/grouping.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/summary-row.md'

package/skills/migrating-to-2/SKILL.md

@@ -15,7 +15,7 @@ description: >
 metadata:
   type: lifecycle
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/migrating-to-2.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/src/index.ts'

package/skills/options/SKILL.md

@@ -14,7 +14,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/use-tm-data-grid.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/persistence.md'
@@ -57,7 +57,7 @@ rather than forwarded to TanStack.
 | `enableSorting` | `boolean` | `true` | Enables sorting for the table. |
 | `enableColumnFilters` | `boolean` | `true` | Enables filtering for the table. |
 | `enableHiding` | `boolean` | `true` | Enables column visibility for the table. |
-| `enableColumnPinning` | `boolean` | `true` | Enables pinning for the table. |
+| `enableColumnPinning` | `boolean` | `true` | Enables pinning for the table. `false` removes the pin menu items; the generated lanes stay pinned. |
 | `enableColumnResizing` | `boolean` | `true` | Enables resizing for the table. |
 | `enableColumnOrdering` | `boolean` | `true` | Enables header dragging and the move menu items. Defined by the grid. |
 | `enablePagination` | `boolean` | `false` | Enables client-side paging and the `Footer` pager. Implied by `manualPagination`. Defined by the grid. |

package/skills/rows/SKILL.md

@@ -17,7 +17,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/row-selection.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/row-interaction.md'

package/skills/server-side/SKILL.md

@@ -15,7 +15,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/server-side.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/server-query.md'

package/skills/testing/SKILL.md

@@ -13,7 +13,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/testing.md'
   - 'Jielga/TMDataGrid:playwright/support/DataGrid.ts'

package/skills/testing-components/SKILL.md

@@ -12,7 +12,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/testing.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/test/gallery/main.tsx'

package/skills/testing-editing/SKILL.md

@@ -12,7 +12,7 @@ description: >
 metadata:
   type: core
   library: '@jielga/tmdatagrid'
-  library_version: '2.1.0'
+  library_version: '2.1.1'
 sources:
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/testing.md'
   - 'Jielga/TMDataGrid:packages/tmdatagrid/docs/editing.md'

package/src/components/TMDataGridTable.tsx

@@ -159,9 +159,9 @@ function keepColumnInView({
  * | ---- | ------- |
  * | On a data row | the column's `cell`, as always |
  * | Placeholder - a grouped column other than this row's own | nothing |
- * | Aggregated, column declares `aggregatedCell` | that |
- * | Aggregated, column declares an `aggregationFn` | the column's `cell`, over the aggregate |
- * | Aggregated, column declares neither | nothing |
+ * | Group summary, column declares `aggregatedCell` | that |
+ * | Group summary, column declares an `aggregationFn` | the column's `cell`, over the aggregate |
+ * | Group summary, column declares neither | nothing |
  *
  * That last row is the one worth spelling out. Without an aggregation function
  * `getValue()` is `undefined` on a group row, so the column's own renderer
@@ -174,17 +174,31 @@ function renderCellContent(
   cell: Cell<TMDataGridFeatures, TMDataGridRowData, unknown>,
 ) {
   if (cell.getIsPlaceholder()) return null;
-  if (!cell.getIsAggregated()) {
+  if (!isGroupSummaryCell(cell)) {
     return flexRender(cell.column.columnDef.cell, cell.getContext());
   }
-  const { aggregatedCell, aggregationFn } = cell.column.columnDef;
+  const { aggregatedCell } = cell.column.columnDef;
   if (aggregatedCell !== undefined) {
     return flexRender(aggregatedCell, cell.getContext());
   }
-  if (aggregationFn === undefined) return null;
+  if (!cell.getIsAggregated()) return null;
   return flexRender(cell.column.columnDef.cell, cell.getContext());
 }
 
+/**
+ * A cell on a group row in any column but the one the row groups by - the
+ * cells that summarise the group rather than show a record.
+ *
+ * Not `cell.getIsAggregated()`: since TanStack Table 9.2 that is true only
+ * when the column has an aggregation function, so it cannot tell a group
+ * row's blank cell from a data row's cell.
+ */
+function isGroupSummaryCell(
+  cell: Cell<TMDataGridFeatures, TMDataGridRowData, unknown>,
+): boolean {
+  return cell.row.getIsGrouped() && !cell.getIsGrouped();
+}
+
 /**
  * What a cell needs to take part in cell selection, or `undefined` while the
  * feature is off - which is what keeps a plain grid's cells exactly as they
@@ -401,7 +415,7 @@ function TMDataGridBodyCell({
               draftValues !== undefined &&
               contentOverride === undefined &&
               !cell.getIsPlaceholder() &&
-              !cell.getIsAggregated()
+              !isGroupSummaryCell(cell)
                 ? draftCellContext(cell, draftValues)
                 : undefined;
             // Highlighting reproduces the *default* renderer, value then
@@ -412,7 +426,7 @@ function TMDataGridBodyCell({
               highlightNeedles !== undefined &&
               contentOverride === undefined &&
               !cell.getIsPlaceholder() &&
-              !cell.getIsAggregated() &&
+              !isGroupSummaryCell(cell) &&
               cell.column.columnDef.cell === table.getDefaultColumnDef().cell
             ) {
               const value =

package/src/components/generatedColumns.tsx

@@ -48,8 +48,8 @@ export function createSelectColumn<TData extends RowData>(
     enablePinning: false,
     header: ({ table }) => <SelectAllHeader table={table} />,
     cell: ({ row }) => <SelectRowCheckbox row={row} />,
-    // Every cell on a group row that is not the grouped column counts as
-    // aggregated, this lane included, and an aggregated cell with nothing
+    // Every cell on a group row that is not the grouped column is a group
+    // summary cell, this lane included, and a summary cell with nothing
     // declared renders blank. Without this the checkbox would disappear from
     // exactly the rows that select a whole group. See renderCellContent.
     aggregatedCell: ({ row }) => <SelectRowCheckbox row={row} />,
@@ -90,10 +90,10 @@ export function createGroupColumn<TData extends RowData>(
     // Structurally pinned to the left; users shouldn't be able to move it.
     enablePinning: false,
     cell: ({ row }) => <GroupCell row={row} />,
-    // A group row has subRows, so every cell on it that is not the grouped
-    // column reports `getIsAggregated()` - this lane included. Without an
-    // `aggregatedCell` the body would take that as "nothing to summarise" and
-    // render the tree lane blank on exactly the rows it exists for.
+    // Every cell on a group row that is not the grouped column is a group
+    // summary cell - this lane included. Without an `aggregatedCell` the body
+    // would take that as "nothing to summarise" and render the tree lane
+    // blank on exactly the rows it exists for. See renderCellContent.
     aggregatedCell: ({ row }) => <GroupCell row={row} />,
     header: ({ table }) => <GroupHeader table={table} />,
   };
@@ -142,7 +142,7 @@ export function createDetailsColumn<TData extends RowData>(
     header: ({ table }) => <DetailsHeader table={table} />,
     cell: ({ row }) => <DetailsCell row={row} />,
     // Deliberately no `aggregatedCell`: on a group row every cell outside the
-    // grouped column counts as aggregated, and blank is the right answer here -
+    // grouped column is a summary cell, and blank is the right answer here -
     // groups expand into their rows, not into a panel.
   };
 }
@@ -182,7 +182,7 @@ export function createEditColumn<TData extends RowData>(
     header: () => null,
     cell: ({ row }) => <EditLaneCell row={row} />,
     // Group rows: same reasoning as the checkbox lane - without this the
-    // cell renders blank on aggregated rows, but here blank is also correct,
+    // cell renders blank on group rows, but here blank is also correct,
     // so the aggregated cell renders the same (null for groups).
     aggregatedCell: ({ row }) => <EditLaneCell row={row} />,
   };

package/src/useTMDataGrid.tsx

@@ -1238,7 +1238,6 @@ export function useTMDataGrid<TData extends RowData>({
     ],
   );
 
-  const pinningEnabled = options.enableColumnPinning !== false;
   const selectColumnEnabled = features.selectColumn;
   const groupColumnEnabled = features.grouping;
   // The lane that opens the panels. Nothing to switch on: a grid with no
@@ -1270,13 +1269,15 @@ export function useTMDataGrid<TData extends RowData>({
   // and re-applied on top of anything restored: a snapshot cannot unpin one,
   // and a snapshot taken before `detailsColumnPosition` changed cannot pin the
   // details lane to the edge it has since left.
+  //
+  // Not gated on `enableColumnPinning`: that switch takes pinning away from the
+  // user, not from the grid's own lanes, which hold the edges either way.
   const structuralPinning = (
     restored: Partial<ColumnPinningState> | undefined,
   ): ColumnPinningState => {
     const own = (ids: string[] | undefined) =>
       (ids ?? []).filter((id) => !isGeneratedColumn(id));
-    const lane = (enabled: boolean, id: string) =>
-      enabled && pinningEnabled ? [id] : [];
+    const lane = (enabled: boolean, id: string) => (enabled ? [id] : []);
     const detailsOn = (edge: "left" | "right") =>
       detailsColumnEnabled && detailsColumnPosition === edge;
     return {