@xcelsior/ui-spreadsheets 1.1.1 → 1.1.3

package/dist/index.mjs

@@ -329,7 +329,7 @@ var SpreadsheetCell = ({
       onKeyDown: handleCellKeyDown,
       "data-cell-id": `${rowId}-${column.id}`,
       className: cn(
-        "border border-gray-200 group cursor-pointer select-none",
+        "border border-gray-200 group cursor-pointer transition-colors select-none",
         compactMode ? "text-[10px]" : "text-xs",
         cellPadding,
         column.align === "right" && "text-right",
@@ -1080,71 +1080,56 @@ function ColumnHeaderActions({
   unpinnedTitle = "Pin column",
   className
 }) {
-  const handleClick = (e) => {
-    e.stopPropagation();
-  };
-  const handleKeyDown = (e) => {
-    e.stopPropagation();
-  };
-  return /* @__PURE__ */ jsxs4(
-    "button",
-    {
-      type: "button",
-      className: cn("flex items-center gap-0.5", className),
-      onClick: handleClick,
-      onKeyDown: handleKeyDown,
-      children: [
-        enableFiltering && onFilterClick && /* @__PURE__ */ jsx4(
-          "button",
-          {
-            type: "button",
-            onClick: (e) => {
-              e.stopPropagation();
-              onFilterClick();
-            },
-            className: cn(
-              "p-0.5 hover:bg-gray-200 rounded transition-opacity",
-              hasActiveFilter ? "text-blue-600 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
-            ),
-            title: filterTitle,
-            children: /* @__PURE__ */ jsx4(HiFilter, { className: "h-3 w-3" })
-          }
+  return /* @__PURE__ */ jsxs4("div", { className: cn("flex items-center gap-0.5", className), children: [
+    enableFiltering && onFilterClick && /* @__PURE__ */ jsx4(
+      "button",
+      {
+        type: "button",
+        onClick: (e) => {
+          e.stopPropagation();
+          onFilterClick();
+        },
+        className: cn(
+          "p-0.5 hover:bg-gray-200 rounded transition-opacity",
+          hasActiveFilter ? "text-blue-600 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
         ),
-        enableHighlighting && onHighlightClick && /* @__PURE__ */ jsx4(
-          "button",
-          {
-            type: "button",
-            onClick: (e) => {
-              e.stopPropagation();
-              onHighlightClick();
-            },
-            className: cn(
-              "p-0.5 hover:bg-gray-200 rounded transition-opacity",
-              hasActiveHighlight ? "text-amber-500 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
-            ),
-            title: highlightTitle,
-            children: /* @__PURE__ */ jsx4(AiFillHighlight, { className: "h-3 w-3" })
-          }
+        title: filterTitle,
+        children: /* @__PURE__ */ jsx4(HiFilter, { className: "h-3 w-3" })
+      }
+    ),
+    enableHighlighting && onHighlightClick && /* @__PURE__ */ jsx4(
+      "button",
+      {
+        type: "button",
+        onClick: (e) => {
+          e.stopPropagation();
+          onHighlightClick();
+        },
+        className: cn(
+          "p-0.5 hover:bg-gray-200 rounded transition-opacity",
+          hasActiveHighlight ? "text-amber-500 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
         ),
-        enablePinning && onPinClick && /* @__PURE__ */ jsx4(
-          "button",
-          {
-            type: "button",
-            onClick: (e) => {
-              e.stopPropagation();
-              onPinClick();
-            },
-            className: cn(
-              "p-0.5 hover:bg-gray-200 rounded transition-opacity",
-              isPinned ? "text-blue-600 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
-            ),
-            title: isPinned ? pinnedTitle : unpinnedTitle,
-            children: isPinned ? /* @__PURE__ */ jsx4(MdPushPin, { className: "h-3 w-3" }) : /* @__PURE__ */ jsx4(MdOutlinePushPin, { className: "h-3 w-3" })
-          }
-        )
-      ]
-    }
-  );
+        title: highlightTitle,
+        children: /* @__PURE__ */ jsx4(AiFillHighlight, { className: "h-3 w-3" })
+      }
+    ),
+    enablePinning && onPinClick && /* @__PURE__ */ jsx4(
+      "button",
+      {
+        type: "button",
+        onClick: (e) => {
+          e.stopPropagation();
+          onPinClick();
+        },
+        className: cn(
+          "p-0.5 hover:bg-gray-200 rounded transition-opacity",
+          isPinned ? "text-blue-600 opacity-100" : "text-gray-400 opacity-0 group-hover:opacity-100"
+        ),
+        title: isPinned ? pinnedTitle : unpinnedTitle,
+        children: isPinned ? /* @__PURE__ */ jsx4(MdPushPin, { className: "h-3 w-3" }) : /* @__PURE__ */ jsx4(MdOutlinePushPin, { className: "h-3 w-3" })
+      }
+    )
+  ] });
 }
 ColumnHeaderActions.displayName = "ColumnHeaderActions";
 
@@ -1239,22 +1224,17 @@ function useSpreadsheetPinning({
   columns,
   columnGroups,
   showRowIndex = true,
-  defaultPinnedColumns = [],
-  defaultPinRowIndex = false
+  defaultPinnedColumns = []
 }) {
   const [pinnedColumns, setPinnedColumns] = useState3(() => {
     const map = /* @__PURE__ */ new Map();
     defaultPinnedColumns.forEach((col) => {
-      if (col !== ROW_INDEX_COLUMN_ID) {
-        map.set(col, "left");
-      }
+      map.set(col, "left");
     });
     return map;
   });
-  const [isRowIndexPinned, setIsRowIndexPinned] = useState3(
-    defaultPinRowIndex || defaultPinnedColumns.includes(ROW_INDEX_COLUMN_ID)
-  );
   const [collapsedGroups, setCollapsedGroups] = useState3(/* @__PURE__ */ new Set());
+  const isRowIndexPinned = pinnedColumns.has(ROW_INDEX_COLUMN_ID);
   const handleTogglePin = useCallback((columnId) => {
     setPinnedColumns((prev) => {
       const newMap = new Map(prev);
@@ -1266,21 +1246,12 @@ function useSpreadsheetPinning({
       return newMap;
     });
   }, []);
-  const handleToggleRowIndexPin = useCallback(() => {
-    setIsRowIndexPinned((prev) => !prev);
-  }, []);
   const setPinnedColumnsFromIds = useCallback((columnIds) => {
     const map = /* @__PURE__ */ new Map();
     columnIds.forEach((col) => {
-      if (col !== ROW_INDEX_COLUMN_ID) {
-        map.set(col, "left");
-      }
+      map.set(col, "left");
     });
     setPinnedColumns(map);
-    setIsRowIndexPinned(columnIds.includes(ROW_INDEX_COLUMN_ID));
-  }, []);
-  const setRowIndexPinned = useCallback((pinned) => {
-    setIsRowIndexPinned(pinned);
   }, []);
   const handleToggleGroupCollapse = useCallback((groupId) => {
     setCollapsedGroups((prev) => {
@@ -1306,14 +1277,17 @@ function useSpreadsheetPinning({
         return pinnedColumns.has(column.id);
       });
     }
-    if (pinnedColumns.size === 0) {
+    const nonRowIndexPinned = Array.from(pinnedColumns.keys()).filter(
+      (id) => id !== ROW_INDEX_COLUMN_ID
+    );
+    if (nonRowIndexPinned.length === 0) {
       return result;
     }
     const leftPinned = [];
     const unpinned = [];
     const rightPinned = [];
-    const pinnedLeftIds = Array.from(pinnedColumns.entries()).filter(([, side]) => side === "left").map(([id]) => id);
-    const pinnedRightIds = Array.from(pinnedColumns.entries()).filter(([, side]) => side === "right").map(([id]) => id);
+    const pinnedLeftIds = Array.from(pinnedColumns.entries()).filter(([id, side]) => side === "left" && id !== ROW_INDEX_COLUMN_ID).map(([id]) => id);
+    const pinnedRightIds = Array.from(pinnedColumns.entries()).filter(([id, side]) => side === "right" && id !== ROW_INDEX_COLUMN_ID).map(([id]) => id);
     for (const column of result) {
       const pinSide = pinnedColumns.get(column.id);
       if (pinSide === "left") {
@@ -1328,23 +1302,24 @@ function useSpreadsheetPinning({
     rightPinned.sort((a, b) => pinnedRightIds.indexOf(a.id) - pinnedRightIds.indexOf(b.id));
     return [...leftPinned, ...unpinned, ...rightPinned];
   }, [columns, columnGroups, collapsedGroups, pinnedColumns]);
-  const getRowIndexLeftOffset = useCallback(() => {
-    return 0;
-  }, []);
   const getColumnLeftOffset = useCallback(
     (columnId) => {
-      const pinnedLeft = Array.from(pinnedColumns.entries()).filter(([, side]) => side === "left").map(([id]) => id);
+      if (columnId === ROW_INDEX_COLUMN_ID) {
+        return 0;
+      }
+      const pinnedLeft = Array.from(pinnedColumns.entries()).filter(([id, side]) => side === "left" && id !== ROW_INDEX_COLUMN_ID).map(([id]) => id);
       const index = pinnedLeft.indexOf(columnId);
-      const baseOffset = showRowIndex && isRowIndexPinned ? ROW_INDEX_COLUMN_WIDTH : 0;
+      const isRowIndexPinnedNow = pinnedColumns.has(ROW_INDEX_COLUMN_ID);
+      const baseOffset = showRowIndex && isRowIndexPinnedNow ? ROW_INDEX_COLUMN_WIDTH : 0;
       if (index === -1) return baseOffset;
       let offset = baseOffset;
       for (let i = 0; i < index; i++) {
         const col = columns.find((c) => c.id === pinnedLeft[i]);
-        offset += col?.width || col?.minWidth || 100;
+        offset += col?.minWidth || col?.width || 100;
       }
       return offset;
     },
-    [pinnedColumns, columns, showRowIndex, isRowIndexPinned]
+    [pinnedColumns, columns, showRowIndex]
   );
   const isColumnPinned = useCallback(
     (columnId) => {
@@ -1364,12 +1339,9 @@ function useSpreadsheetPinning({
     collapsedGroups,
     visibleColumns,
     handleTogglePin,
-    handleToggleRowIndexPin,
     handleToggleGroupCollapse,
     setPinnedColumnsFromIds,
-    setRowIndexPinned,
     getColumnLeftOffset,
-    getRowIndexLeftOffset,
     isColumnPinned,
     getColumnPinSide
   };
@@ -2202,13 +2174,14 @@ function useSpreadsheetFiltering({
   onSortChange,
   serverSide = false,
   controlledFilters,
-  controlledSortConfig
+  controlledSortConfig,
+  defaultSortConfig
 }) {
   const [internalFilters, setInternalFilters] = useState7(
     {}
   );
   const [internalSortConfig, setInternalSortConfig] = useState7(
-    null
+    defaultSortConfig ?? null
   );
   const [activeFilterColumn, setActiveFilterColumn] = useState7(null);
   const filters = controlledFilters ?? internalFilters;
@@ -3336,7 +3309,8 @@ function Spreadsheet({
     onSortChange,
     serverSide,
     controlledFilters,
-    controlledSortConfig: controlledSortConfig ?? spreadsheetSettings?.defaultSort
+    controlledSortConfig,
+    defaultSortConfig: spreadsheetSettings.defaultSort
   });
   const {
     getCellHighlight,
@@ -3368,7 +3342,8 @@ function Spreadsheet({
     getColumnPinSide
   } = useSpreadsheetPinning({
     columns,
-    columnGroups
+    columnGroups,
+    defaultPinnedColumns: initialSettings?.defaultPinnedColumns
   });
   const {
     getCellComments,
@@ -3437,6 +3412,27 @@ function Spreadsheet({
       defaultSort: sortConfig
     }));
   }, [sortConfig]);
+  useEffect6(() => {
+    const pinnedColumnIds = Array.from(pinnedColumns.keys());
+    setSpreadsheetSettings((prev) => {
+      const prevIds = prev.defaultPinnedColumns;
+      if (prevIds.length === pinnedColumnIds.length && prevIds.every((id, idx) => id === pinnedColumnIds[idx])) {
+        return prev;
+      }
+      return {
+        ...prev,
+        defaultPinnedColumns: pinnedColumnIds
+      };
+    });
+  }, [pinnedColumns]);
+  const isInitialMount = useRef5(true);
+  useEffect6(() => {
+    if (isInitialMount.current) {
+      isInitialMount.current = false;
+      return;
+    }
+    onSettingsChange?.(spreadsheetSettings);
+  }, [spreadsheetSettings, onSettingsChange]);
   const applyUndo = useCallback7(() => {
     const entry = popUndoEntry();
     if (!entry || !onCellsEdit) return;