fix: don't override sort & sticky on data change

Author: Amelia Wattenberger

src/components/grid.tsx

@@ -163,13 +163,13 @@ export function Grid(props: GridProps) {
   React.useEffect(() => {
     if (props.defaultSort)
       handleSortChange(props.defaultSort[0], props.defaultSort[1]);
-  }, [props.defaultSort?.join(','), props.data]);
+  }, [props.defaultSort?.join(',')]);
 
   React.useEffect(updateColumnNames, [props.data, stickyColumnName]);
   React.useEffect(() => {
     if (props.defaultStickyColumnName)
       handleStickyColumnNameChange(props.defaultStickyColumnName);
-  }, [props.defaultStickyColumnName, props.data]);
+  }, [props.defaultStickyColumnName]);
 
   React.useEffect(() => {
     handleIsEditableChange(!!props.isEditable);

src/store.ts

@@ -120,17 +120,26 @@ export const createGridStore = () =>
           const columnNames = data.length
             ? Object.keys(data[0]).filter((d) => !utilKeys.includes(d))
             : [];
-          draft.stickyColumnName = columnNames[0];
-          draft.sort = columnNames[0]
-            ? [
-                columnNames[0],
-                // @ts-ignore
-                cellTypeMap[draft.cellTypes[columnNames[0]]]?.sortValueType ===
-                'string'
-                  ? 'asc'
-                  : 'desc',
-              ]
-            : [];
+
+          if (
+            !draft.stickyColumnName ||
+            !columnNames.includes(draft.stickyColumnName)
+          ) {
+            draft.stickyColumnName = columnNames[0];
+          }
+
+          if (!draft.sort.length) {
+            draft.sort = draft.stickyColumnName
+              ? [
+                  draft.stickyColumnName,
+                  // @ts-ignore
+                  cellTypeMap[draft.cellTypes[draft.stickyColumnName]]
+                    ?.sortValueType === 'string'
+                    ? 'asc'
+                    : 'desc',
+                ]
+              : [];
+          }
         }),
       handleMetadataChange: (metadata) =>
         set((draft) => {