@jinaan/uikit 0.1.49 → 0.1.50

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jinaan/uikit",
-  "version": "0.1.49",
+  "version": "0.1.50",
   "description": "Reusable React and Tailwind UI components for Jinaan projects.",
   "license": "UNLICENSED",
   "type": "module",

package/src/data-table.tsx

@@ -441,16 +441,7 @@ export function DataTable<T>({
   useLayoutEffect(() => {
     const table = bodyTableRef.current;
     if (!table) return;
-    if (structureRef.current !== structureKey || contentRef.current !== contentKey) {
-      // Column set or row content changed: drop the pinned widths so the body
-      // re-renders content-sized (table-auto) and the next pass measures
-      // intrinsic widths again. Without this, the effect would measure the
-      // stale colgroup's shifted layout and pin the wrong (truncating) widths.
-      structureRef.current = structureKey;
-      contentRef.current = contentKey;
-      setColumnWidths((prev) => (prev.length === 0 ? prev : []));
-      return;
-    }
+
     const measure = () => {
       const cells = table.tBodies[0]?.rows.item(0)?.cells;
       if (!cells || cells.length === 0) {
@@ -481,6 +472,23 @@ export function DataTable<T>({
         prev.length === widths.length && prev.every((w, i) => w === widths[i]) ? prev : widths,
       );
     };
+    if (structureRef.current !== structureKey || contentRef.current !== contentKey) {
+      // Column set or row content changed: drop the pinned widths so the body
+      // re-renders content-sized (table-auto) and the next pass measures
+      // intrinsic widths again. Without this, the effect would measure the
+      // stale colgroup's shifted layout and pin the wrong (truncating) widths.
+      structureRef.current = structureKey;
+      contentRef.current = contentKey;
+      if (pinned) {
+        // The widths -> [] state flip re-runs this effect (see `pinned` in the
+        // deps); measuring now would read the still-pinned layout.
+        setColumnWidths([]);
+        return;
+      }
+      // Already unpinned: no state flip would re-run the effect, so measure
+      // the content-sized layout right away.
+    }
+
     measure();
     syncBandScroll();
     if (typeof ResizeObserver === "undefined") return;