@svgrid/grid 2.6.22 → 2.6.23

package/src/clipboard.ts

@@ -13,6 +13,7 @@ import {
   getCellKey,
 } from "./SvGrid.helpers";
 import {
+  createDataIndexLookup,
   getColumnBaseValue,
   isGroupRow,
   toolPanelHeaderLabel,
@@ -22,43 +23,61 @@ export function createClipboard<
   TFeatures extends TableFeatures = TableFeatures,
   TData extends RowData = RowData,
 >(ctx: any) {
+  /**
+   * `rowIndex` here is a DISPLAY index - a position in `allRows`, the space the
+   * selection rectangle, the fill drag and `isCellEditableAt` all work in.
+   * These two functions used to index `internalData` with it directly, which is
+   * the same row only while the grid is unsorted, unfiltered and unpaginated.
+   * Sort a column and the fill handle read its pattern from one row and wrote
+   * to another; enterprise bulk edit wrote to the wrong rows the same way.
+   *
+   * `row.original` IS the object in `data`, so identity resolves the slot no
+   * matter how the view is ordered.
+   */
+  function rowDataAt(rowIndex: number): Record<string, unknown> | undefined {
+    return ctx.allRows[rowIndex]?.original as Record<string, unknown> | undefined;
+  }
+
   /** Read the raw underlying value for the cell at (rowIndex, columnId)
    *  for pattern extraction. */
   function readCellRaw(rowIndex: number, columnId: string): unknown {
-    const row = ctx.internalData[rowIndex];
+    const row = rowDataAt(rowIndex);
     const column = ctx.findColumnById(columnId);
     if (!row || !column?.columnDef.field) return undefined;
-    return (row as Record<string, unknown>)[column.columnDef.field];
+    return row[column.columnDef.field];
   }
 
   /** Write a value into (rowIndex, columnId) without going through the
    *  edit lifecycle. Fires `onCellValueChange` per write so consumers
    *  can react (formula recompute, autosave, etc.). */
   function writeCellRaw(rowIndex: number, columnId: string, value: unknown) {
-    const row = ctx.internalData[rowIndex];
+    const row = rowDataAt(rowIndex);
     const column = ctx.findColumnById(columnId);
     if (!row || !column?.columnDef.field) return;
     const field = column.columnDef.field;
-    const oldValue = (row as Record<string, unknown>)[field];
+    const oldValue = row[field];
     if (oldValue === value) return;
-    // Resolve the row's id BEFORE swapping internalData - otherwise the
-    // recomputed `allRows` references the new row object and the
-    // `r.original === row` lookup fails, dropping our edit out of the
-    // `editedCellValues` map (which getCellDisplayValue consults first).
-    const rowId = ctx.allRows.find((r: any) => r.original === row)?.id;
+    // The slot in `data` backing this displayed row. Also read BEFORE swapping
+    // internalData: afterwards the recomputed `allRows` references the new row
+    // object, so an `original`-based lookup would miss and drop the edit out of
+    // `editedCellValues` (which getCellDisplayValue consults first).
+    const dataIndex = ctx.internalData.indexOf(row as TData);
+    if (dataIndex < 0) return;
+    const rowId = ctx.allRows[rowIndex]?.id;
     const next = ctx.internalData.slice() as Array<TData>;
-    next[rowIndex] = { ...row, [field]: value } as TData;
+    next[dataIndex] = { ...row, [field]: value } as TData;
     ctx.internalData = next;
     if (rowId) {
       const key = getCellKey(rowId, columnId);
       ctx.editedCellValues = { ...ctx.editedCellValues, [key]: value };
     }
     ctx.props.onCellValueChange?.({
-      rowIndex,
+      // The position in `props.data`, matching what the inline editor reports.
+      rowIndex: dataIndex,
       columnId,
       oldValue,
       newValue: value,
-      row: next[rowIndex] as TData,
+      row: next[dataIndex] as TData,
     });
   }
 
@@ -71,6 +90,7 @@ export function createClipboard<
     // Clear fillDrag FIRST so a thrown error doesn't leave the grid
     // stuck tracking the pointer.
     ctx.fillDrag = null;
+    stopEdgeScroll();
     const newMinRow = Math.min(d.sourceMinRow, d.targetRow);
     const newMaxRow = Math.max(d.sourceMaxRow, d.targetRow);
     const newMinCol = Math.min(d.sourceMinCol, d.targetCol);
@@ -222,24 +242,121 @@ export function createClipboard<
     // (`onWindowPointerMove` / `endDragSelection`) keep tracking.
   }
 
-  /** Pointermove during fill-drag. We don't get cell coords from the
-   *  event directly - find the td under the pointer via elementFromPoint
-   *  and read its data-svgrid-row / data-svgrid-col attributes. */
-  function onFillPointerMove(event: PointerEvent) {
-    if (!ctx.fillDrag) return;
-    const el = document.elementFromPoint(
-      event.clientX,
-      event.clientY,
-    ) as HTMLElement | null;
+  /** The grid cell under a viewport point, or null. Drags cannot read their
+   *  target off the event: the pointer is over whatever `<td>` happens to be
+   *  there, and during a fill/move drag the event target is the cell the drag
+   *  STARTED on. */
+  function cellAtPoint(x: number, y: number): { row: number; col: number } | null {
+    const el = document.elementFromPoint(x, y) as HTMLElement | null;
     const cell = el?.closest(
       "td[data-svgrid-row][data-svgrid-col]",
     ) as HTMLElement | null;
-    if (!cell) return;
-    const r = Number(cell.dataset.svgridRow);
-    const c = Number(cell.dataset.svgridCol);
-    if (!Number.isFinite(r) || !Number.isFinite(c)) return;
-    if (r === ctx.fillDrag.targetRow && c === ctx.fillDrag.targetCol) return;
-    ctx.fillDrag = { ...ctx.fillDrag, targetRow: r, targetCol: c };
+    if (!cell) return null;
+    const row = Number(cell.dataset.svgridRow);
+    const col = Number(cell.dataset.svgridCol);
+    if (!Number.isFinite(row) || !Number.isFinite(col)) return null;
+    return { row, col };
+  }
+
+  // ---- Edge auto-scroll -------------------------------------------------
+  // Shared by all three drags that extend a rectangle - fill, range move, and
+  // plain drag-select. Without it a drag can only reach cells that are already
+  // on screen, which on a grid the size this one is built for means most of
+  // them are unreachable.
+
+  /** How far from the viewport edge the scroll starts, in CSS pixels. */
+  const EDGE_BAND_PX = 40;
+  /** Fastest scroll, in pixels per frame, reached at the very edge. */
+  const EDGE_MAX_PX = 20;
+
+  let edgeFrame: number | null = null;
+  let edgePointer: { x: number; y: number } | null = null;
+
+  function stopEdgeScroll() {
+    if (edgeFrame != null) cancelAnimationFrame(edgeFrame);
+    edgeFrame = null;
+    edgePointer = null;
+  }
+
+  /**
+   * Pixels to scroll this frame along one axis. Zero in the middle, ramping to
+   * EDGE_MAX_PX at the edge and staying there once the pointer leaves the
+   * container entirely - a linear ramp rather than a constant step, so nudging
+   * one row past the edge creeps instead of lurching.
+   */
+  function edgeVelocity(pos: number, min: number, max: number) {
+    if (pos < min + EDGE_BAND_PX) {
+      const depth = Math.min(EDGE_BAND_PX, min + EDGE_BAND_PX - pos);
+      return -Math.ceil((depth / EDGE_BAND_PX) * EDGE_MAX_PX);
+    }
+    if (pos > max - EDGE_BAND_PX) {
+      const depth = Math.min(EDGE_BAND_PX, pos - (max - EDGE_BAND_PX));
+      return Math.ceil((depth / EDGE_BAND_PX) * EDGE_MAX_PX);
+    }
+    return 0;
+  }
+
+  /** Point the in-flight drag at whatever cell is under (x, y) now. */
+  function retargetDrag(x: number, y: number) {
+    const hit = cellAtPoint(x, y);
+    if (!hit) return;
+    if (ctx.moveDrag) {
+      if (hit.row === ctx.moveDrag.targetRow && hit.col === ctx.moveDrag.targetCol) return;
+      ctx.moveDrag = { ...ctx.moveDrag, targetRow: hit.row, targetCol: hit.col };
+    } else if (ctx.fillDrag) {
+      if (hit.row === ctx.fillDrag.targetRow && hit.col === ctx.fillDrag.targetCol) return;
+      ctx.fillDrag = { ...ctx.fillDrag, targetRow: hit.row, targetCol: hit.col };
+    } else if (ctx.isDraggingSelection) {
+      ctx.extendSelection(hit.row, hit.col);
+      ctx.setActiveCell(hit.row, hit.col);
+    }
+  }
+
+  function stepEdgeScroll() {
+    edgeFrame = null;
+    const scroller = ctx.scrollContainer as HTMLElement | null;
+    const p = edgePointer;
+    const dragging = ctx.moveDrag || ctx.fillDrag || ctx.isDraggingSelection;
+    if (!scroller || !p || !dragging) {
+      stopEdgeScroll();
+      return;
+    }
+    const rect = scroller.getBoundingClientRect();
+    const dx = edgeVelocity(p.x, rect.left, rect.right);
+    const dy = edgeVelocity(p.y, rect.top, rect.bottom);
+    if (dx || dy) {
+      const fromX = scroller.scrollLeft;
+      const fromY = scroller.scrollTop;
+      scroller.scrollLeft += dx;
+      scroller.scrollTop += dy;
+      // Only re-target when the view actually moved. At either end the scroll
+      // clamps, and re-targeting then would fight a pointer that is parked
+      // still - the selection would flicker between two cells every frame.
+      if (scroller.scrollLeft !== fromX || scroller.scrollTop !== fromY) {
+        retargetDrag(p.x, p.y);
+      }
+    }
+    edgeFrame = requestAnimationFrame(stepEdgeScroll);
+  }
+
+  /**
+   * Record where the pointer is and make sure the scroll loop is running. The
+   * loop keeps going after the pointer stops moving, which is the whole point:
+   * a user parks the cursor at the edge and expects the grid to keep coming.
+   */
+  function trackEdgeScroll(event: PointerEvent) {
+    edgePointer = { x: event.clientX, y: event.clientY };
+    if (edgeFrame == null) edgeFrame = requestAnimationFrame(stepEdgeScroll);
+  }
+
+  /** Pointermove during fill-drag. */
+  function onFillPointerMove(event: PointerEvent) {
+    if (!ctx.fillDrag) return;
+    trackEdgeScroll(event);
+    const hit = cellAtPoint(event.clientX, event.clientY);
+    if (!hit) return;
+    if (hit.row === ctx.fillDrag.targetRow && hit.col === ctx.fillDrag.targetCol) return;
+    ctx.fillDrag = { ...ctx.fillDrag, targetRow: hit.row, targetCol: hit.col };
   }
 
   function onFillPointerUp() {
@@ -247,6 +364,282 @@ export function createClipboard<
     applyFillPattern();
   }
 
+  // ---- Range move / copy (drag the selection border) --------------------
+
+  /** How thick the draggable strip on a range border is, in CSS pixels.
+   *  Anything further inside the cell keeps its old behaviour of starting a
+   *  fresh selection, so this only claims the outer sliver. */
+  const MOVE_GRAB_PX = 4;
+
+  /** Is (clientX, clientY) on the grab strip of the range border that the
+   *  cell at (rowIndex, colIndex) sits on? Returns false for any cell that
+   *  is not on a range edge, so the common case costs one call to
+   *  `getCellRangeEdges` - which itself bails immediately with no selection. */
+  function isOnMoveGrabStrip(
+    cell: HTMLElement,
+    rowIndex: number,
+    colIndex: number,
+    clientX: number,
+    clientY: number,
+  ): boolean {
+    if (!ctx.moveCellsEffective) return false;
+    const edges = ctx.getCellRangeEdges(rowIndex, colIndex);
+    if (!edges) return false;
+    // One layout read per pointermove, and only once the pointer is over a
+    // cell that IS on a range edge. Reading a single rect is cheap unless
+    // the DOM is dirty, and the only write this path makes is a boolean flag
+    // that changes at most twice per border crossing.
+    const box = cell.getBoundingClientRect();
+    const x = clientX - box.left;
+    const y = clientY - box.top;
+    return (
+      (edges.top && y >= 0 && y <= MOVE_GRAB_PX) ||
+      (edges.bottom && box.height - y >= 0 && box.height - y <= MOVE_GRAB_PX) ||
+      (edges.left && x >= 0 && x <= MOVE_GRAB_PX) ||
+      (edges.right && box.width - x >= 0 && box.width - x <= MOVE_GRAB_PX)
+    );
+  }
+
+  /** Border pointerdown - seed a range move. Returns false when the pointer
+   *  was not on the grab strip, which is the caller's signal to fall through
+   *  to normal selection behaviour. */
+  function startMoveDrag(
+    event: PointerEvent,
+    rowIndex: number,
+    colIndex: number,
+  ): boolean {
+    if (event.button !== 0) return false;
+    // Mouse and pen only, the same policy drag-select uses (issue #23) and for
+    // a worse reason: this path DOES preventDefault, so a finger that happened
+    // to land within 4px of a selection border would start a move and kill the
+    // browser's native scroll for that gesture. There is no hover on touch
+    // either, so the border carries no affordance to find in the first place.
+    // Touch already has cut/paste for the same job.
+    if (event.pointerType === "touch") return false;
+    // `event.target`, not `currentTarget`: Svelte 5 delegates pointerdown to
+    // the root, and the strip test needs the cell's own rect - the target can
+    // be a span inside the cell, so walk up to the td.
+    const cell = (event.target as HTMLElement | null)?.closest?.(
+      "td[data-svgrid-row][data-svgrid-col]",
+    ) as HTMLElement | null;
+    if (!cell) return false;
+    if (!isOnMoveGrabStrip(cell, rowIndex, colIndex, event.clientX, event.clientY))
+      return false;
+    const rect = moveSourceRect(rowIndex, colIndex);
+    if (!rect) return false;
+    event.stopPropagation();
+    // preventDefault stops the drag from rubber-banding a text selection over
+    // the rows it passes, which also means the grid never gets focus from the
+    // gesture - so take it explicitly, or the keyboard stops working after a
+    // move.
+    event.preventDefault();
+    ctx.gridRootEl?.focus({ preventScroll: true });
+    ctx.moveDrag = {
+      ...rect,
+      grabRow: rowIndex,
+      grabCol: colIndex,
+      targetRow: rowIndex,
+      targetCol: colIndex,
+      copy: event.ctrlKey || event.metaKey,
+    };
+    // No `setPointerCapture`, for the same reason the fill handle skips it:
+    // the drag finds its target cell with `elementFromPoint`, and capture
+    // would route every move back to the cell the drag started on.
+    return true;
+  }
+
+  /** The selection rectangle that CONTAINS (rowIndex, colIndex), which is the
+   *  one whose border the user grabbed. With several committed ranges the
+   *  grabbed one is the one that moves, not whichever happens to be active. */
+  function moveSourceRect(rowIndex: number, colIndex: number) {
+    for (const rect of ctx.getSelectionRects()) {
+      if (
+        rowIndex >= rect.minRow &&
+        rowIndex <= rect.maxRow &&
+        colIndex >= rect.minCol &&
+        colIndex <= rect.maxCol
+      ) {
+        return {
+          sourceMinRow: rect.minRow,
+          sourceMaxRow: rect.maxRow,
+          sourceMinCol: rect.minCol,
+          sourceMaxCol: rect.maxCol,
+        };
+      }
+    }
+    return null;
+  }
+
+  function onMovePointerMove(event: PointerEvent) {
+    const d = ctx.moveDrag;
+    if (!d) return;
+    trackEdgeScroll(event);
+    const hit = cellAtPoint(event.clientX, event.clientY);
+    // Track the modifier even when the pointer is between cells, so releasing
+    // Ctrl over a gap still flips a copy back to a move.
+    const copy = event.ctrlKey || event.metaKey;
+    if (!hit) {
+      if (copy !== d.copy) ctx.moveDrag = { ...d, copy };
+      return;
+    }
+    if (hit.row === d.targetRow && hit.col === d.targetCol && copy === d.copy) return;
+    ctx.moveDrag = { ...d, targetRow: hit.row, targetCol: hit.col, copy };
+  }
+
+  function onMovePointerUp(event?: PointerEvent) {
+    if (!ctx.moveDrag) return;
+    // Take the modifier from the RELEASE. Reading it only on pointermove made
+    // "drag, then press Ctrl, then let go" a move - which is the natural way
+    // to do it, and is a copy in every spreadsheet.
+    if (event) {
+      const copy = event.ctrlKey || event.metaKey;
+      if (copy !== ctx.moveDrag.copy) ctx.moveDrag = { ...ctx.moveDrag, copy };
+    }
+    applyMoveRange();
+  }
+
+  /** Where a move would land, or null when the drag has not left the spot it
+   *  started on. Shared by the drop and by the preview marquee so the outline
+   *  can never promise a rectangle the drop would refuse. */
+  function moveDestRect(d: {
+    sourceMinRow: number;
+    sourceMaxRow: number;
+    sourceMinCol: number;
+    sourceMaxCol: number;
+    grabRow: number;
+    grabCol: number;
+    targetRow: number;
+    targetCol: number;
+  }) {
+    const dr = d.targetRow - d.grabRow;
+    const dc = d.targetCol - d.grabCol;
+    if (dr === 0 && dc === 0) return null;
+    const minRow = d.sourceMinRow + dr;
+    const maxRow = d.sourceMaxRow + dr;
+    const minCol = d.sourceMinCol + dc;
+    const maxCol = d.sourceMaxCol + dc;
+    if (minRow < 0 || minCol < 0) return null;
+    if (maxRow >= ctx.allRows.length) return null;
+    if (maxCol >= ctx.allColumns.length) return null;
+    return { minRow, maxRow, minCol, maxCol };
+  }
+
+  /**
+   * Commit a range move (or copy). Refuses the whole operation rather than
+   * applying the part that fits: a move that silently skipped a read-only or
+   * grouped destination cell would leave the range split across two places
+   * with no way to tell which half landed.
+   */
+  function applyMoveRange() {
+    const d = ctx.moveDrag;
+    if (!d) return;
+    // Clear the drag FIRST so a thrown error can't leave the grid tracking
+    // the pointer forever, exactly as applyFillPattern does.
+    ctx.moveDrag = null;
+    stopEdgeScroll();
+    const dest = moveDestRect(d);
+    if (!dest) return;
+
+    const height = d.sourceMaxRow - d.sourceMinRow + 1;
+    const width = d.sourceMaxCol - d.sourceMinCol + 1;
+
+    // Every column the move touches must be field-backed on BOTH sides: a
+    // computed / accessor column has nowhere to write the value back to.
+    for (let c = 0; c < width; c += 1) {
+      if (!ctx.allColumns[d.sourceMinCol + c]?.columnDef.field) return;
+      if (!ctx.allColumns[dest.minCol + c]?.columnDef.field) return;
+    }
+    // Destination must be writable, and so must the source when the values
+    // are leaving it. Group rows fail this through isCellEditableAt.
+    for (let r = 0; r < height; r += 1) {
+      for (let c = 0; c < width; c += 1) {
+        if (!ctx.isCellEditableAt(dest.minRow + r, dest.minCol + c)) return;
+        if (
+          !d.copy &&
+          !ctx.isCellEditableAt(d.sourceMinRow + r, d.sourceMinCol + c)
+        )
+          return;
+      }
+    }
+
+    // Snapshot the source before touching anything - source and destination
+    // overlap on any short drag, so reading lazily would move already-moved
+    // values a second time.
+    const values: unknown[][] = [];
+    for (let r = 0; r < height; r += 1) {
+      const rowValues: unknown[] = [];
+      for (let c = 0; c < width; c += 1) {
+        const column = ctx.allColumns[d.sourceMinCol + c];
+        rowValues.push(readCellRaw(d.sourceMinRow + r, column.id));
+      }
+      values.push(rowValues);
+    }
+
+    const steps: Array<{
+      rowId: string;
+      columnId: string;
+      field: string;
+      before: unknown;
+      after: unknown;
+    }> = [];
+    const write = (rowIndex: number, colIndex: number, value: unknown) => {
+      const column = ctx.allColumns[colIndex];
+      if (!column?.columnDef.field) return;
+      const before = readCellRaw(rowIndex, column.id);
+      if (before === value) return;
+      // Resolve the row id BEFORE the write: writeCellRaw swaps in a fresh
+      // row object, so an `original`-based lookup afterwards would miss.
+      const rowId = ctx.allRows[rowIndex]?.id;
+      writeCellRaw(rowIndex, column.id, value);
+      if (rowId) {
+        steps.push({
+          rowId,
+          columnId: column.id,
+          field: column.columnDef.field as string,
+          before,
+          after: value,
+        });
+      }
+    };
+
+    // Clear the whole source first, then write the destination. With the
+    // values already snapshotted this is correct even when the rectangles
+    // overlap: the destination write always lands after the clear.
+    if (!d.copy) {
+      for (let r = 0; r < height; r += 1) {
+        for (let c = 0; c < width; c += 1) {
+          write(d.sourceMinRow + r, d.sourceMinCol + c, null);
+        }
+      }
+    }
+    for (let r = 0; r < height; r += 1) {
+      for (let c = 0; c < width; c += 1) {
+        write(dest.minRow + r, dest.minCol + c, values[r]![c]);
+      }
+    }
+
+    // One undo entry per changed cell, appended together so a single Ctrl+Z
+    // sequence walks the whole move back. Same shape and truncation rule as
+    // a whole-row commit.
+    if (steps.length) {
+      let hist = ctx.history.slice(0, ctx.historyPtr + 1);
+      for (const step of steps) hist.push(step);
+      if (hist.length > ctx.UNDO_LIMIT) hist = hist.slice(hist.length - ctx.UNDO_LIMIT);
+      ctx.history = hist;
+      ctx.historyPtr = ctx.history.length - 1;
+      ctx.historyVersion += 1;
+    }
+
+    // Follow the values: the range stays selected where it landed, so a
+    // second drag can chain off the first the way it does in a spreadsheet.
+    ctx.selectionRanges = [];
+    ctx.selectionRange = {
+      anchor: { rowIndex: dest.minRow, colIndex: dest.minCol },
+      focus: { rowIndex: dest.maxRow, colIndex: dest.maxCol },
+    };
+    ctx.setActiveCell(dest.minRow, dest.minCol);
+  }
+
   function toggleBooleanCell(rowIndex: number, colIndex: number) {
     const row = ctx.allRows[rowIndex];
     const column = ctx.allColumns[colIndex];
@@ -393,15 +786,14 @@ export function createClipboard<
       newValue: unknown
       row: TData
     }> = [];
+    const dataIndexOf = createDataIndexLookup(next);
     for (let r = startRow; r <= endRow; r += 1) {
       const row = ctx.allRows[r];
       if (!row || isGroupRow(row)) continue;
-      const dataIndex = Number(row.id);
-      if (
-        !Number.isInteger(dataIndex) ||
-        dataIndex < 0 ||
-        dataIndex >= next.length
-      ) continue;
+      // By identity, not `Number(row.id)` - that is the array index only under
+      // the default getRowId, so a custom one cleared the wrong row.
+      const dataIndex = dataIndexOf(row);
+      if (dataIndex < 0) continue;
       const originalRow = next[dataIndex];
       if (!originalRow) continue;
       const updated: Record<string, unknown> = {
@@ -465,6 +857,14 @@ export function createClipboard<
     startFillDrag,
     onFillPointerMove,
     onFillPointerUp,
+    isOnMoveGrabStrip,
+    startMoveDrag,
+    onMovePointerMove,
+    onMovePointerUp,
+    trackEdgeScroll,
+    stopEdgeScroll,
+    applyMoveRange,
+    moveDestRect,
     toggleBooleanCell,
     copySelectionToClipboard,
     clearSelectedCells,

package/src/date-string-column.test.ts

@@ -0,0 +1,87 @@
+/**
+ * `cellDataType: 'dateString'` columns, which hold an ISO date STRING
+ * ('2026-06-27') rather than a Date or a timestamp.
+ *
+ * Two bugs lived here, both reported from the same demo:
+ *
+ *  1. Committing an edit ran through `parseEditorValue('date', ...)`, which
+ *     does `new Date(v).toISOString()` - so '2026-12-25' was stored as
+ *     '2026-12-25T01:00:00.000Z'. The cell then showed a timestamp next to
+ *     neighbours showing plain dates. `dateString` resolves to
+ *     `editorType: 'date'`, so nothing downstream knew the difference.
+ *
+ *  2. That conversion goes through UTC. A local midnight either side of the
+ *     meridian lands on the previous or next day, so the stored CALENDAR DATE
+ *     depended on where the user was sitting.
+ */
+import { describe, expect, it } from 'vitest'
+import { parseEditorValue } from './editors/cell-editors'
+import { toIsoDateLocal, usesRichDateFilter } from './SvGrid.helpers'
+
+describe('parseEditorValue with dateOnly', () => {
+  it('keeps an ISO date exactly as picked', () => {
+    expect(parseEditorValue('date', '2026-12-25', { dateOnly: true })).toBe('2026-12-25')
+  })
+
+  it('trims a datetime down to its calendar date', () => {
+    expect(parseEditorValue('date', '2026-12-25T09:30', { dateOnly: true })).toBe('2026-12-25')
+  })
+
+  it('never sends the value through UTC', () => {
+    // The bug: a date whose UTC form is the day before. Whatever the runner's
+    // zone, the day the user picked has to be the day that is stored.
+    for (const raw of ['2026-01-01', '2026-12-31', '2026-06-15']) {
+      expect(parseEditorValue('date', raw, { dateOnly: true })).toBe(raw)
+    }
+  })
+
+  it('uses local parts when the input is not already ISO', () => {
+    const d = new Date(2026, 11, 25, 23, 30) // local 25 Dec, late evening
+    const out = parseEditorValue('date', d.toString(), { dateOnly: true })
+    expect(out).toBe('2026-12-25')
+  })
+
+  it('returns null for something that is not a date at all', () => {
+    expect(parseEditorValue('date', 'not a date', { dateOnly: true })).toBeNull()
+  })
+
+  it('leaves the normal date editor alone - it still stores a timestamp', () => {
+    // `cellDataType: 'date'` columns hold Date values and DO want the ISO
+    // timestamp. Only dateString opts out.
+    const out = parseEditorValue('date', '2026-12-25')
+    expect(String(out)).toMatch(/^2026-12-2[45]T/)
+  })
+})
+
+describe('toIsoDateLocal', () => {
+  it('formats from local parts, not UTC', () => {
+    expect(toIsoDateLocal(new Date(2026, 0, 1, 0, 30))).toBe('2026-01-01')
+    expect(toIsoDateLocal(new Date(2026, 11, 31, 23, 30))).toBe('2026-12-31')
+  })
+
+  it('is empty for no date', () => {
+    expect(toIsoDateLocal(null)).toBe('')
+    expect(toIsoDateLocal(undefined)).toBe('')
+    expect(toIsoDateLocal(new Date('nonsense'))).toBe('')
+  })
+})
+
+describe('usesRichDateFilter', () => {
+  it('is on for the date-ish editors', () => {
+    for (const t of ['date', 'datetime', 'time'] as const) {
+      expect(usesRichDateFilter(t), t).toBe(true)
+    }
+  })
+
+  it('honours the -native opt-out, same as the cell editor', () => {
+    for (const t of ['date-native', 'datetime-native', 'time-native'] as const) {
+      expect(usesRichDateFilter(t), t).toBe(false)
+    }
+  })
+
+  it('is off for everything else', () => {
+    for (const t of ['text', 'number', 'list', 'checkbox', undefined] as const) {
+      expect(usesRichDateFilter(t as never), String(t)).toBe(false)
+    }
+  })
+})

package/src/editing.test.ts

@@ -925,6 +925,31 @@ describe('clipboard fallbacks (non-paste branches only)', () => {
     expect(ctx.internalData[1]).toMatchObject({ a: 'Z', b: 'W' })
   })
 
+  it('pastes into the selected row when getRowId does not return the array index', () => {
+    // Regression: the row -> data-slot lookup was `Number(row.id)`, which is
+    // the array index only under the DEFAULT getRowId. With 1-based ids
+    // (`getRowId={(t) => String(t.id)}`) every paste landed one row BELOW the
+    // selection; with non-numeric ids it parsed to NaN and wrote nothing.
+    Object.defineProperty(navigator, 'clipboard', {
+      value: undefined,
+      configurable: true,
+    })
+    const { ctx, ed } = editingFor({
+      columns: [{ id: 'a', field: 'a', editorType: 'text', editable: true }],
+      data: [{ a: 'x0' }, { a: 'x1' }, { a: 'x2' }],
+    })
+    ctx._rows.forEach((r: any, i: number) => { r.id = String(i + 1) })
+    ctx.selectionRange = {
+      anchor: { rowIndex: 1, colIndex: 0 },
+      focus: { rowIndex: 1, colIndex: 0 },
+    }
+    ed.onGridPaste({
+      clipboardData: { getData: () => 'PASTED' },
+      preventDefault: vi.fn(),
+    } as unknown as ClipboardEvent)
+    expect(ctx.internalData.map((d: any) => d.a)).toEqual(['x0', 'PASTED', 'x2'])
+  })
+
   it('onGridPaste bails while a cell is being edited (native editor paste wins)', () => {
     Object.defineProperty(navigator, 'clipboard', {
       value: undefined, // fallback path (Firefox / insecure) - would otherwise hijack