@skygraph/react 0.5.3 → 0.5.4

package/README.md

@@ -1,6 +1,6 @@
 # @skygraph/react
 
-React adapter for [SkyGraph](https://skygraph.ruslansinkevich.ru/) — 50+ components, hooks and `ConfigProvider` built on the framework-agnostic [`@skygraph/core`](../core) engine. Shares the `.sg-*` CSS contract with the Vue and Angular adapters via [`@skygraph/styles`](../styles).
+React adapter for [SkyGraph](https://skygraph.ruslansinkevich.ru/) — components, hooks and `ConfigProvider` built on the framework-agnostic [`@skygraph/core`](../core) engine. Shares the `.sg-*` CSS contract with the Vue and Angular adapters via [`@skygraph/styles`](../styles).
 
 > **Easier path — the meta-package:**
 >
@@ -33,7 +33,7 @@ export function App() {
 }
 ```
 
-## Components (53 documented)
+## Components
 
 **Data entry:** Button, Input, InputNumber, Textarea, Select, Checkbox, Radio, Switch, Slider, DatePicker, TimePicker, AutoComplete, Rate, Upload, ColorPicker, Cascader, TreeSelect, Mentions
 

package/dist/{chunk-P4J4PFBG.js → chunk-7AA2JPZB.js}

@@ -1596,14 +1596,11 @@ function TableBody(props) {
       onContextMenu({ x: e.clientX, y: e.clientY, items });
     }
   };
-  const handleCellCopy = useCallback2(
-    async (rowId, colKey, value) => {
-      await copyToClipboard(String(value ?? ""));
-      setCopiedCell(`${rowId}:${colKey}`);
-      setTimeout(() => setCopiedCell(null), 1500);
-    },
-    []
-  );
+  const handleCellCopy = useCallback2(async (rowId, colKey, value) => {
+    await copyToClipboard(String(value ?? ""));
+    setCopiedCell(`${rowId}:${colKey}`);
+    setTimeout(() => setCopiedCell(null), 1500);
+  }, []);
   if (flatRows.length === 0) {
     return /* @__PURE__ */ jsx6(
       "div",
@@ -1668,7 +1665,13 @@ function TableBody(props) {
               {
                 className: "sg-table-td sg-table-cell-drag-handle",
                 role: "cell",
-                style: { cursor: "grab", userSelect: "none", display: "flex", alignItems: "center", justifyContent: "center" },
+                style: {
+                  cursor: "grab",
+                  userSelect: "none",
+                  display: "flex",
+                  alignItems: "center",
+                  justifyContent: "center"
+                },
                 children: "\u283F"
               }
             ),
@@ -1680,10 +1683,24 @@ function TableBody(props) {
                 role: "cell",
                 onClick: (e) => {
                   e.stopPropagation();
-                  onSelectRow(row.id, row.data);
+                  if (e.target === e.currentTarget) {
+                    onSelectRow(row.id, row.data);
+                  }
                 },
-                children: rowSelection.type === "radio" ? /* @__PURE__ */ jsx6("input", { type: "radio", checked: !!isSelected, readOnly: true }) : /* @__PURE__ */ jsx6(Checkbox, { checked: !!isSelected, onChange: () => {
-                } })
+                children: rowSelection.type === "radio" ? /* @__PURE__ */ jsx6(
+                  "input",
+                  {
+                    type: "radio",
+                    checked: !!isSelected,
+                    onChange: () => onSelectRow(row.id, row.data)
+                  }
+                ) : /* @__PURE__ */ jsx6(
+                  Checkbox,
+                  {
+                    checked: !!isSelected,
+                    onChange: () => onSelectRow(row.id, row.data)
+                  }
+                )
               }
             ),
             expandable && /* @__PURE__ */ jsx6("div", { className: "sg-table-td sg-table-cell-expand", role: "cell", children: canExpand && /* @__PURE__ */ jsx6(
@@ -1706,10 +1723,8 @@ function TableBody(props) {
               const isFocused = keyboardNavigation && focusedCell?.row === rowIdx && focusedCell?.col === colIndex;
               const isCopied = copiedCell === cellKey;
               const cellStyle = { ...fixedStyle(col) };
-              if (span?.colSpan && span.colSpan > 1)
-                cellStyle.gridColumn = `span ${span.colSpan}`;
-              if (span?.rowSpan && span.rowSpan > 1)
-                cellStyle.gridRow = `span ${span.rowSpan}`;
+              if (span?.colSpan && span.colSpan > 1) cellStyle.gridColumn = `span ${span.colSpan}`;
+              if (span?.rowSpan && span.rowSpan > 1) cellStyle.gridRow = `span ${span.rowSpan}`;
               const isFirstCol = colIndex === 0;
               const treeIndent = isTreeMode && isFirstCol ? row.depth * indentSize : 0;
               const cellClsExtra = typeof col.cellClassName === "function" ? col.cellClassName(val, row.data, row.id) : col.cellClassName ?? "";
@@ -2557,6 +2572,13 @@ function useTableState(props) {
     [table, refresh]
   );
   const resizeRef = useRef6(null);
+  const resizeCleanupRef = useRef6(null);
+  useEffect6(
+    () => () => {
+      resizeCleanupRef.current?.();
+    },
+    []
+  );
   const [contextMenu, setContextMenu] = useState7(null);
   const columns = useMemo3(() => {
     const base = hasColumnGroups ? leafColumns : (() => {
@@ -2682,9 +2704,8 @@ function useTableState(props) {
   );
   const handleToggleExpand = (id) => {
     const open = expandedKeys.has(id);
-    if (expandable?.onExpand) {
-      expandable.onExpand(!open, id);
-    } else {
+    expandable?.onExpand?.(!open, id);
+    if (expandable?.expandedKeys == null) {
       setInternalExpanded((prev) => {
         const next = new Set(prev);
         if (open) {
@@ -2760,28 +2781,34 @@ function useTableState(props) {
   const handleResizeStart = (e, colKey) => {
     e.preventDefault();
     e.stopPropagation();
+    resizeCleanupRef.current?.();
     resizeRef.current = {
       col: colKey,
       startX: e.clientX,
       startW: colWidths[colKey] ?? DEFAULT_COL_WIDTH
     };
     const onMove = (ev) => {
-      if (!resizeRef.current) return;
-      const diff = ev.clientX - resizeRef.current.startX;
-      const min = leafColumns.find((c) => c.key === resizeRef.current.col)?.minWidth ?? MIN_COL_WIDTH;
+      const ref = resizeRef.current;
+      if (!ref) return;
+      const diff = ev.clientX - ref.startX;
+      const min = leafColumns.find((c) => c.key === ref.col)?.minWidth ?? MIN_COL_WIDTH;
+      const nextWidth = Math.max(min, ref.startW + diff);
+      const col = ref.col;
       setColWidths((prev) => ({
         ...prev,
-        [resizeRef.current.col]: Math.max(min, resizeRef.current.startW + diff)
+        [col]: nextWidth
       }));
-      setUserResizedCols(
-        (prev) => prev.has(resizeRef.current.col) ? prev : new Set(prev).add(resizeRef.current.col)
-      );
+      setUserResizedCols((prev) => prev.has(col) ? prev : new Set(prev).add(col));
     };
-    const onUp = () => {
-      const ref = resizeRef.current;
+    const cleanup = () => {
       resizeRef.current = null;
       document.removeEventListener("mousemove", onMove);
       document.removeEventListener("mouseup", onUp);
+      resizeCleanupRef.current = null;
+    };
+    const onUp = () => {
+      const ref = resizeRef.current;
+      cleanup();
       if (!ref) return;
       setColWidths((prev) => {
         const w = prev[ref.col];
@@ -2789,6 +2816,7 @@ function useTableState(props) {
         return prev;
       });
     };
+    resizeCleanupRef.current = cleanup;
     document.addEventListener("mousemove", onMove);
     document.addEventListener("mouseup", onUp);
   };
@@ -2809,13 +2837,11 @@ function useTableState(props) {
       setDragOver(null);
       return;
     }
-    setColOrder((prev) => {
-      const next = prev.filter((k) => k !== dragCol);
-      const idx = next.indexOf(targetKey);
-      next.splice(idx, 0, dragCol);
-      onColumnOrderChange?.(next);
-      return next;
-    });
+    const next = colOrder.filter((k) => k !== dragCol);
+    const idx = next.indexOf(targetKey);
+    next.splice(idx, 0, dragCol);
+    setColOrder(next);
+    onColumnOrderChange?.(next);
     setDragCol(null);
     setDragOver(null);
   };
@@ -2846,31 +2872,24 @@ function useTableState(props) {
     }));
     table.groupBy(groupBy, aggregates2);
   }, [groupBy, leafColumns, table]);
-  const [expandedGroups, setExpandedGroups] = useState7(() => {
-    return defaultGroupExpanded ? /* @__PURE__ */ new Set(["__all__"]) : /* @__PURE__ */ new Set();
-  });
+  const [expandedGroups, setExpandedGroups] = useState7(() => /* @__PURE__ */ new Set());
+  const allGroupKeysRef = useRef6([]);
+  const groupsSeededRef = useRef6(false);
   const toggleGroupExpand = useCallback3(
     (groupKey) => {
+      const willExpand = !expandedGroups.has(groupKey);
       setExpandedGroups((prev) => {
         const next = new Set(prev);
-        const wasExpanded = next.has(groupKey);
-        if (wasExpanded) {
-          next.delete(groupKey);
-        } else {
-          next.add(groupKey);
-        }
-        onGroupExpandChange?.(groupKey, !wasExpanded);
+        if (next.has(groupKey)) next.delete(groupKey);
+        else next.add(groupKey);
         return next;
       });
+      onGroupExpandChange?.(groupKey, willExpand);
     },
-    [onGroupExpandChange]
+    [onGroupExpandChange, expandedGroups]
   );
   const expandAllGroups = useCallback3(() => {
-    setExpandedGroups((prev) => {
-      const next = new Set(prev);
-      next.add("__all__");
-      return next;
-    });
+    setExpandedGroups(new Set(allGroupKeysRef.current));
   }, []);
   const collapseAllGroups = useCallback3(() => {
     setExpandedGroups(/* @__PURE__ */ new Set());
@@ -2963,11 +2982,32 @@ function useTableState(props) {
     }
     return flattenTreeRows(visibleRows, childrenKey, treeExpanded);
   }, [visibleRows, isTreeMode, childrenKey, treeExpanded]);
+  const allGroupKeys = useMemo3(() => {
+    if (!groupBy) return [];
+    const seen = /* @__PURE__ */ new Set();
+    const keys = [];
+    for (const r of baseFlatRows) {
+      const k = String(r.data[groupBy] ?? "Other");
+      if (!seen.has(k)) {
+        seen.add(k);
+        keys.push(k);
+      }
+    }
+    return keys;
+  }, [baseFlatRows, groupBy]);
+  allGroupKeysRef.current = allGroupKeys;
+  useEffect6(() => {
+    groupsSeededRef.current = false;
+  }, [groupBy]);
+  useEffect6(() => {
+    if (!groupBy || !defaultGroupExpanded) return;
+    if (groupsSeededRef.current || allGroupKeys.length === 0) return;
+    groupsSeededRef.current = true;
+    setExpandedGroups(new Set(allGroupKeys));
+  }, [groupBy, defaultGroupExpanded, allGroupKeys]);
   const groupedRows = useMemo3(() => {
     if (!groupBy) return baseFlatRows;
-    const allExpanded = expandedGroups.has("__all__");
-    const effectiveExpanded = allExpanded ? /* @__PURE__ */ new Set([...expandedGroups, ...baseFlatRows.map((r) => String(r.data[groupBy] ?? "Other"))]) : expandedGroups;
-    return groupByColumn(baseFlatRows, groupBy, columns, effectiveExpanded);
+    return groupByColumn(baseFlatRows, groupBy, columns, expandedGroups);
   }, [baseFlatRows, groupBy, columns, expandedGroups]);
   const {
     top: pinnedTop,
@@ -3838,4 +3878,4 @@ export {
   printElement,
   Table
 };
-//# sourceMappingURL=chunk-P4J4PFBG.js.map
+//# sourceMappingURL=chunk-7AA2JPZB.js.map