@ai-matrx/design-system 0.59.0 → 0.59.1

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.59.1
+### Unreleased
+
+- **A row whose own facts moved is drawn again for real.** 0.59.0's element comparison returned a
+  row's old element whenever the trees matched — so a host that reads its state through a stable
+  handle and names the changed row with `rowVersion` (records-ui's grid) never showed an opened
+  cell: Enter, F2 and typing did nothing, and every records-ui grid keyboard test failed. The
+  comparison now runs only when nothing but the host's functions/objects moved. Guard:
+  `MatrxDataTable.a-row-version-redraws-its-row.test.tsx` (red on 0.59.0).
+
 ## 0.59.0
 
 - **A host's inline functions never redraw the table.** A host that writes `cell`,

package/dist/data-table/index.cjs

@@ -35792,9 +35792,9 @@ var RowDrawings = class {
    * The element to hand React for this row. The SAME object as last time when `raw` draws what the
    * last one drew (its handlers are re-pointed); a new one otherwise.
    */
-  draw(rowId, raw) {
+  draw(rowId, raw, fresh = false) {
     const previous = this.rows.get(rowId);
-    if (previous) {
+    if (previous && !fresh) {
       const updates = [];
       if (sameDrawing(previous.raw, raw, "", "", 0, updates) && updates.every(([path2]) => previous.slots.has(path2))) {
         for (const [path2, fn] of updates) previous.slots.get(path2).fn = fn;
@@ -52709,8 +52709,12 @@ function versionFacts(version2) {
 function sameBodyRow(a, b) {
   return a.row === b.row && a.index === b.index && a.epoch === b.epoch && a.probe === b.probe && a.rowId === b.rowId && a.facts.length === b.facts.length && a.facts.every((fact, i) => Object.is(fact, b.facts[i]));
 }
-var MatrxTableBodyRow = (0, import_react29.memo)(function MatrxTableBodyRow2({ rowId, row, index, draw }) {
-  return draw(rowId, row, index);
+var MatrxTableBodyRow = (0, import_react29.memo)(function MatrxTableBodyRow2(props) {
+  const { rowId, row, index, draw } = props;
+  const last = (0, import_react29.useRef)(null);
+  const fresh = last.current === null || !sameBodyRow({ ...last.current, probe: props.probe }, props);
+  last.current = props;
+  return draw(rowId, row, index, fresh);
 }, sameBodyRow);
 var MatrxTableChrome = (0, import_react29.memo)(function MatrxTableChrome2({ render }) {
   return /* @__PURE__ */ (0, import_jsx_runtime59.jsx)(import_jsx_runtime59.Fragment, { children: render() });
@@ -55229,7 +55233,7 @@ function MatrxDataTableCore(coreProps) {
   const rowProbe = useRenderEpoch(rowShape.filter((value) => typeof value === "function" || typeof value === "object" && value !== null));
   const rowDrawings = (0, import_react29.useRef)(null);
   rowDrawings.current ??= new RowDrawings();
-  const drawBodyRow = (rowId, row, index) => rowDrawings.current.draw(rowId, renderRecordRow(row, index));
+  const drawBodyRow = (rowId, row, index, fresh) => rowDrawings.current.draw(rowId, renderRecordRow(row, index), fresh);
   chromeLatest.current = { processed, data, toggleAllOnPage, showMore: progressiveRows.showMore, applySavedView };
   const chromeGeometry = [
     ...Object.entries(effectiveColumnWidths).flat(),