@rowsncolumns/spreadsheet-state 29.1.16 → 29.1.17

package/dist/cjs/hooks/use-on-request-create-table.d.ts

@@ -5,9 +5,9 @@ export type UseOnRequestCreateTableProps = {
 };
 /**
  * Open Excel's "Create Table" dialog (Ctrl+T / Insert → Table / Format as Table): the range
- * pre-fills with the selection, else the data region around the active cell (Ctrl+A's region),
- * and "My table has headers" is guessed from that range's first row. The dialog submits through
- * `onCreateTable`.
+ * pre-fills with the selection, else — with none, or one collapsed on a single cell — the data
+ * region around the active cell (Ctrl+A's region), and "My table has headers" is guessed from
+ * that range's first row. The dialog submits through `onCreateTable`.
  */
 export declare const useOnRequestCreateTable: ({ getEffectiveExtendedValue, }: UseOnRequestCreateTableProps) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[], theme?: TableTheme, bandedRange?: BandedDefinition) => void;
 //# sourceMappingURL=use-on-request-create-table.d.ts.map

package/dist/cjs/hooks/use-on-request-create-table.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-on-request-create-table.d.ts","sourceRoot":"","sources":["../../../hooks/use-on-request-create-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EACb,aAAa,EACb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,gBAAgB,EAChB,mBAAmB,EACnB,UAAU,EACX,MAAM,2BAA2B,CAAC;AASnC,MAAM,MAAM,4BAA4B,GAAG;IACzC,yBAAyB,EAAE,CACzB,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,KAChB,aAAa,GAAG,IAAI,GAAG,SAAS,CAAC;CACvC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,uBAAuB,GAAI,gCAErC,4BAA4B,eAMhB,MAAM,cACH,aAAa,cACb,aAAa,CAAC,mBAAmB,CAAC,EAAE,UACxC,UAAU,gBACJ,gBAAgB,SA6BnC,CAAC"}
+{"version":3,"file":"use-on-request-create-table.d.ts","sourceRoot":"","sources":["../../../hooks/use-on-request-create-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EACb,aAAa,EACb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,gBAAgB,EAChB,mBAAmB,EACnB,UAAU,EACX,MAAM,2BAA2B,CAAC;AASnC,MAAM,MAAM,4BAA4B,GAAG;IACzC,yBAAyB,EAAE,CACzB,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,KAChB,aAAa,GAAG,IAAI,GAAG,SAAS,CAAC;CACvC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,uBAAuB,GAAI,gCAErC,4BAA4B,eAMhB,MAAM,cACH,aAAa,cACb,aAAa,CAAC,mBAAmB,CAAC,EAAE,UACxC,UAAU,gBACJ,gBAAgB,SAkCnC,CAAC"}

package/dist/cjs/hooks/use-on-request-create-table.js

@@ -6,9 +6,9 @@ import { useSetCreatingTable } from "../atoms/creating-table.atom";
 import { useSetCreateTableDialogValue } from "../atoms/dialog-state.atom";
 /**
  * Open Excel's "Create Table" dialog (Ctrl+T / Insert → Table / Format as Table): the range
- * pre-fills with the selection, else the data region around the active cell (Ctrl+A's region),
- * and "My table has headers" is guessed from that range's first row. The dialog submits through
- * `onCreateTable`.
+ * pre-fills with the selection, else — with none, or one collapsed on a single cell — the data
+ * region around the active cell (Ctrl+A's region), and "My table has headers" is guessed from
+ * that range's first row. The dialog submits through `onCreateTable`.
  */
 export var useOnRequestCreateTable = function (_a) {
     var getEffectiveExtendedValue = _a.getEffectiveExtendedValue;
@@ -18,7 +18,11 @@ export var useOnRequestCreateTable = function (_a) {
     return useCallback(function (sheetId, activeCell, selections, theme, bandedRange) {
         var _a;
         var lastSelection = selections === null || selections === void 0 ? void 0 : selections[selections.length - 1];
-        var range = lastSelection
+        var collapsed = !lastSelection ||
+            (lastSelection.range.startRowIndex === lastSelection.range.endRowIndex &&
+                lastSelection.range.startColumnIndex ===
+                    lastSelection.range.endColumnIndex);
+        var range = !collapsed
             ? lastSelection.range
             : (_a = getContentfulRangeAroundCell === null || getContentfulRangeAroundCell === void 0 ? void 0 : getContentfulRangeAroundCell(sheetId, activeCell)) !== null && _a !== void 0 ? _a : selectionFromActiveCell(activeCell)[0].range;
         setCreatingTable({

package/dist/cjs/interface/delete-cells.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAK/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,gUAoB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SAgSpC,CAAC"}
+{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAM/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,gUAoB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SA+TpC,CAAC"}

package/dist/cjs/interface/delete-cells.js

@@ -5,6 +5,7 @@ import { produceWithPatches } from "immer";
 import { createSheetCellIdentifier } from "../api/formula";
 import { createSheetDraft } from "../api/immer-utils";
 import { SheetCell } from "../sheet-cell";
+import { releaseTableFilterRows } from "./remove-table";
 import { getTableHeaderRefills } from "./table-utils";
 export var deleteCells = function (_a) {
     var locale = _a.locale, rowCount = _a.rowCount, cellXfsRegistry = _a.cellXfsRegistry, sharedStringRegistry = _a.sharedStringRegistry, getDataValidationById = _a.getDataValidationById, getDependents = _a.getDependents, getDataRowCount = _a.getDataRowCount, onChangeSheetData = _a.onChangeSheetData, onChangeSheets = _a.onChangeSheets, onChangeTables = _a.onChangeTables, onChangeCitations = _a.onChangeCitations, onChangeActiveCell = _a.onChangeActiveCell, enqueueCalculation = _a.enqueueCalculation, enqueueStructureChange = _a.enqueueStructureChange, createHistory = _a.createHistory, isArrayFormula = _a.isArrayFormula, getRowHeight = _a.getRowHeight, tables = _a.tables, generatedNames = _a.generatedNames;
@@ -18,13 +19,28 @@ export var deleteCells = function (_a) {
             : selectionFromActiveCell(cell);
         var saveHistory = createHistory();
         var maxRowCount = getDataRowCount();
-        // A table survives Delete / Clear (Excel): only its cells empty out, and
-        // cleared header cells are re-named `ColumnN` so the header row and
-        // `columns[]` stay in step.
+        // Delete / Clear over a selection that covers a whole table deletes the table (Excel's
+        // "select the table and press Delete"): the table, its filter and its style go with the
+        // cells, and the structured references into it error as they do for any deleted table
+        // (Convert to Range is the path that rewrites them to A1).
+        var removedTables = shouldClearContents
+            ? (tables !== null && tables !== void 0 ? tables : []).filter(function (table) {
+                return table.sheetId === sheetId &&
+                    finalSelections.some(function (selection) {
+                        return areaInsideArea(table.range, selection.range);
+                    });
+            })
+            : [];
+        var keptTables = removedTables.length
+            ? tables === null || tables === void 0 ? void 0 : tables.filter(function (table) { return !removedTables.includes(table); })
+            : tables;
+        // A table survives Delete / Clear over part of it (Excel): only its cells
+        // empty out, and cleared header cells are re-named `ColumnN` so the header
+        // row and `columns[]` stay in step.
         var headerRefills = shouldClearContents
             ? getTableHeaderRefills({
                 sheetId: sheetId,
-                tables: tables,
+                tables: keptTables,
                 selections: finalSelections,
                 generatedNames: generatedNames,
             })
@@ -170,17 +186,36 @@ export var deleteCells = function (_a) {
             });
             return nextState;
         });
-        // Rename the columns whose header cells were cleared
-        if (headerRefills.length) {
+        // Drop the tables the selection covered, and rename the columns whose header cells were cleared
+        if (headerRefills.length || removedTables.length) {
             onChangeTables === null || onChangeTables === void 0 ? void 0 : onChangeTables(function (prevTables) {
                 var _a = __read(produceWithPatches(prevTables, function (draft) {
-                    var e_4, _a;
-                    var _b;
-                    var _loop_1 = function (tableId, dimensionIndex, name_2) {
+                    var e_4, _a, e_5, _b;
+                    var _c;
+                    var _loop_1 = function (removed) {
+                        var tableIndex = draft.findIndex(function (table) { return table.id == removed.id; });
+                        if (tableIndex !== -1) {
+                            draft.splice(tableIndex, 1);
+                        }
+                    };
+                    try {
+                        for (var removedTables_1 = __values(removedTables), removedTables_1_1 = removedTables_1.next(); !removedTables_1_1.done; removedTables_1_1 = removedTables_1.next()) {
+                            var removed = removedTables_1_1.value;
+                            _loop_1(removed);
+                        }
+                    }
+                    catch (e_4_1) { e_4 = { error: e_4_1 }; }
+                    finally {
+                        try {
+                            if (removedTables_1_1 && !removedTables_1_1.done && (_a = removedTables_1.return)) _a.call(removedTables_1);
+                        }
+                        finally { if (e_4) throw e_4.error; }
+                    }
+                    var _loop_2 = function (tableId, dimensionIndex, name_2) {
                         var table = draft.find(function (table) { return table.sheetId === sheetId && table.id === tableId; });
                         if (!table)
                             return "continue";
-                        (_b = table.columns) !== null && _b !== void 0 ? _b : (table.columns = []);
+                        (_c = table.columns) !== null && _c !== void 0 ? _c : (table.columns = []);
                         var column = table.columns[dimensionIndex];
                         if (column) {
                             column.name = name_2;
@@ -191,16 +226,16 @@ export var deleteCells = function (_a) {
                     };
                     try {
                         for (var headerRefills_2 = __values(headerRefills), headerRefills_2_1 = headerRefills_2.next(); !headerRefills_2_1.done; headerRefills_2_1 = headerRefills_2.next()) {
-                            var _c = headerRefills_2_1.value, tableId = _c.tableId, dimensionIndex = _c.dimensionIndex, name_2 = _c.name;
-                            _loop_1(tableId, dimensionIndex, name_2);
+                            var _d = headerRefills_2_1.value, tableId = _d.tableId, dimensionIndex = _d.dimensionIndex, name_2 = _d.name;
+                            _loop_2(tableId, dimensionIndex, name_2);
                         }
                     }
-                    catch (e_4_1) { e_4 = { error: e_4_1 }; }
+                    catch (e_5_1) { e_5 = { error: e_5_1 }; }
                     finally {
                         try {
-                            if (headerRefills_2_1 && !headerRefills_2_1.done && (_a = headerRefills_2.return)) _a.call(headerRefills_2);
+                            if (headerRefills_2_1 && !headerRefills_2_1.done && (_b = headerRefills_2.return)) _b.call(headerRefills_2);
                         }
-                        finally { if (e_4) throw e_4.error; }
+                        finally { if (e_5) throw e_5.error; }
                     }
                 }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
                 saveHistory({
@@ -208,6 +243,13 @@ export var deleteCells = function (_a) {
                 });
                 return nextState;
             });
+            releaseTableFilterRows({
+                tables: removedTables,
+                onChangeSheets: onChangeSheets,
+                saveHistory: saveHistory,
+                getDependents: getDependents,
+                enqueueCalculation: enqueueCalculation,
+            });
         }
         // Update citations - retire citations whose cited cells lose their contents.
         // Clearing formats only leaves the cited values in place, so the citations stay.
@@ -215,7 +257,7 @@ export var deleteCells = function (_a) {
             onChangeCitations === null || onChangeCitations === void 0 ? void 0 : onChangeCitations(function (prevCitations) {
                 var updatedAt = new Date().toISOString();
                 var _a = __read(produceWithPatches(prevCitations, function (draft) {
-                    var _loop_2 = function (idx) {
+                    var _loop_3 = function (idx) {
                         var citation = draft[idx];
                         // Check if citation is active, on the same sheet and inside a cleared selection
                         if (citation.active !== false &&
@@ -229,7 +271,7 @@ export var deleteCells = function (_a) {
                         }
                     };
                     for (var idx = draft.length - 1; idx >= 0; idx--) {
-                        _loop_2(idx);
+                        _loop_3(idx);
                     }
                 }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
                 saveHistory({
@@ -241,11 +283,11 @@ export var deleteCells = function (_a) {
         // Update banded cells
         onChangeSheets === null || onChangeSheets === void 0 ? void 0 : onChangeSheets(function (prevSheets) {
             var _a = __read(produceWithPatches(prevSheets, function (draft) {
-                var e_5, _a;
+                var e_6, _a;
                 var _b, _c, _d, _e, _f;
                 var sheet = draft.find(function (sheet) { return sheet.sheetId === sheetId; });
                 if (sheet) {
-                    var _loop_3 = function (selection) {
+                    var _loop_4 = function (selection) {
                         for (var rowIndex = selection.range.startRowIndex; rowIndex <=
                             Math.min(maxRowCount !== null && maxRowCount !== void 0 ? maxRowCount : rowCount, selection.range.endRowIndex); rowIndex++) {
                             var isRowResizedByUser = !!((_c = (_b = sheet === null || sheet === void 0 ? void 0 : sheet.rowMetadata) === null || _b === void 0 ? void 0 : _b[rowIndex]) === null || _c === void 0 ? void 0 : _c.resizedByUser);
@@ -270,15 +312,15 @@ export var deleteCells = function (_a) {
                         // Adjust row height
                         for (var finalSelections_3 = __values(finalSelections), finalSelections_3_1 = finalSelections_3.next(); !finalSelections_3_1.done; finalSelections_3_1 = finalSelections_3.next()) {
                             var selection = finalSelections_3_1.value;
-                            _loop_3(selection);
+                            _loop_4(selection);
                         }
                     }
-                    catch (e_5_1) { e_5 = { error: e_5_1 }; }
+                    catch (e_6_1) { e_6 = { error: e_6_1 }; }
                     finally {
                         try {
                             if (finalSelections_3_1 && !finalSelections_3_1.done && (_a = finalSelections_3.return)) _a.call(finalSelections_3);
                         }
-                        finally { if (e_5) throw e_5.error; }
+                        finally { if (e_6) throw e_6.error; }
                     }
                 }
             }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];

package/dist/cjs/interface/remove-table.d.ts

@@ -1,4 +1,15 @@
 import type { CellData, Sheet, TableView } from "@rowsncolumns/spreadsheet";
 import type { UseOnTableProps } from "../hooks/use-on-table";
 export declare const removeTable: <T extends CellData, S extends Sheet>({ locale, cellXfsRegistry, sharedStringRegistry, getSheetName, onChangeTables, onChangeSheets, onChangeSheetData, createHistory, getDependents, getGraphNodes, getBandedCells, enqueueCalculation, enqueueGraphOperation, }: Pick<UseOnTableProps<T, S>, "locale" | "cellXfsRegistry" | "sharedStringRegistry" | "getSheetName" | "onChangeTables" | "onChangeSheets" | "onChangeSheetData" | "getDependents" | "getGraphNodes" | "getBandedCells" | "enqueueCalculation" | "enqueueGraphOperation" | "createHistory">) => (tableView: TableView) => void;
+/**
+ * A removed table's filter goes with it: the rows it hid become visible again, in the history step
+ * of the removal (`saveHistory`). Their values did not change, so only the formulas that read
+ * visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over those rows — are stale; dirtying the
+ * un-hidden cells recalculates exactly those dependents. Shared by Convert to Range
+ * (`removeTable`) and Delete over a whole table (`deleteCells`).
+ */
+export declare const releaseTableFilterRows: <T extends CellData, S extends Sheet>({ tables, onChangeSheets, saveHistory, getDependents, enqueueCalculation, }: Pick<UseOnTableProps<T, S>, "onChangeSheets" | "getDependents" | "enqueueCalculation"> & {
+    tables: TableView[];
+    saveHistory: ReturnType<UseOnTableProps<T, S>["createHistory"]>;
+}) => void;
 //# sourceMappingURL=remove-table.d.ts.map

package/dist/cjs/interface/remove-table.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"remove-table.d.ts","sourceRoot":"","sources":["../../../interface/remove-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAK5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAI7D,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6NAc9D,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACnB,QAAQ,GACR,iBAAiB,GACjB,sBAAsB,GACtB,cAAc,GACd,gBAAgB,GAChB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,eAAe,GACf,gBAAgB,GAChB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,CAClB,MACS,WAAW,SAAS,SAwH7B,CAAC"}
+{"version":3,"file":"remove-table.d.ts","sourceRoot":"","sources":["../../../interface/remove-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAK5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAI7D,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6NAc9D,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACnB,QAAQ,GACR,iBAAiB,GACjB,sBAAsB,GACtB,cAAc,GACd,gBAAgB,GAChB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,eAAe,GACf,gBAAgB,GAChB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,CAClB,MACS,WAAW,SAAS,SA0E7B,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,sBAAsB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6EAMzE,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACrB,gBAAgB,GAAG,eAAe,GAAG,oBAAoB,CAC1D,GAAG;IACF,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,WAAW,EAAE,UAAU,CAAC,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC;CACjE,SAoDA,CAAC"}

package/dist/cjs/interface/remove-table.js

@@ -59,51 +59,82 @@ export var removeTable = function (_a) {
             });
             return nextState;
         });
-        // The table's filter goes with it: rows it hid become visible again. Their values did not
-        // change, so only the formulas that read visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over
-        // those rows — are stale; dirtying the un-hidden cells recalculates exactly those dependents
-        // (the rewritten structured references were re-entered above with `force`).
-        var unhiddenRecalcCells = new Map();
-        onChangeSheets(function (prevSheets) {
-            var _a = __read(produceWithPatches(prevSheets, function (draft) {
-                var e_1, _a;
-                var _b;
+        releaseTableFilterRows({
+            tables: [tableView],
+            onChangeSheets: onChangeSheets,
+            saveHistory: saveHistory,
+            getDependents: getDependents,
+            enqueueCalculation: enqueueCalculation,
+        });
+    };
+};
+/**
+ * A removed table's filter goes with it: the rows it hid become visible again, in the history step
+ * of the removal (`saveHistory`). Their values did not change, so only the formulas that read
+ * visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over those rows — are stale; dirtying the
+ * un-hidden cells recalculates exactly those dependents. Shared by Convert to Range
+ * (`removeTable`) and Delete over a whole table (`deleteCells`).
+ */
+export var releaseTableFilterRows = function (_a) {
+    var tables = _a.tables, onChangeSheets = _a.onChangeSheets, saveHistory = _a.saveHistory, getDependents = _a.getDependents, enqueueCalculation = _a.enqueueCalculation;
+    if (!tables.length)
+        return;
+    var unhiddenRecalcCells = new Map();
+    onChangeSheets(function (prevSheets) {
+        var _a = __read(produceWithPatches(prevSheets, function (draft) {
+            var e_1, _a;
+            var _b;
+            var _loop_1 = function (tableView) {
+                var e_2, _c;
                 var sheet = draft.find(function (sheet) { return sheet.sheetId === tableView.sheetId; });
-                if (sheet && tableView) {
-                    if (!sheet.rowMetadata) {
-                        sheet.rowMetadata = [];
-                    }
-                    var range = tableView.range;
-                    for (var rowIndex = range.startRowIndex; rowIndex <= range.endRowIndex; rowIndex++) {
-                        if ((_b = sheet.rowMetadata[rowIndex]) === null || _b === void 0 ? void 0 : _b.hiddenByFilter) {
-                            sheet.rowMetadata[rowIndex].hiddenByFilter = false;
-                            try {
-                                for (var _c = (e_1 = void 0, __values(markTableAsDirty(tableView.sheetId, __assign(__assign({}, range), { startRowIndex: rowIndex, endRowIndex: rowIndex }), getDependents, enqueueCalculation))), _d = _c.next(); !_d.done; _d = _c.next()) {
-                                    var _e = __read(_d.value, 2), key = _e[0], position = _e[1];
-                                    unhiddenRecalcCells.set(key, position);
-                                }
+                if (!sheet)
+                    return "continue";
+                if (!sheet.rowMetadata) {
+                    sheet.rowMetadata = [];
+                }
+                var range = tableView.range;
+                for (var rowIndex = range.startRowIndex; rowIndex <= range.endRowIndex; rowIndex++) {
+                    if ((_b = sheet.rowMetadata[rowIndex]) === null || _b === void 0 ? void 0 : _b.hiddenByFilter) {
+                        sheet.rowMetadata[rowIndex].hiddenByFilter = false;
+                        try {
+                            for (var _d = (e_2 = void 0, __values(markTableAsDirty(tableView.sheetId, __assign(__assign({}, range), { startRowIndex: rowIndex, endRowIndex: rowIndex }), getDependents, enqueueCalculation))), _e = _d.next(); !_e.done; _e = _d.next()) {
+                                var _f = __read(_e.value, 2), key = _f[0], position = _f[1];
+                                unhiddenRecalcCells.set(key, position);
                             }
-                            catch (e_1_1) { e_1 = { error: e_1_1 }; }
-                            finally {
-                                try {
-                                    if (_d && !_d.done && (_a = _c.return)) _a.call(_c);
-                                }
-                                finally { if (e_1) throw e_1.error; }
+                        }
+                        catch (e_2_1) { e_2 = { error: e_2_1 }; }
+                        finally {
+                            try {
+                                if (_e && !_e.done && (_c = _d.return)) _c.call(_d);
                             }
+                            finally { if (e_2) throw e_2.error; }
                         }
                     }
                 }
-            }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
-            // Save to history
-            saveHistory(__assign({ sheets: { patches: patches, inversePatches: inversePatches } }, (unhiddenRecalcCells.size
-                ? {
-                    recalcCells: {
-                        undo: unhiddenRecalcCells,
-                        redo: unhiddenRecalcCells,
-                    },
+            };
+            try {
+                for (var tables_1 = __values(tables), tables_1_1 = tables_1.next(); !tables_1_1.done; tables_1_1 = tables_1.next()) {
+                    var tableView = tables_1_1.value;
+                    _loop_1(tableView);
                 }
-                : {})));
-            return nextState;
-        });
-    };
+            }
+            catch (e_1_1) { e_1 = { error: e_1_1 }; }
+            finally {
+                try {
+                    if (tables_1_1 && !tables_1_1.done && (_a = tables_1.return)) _a.call(tables_1);
+                }
+                finally { if (e_1) throw e_1.error; }
+            }
+        }), 3), nextState = _a[0], patches = _a[1], inversePatches = _a[2];
+        // Save to history
+        saveHistory(__assign({ sheets: { patches: patches, inversePatches: inversePatches } }, (unhiddenRecalcCells.size
+            ? {
+                recalcCells: {
+                    undo: unhiddenRecalcCells,
+                    redo: unhiddenRecalcCells,
+                },
+            }
+            : {})));
+        return nextState;
+    });
 };

package/dist/esm/hooks/use-on-request-create-table.d.ts

@@ -5,9 +5,9 @@ export type UseOnRequestCreateTableProps = {
 };
 /**
  * Open Excel's "Create Table" dialog (Ctrl+T / Insert → Table / Format as Table): the range
- * pre-fills with the selection, else the data region around the active cell (Ctrl+A's region),
- * and "My table has headers" is guessed from that range's first row. The dialog submits through
- * `onCreateTable`.
+ * pre-fills with the selection, else — with none, or one collapsed on a single cell — the data
+ * region around the active cell (Ctrl+A's region), and "My table has headers" is guessed from
+ * that range's first row. The dialog submits through `onCreateTable`.
  */
 export declare const useOnRequestCreateTable: ({ getEffectiveExtendedValue, }: UseOnRequestCreateTableProps) => (sheetId: number, activeCell: CellInterface, selections: SelectionArea<SelectionAttributes>[], theme?: TableTheme, bandedRange?: BandedDefinition) => void;
 //# sourceMappingURL=use-on-request-create-table.d.ts.map

package/dist/esm/hooks/use-on-request-create-table.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-on-request-create-table.d.ts","sourceRoot":"","sources":["../../../hooks/use-on-request-create-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EACb,aAAa,EACb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,gBAAgB,EAChB,mBAAmB,EACnB,UAAU,EACX,MAAM,2BAA2B,CAAC;AASnC,MAAM,MAAM,4BAA4B,GAAG;IACzC,yBAAyB,EAAE,CACzB,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,KAChB,aAAa,GAAG,IAAI,GAAG,SAAS,CAAC;CACvC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,uBAAuB,GAAI,gCAErC,4BAA4B,eAMhB,MAAM,cACH,aAAa,cACb,aAAa,CAAC,mBAAmB,CAAC,EAAE,UACxC,UAAU,gBACJ,gBAAgB,SA6BnC,CAAC"}
+{"version":3,"file":"use-on-request-create-table.d.ts","sourceRoot":"","sources":["../../../hooks/use-on-request-create-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EACb,aAAa,EACb,aAAa,EACd,MAAM,4BAA4B,CAAC;AACpC,OAAO,KAAK,EACV,gBAAgB,EAChB,mBAAmB,EACnB,UAAU,EACX,MAAM,2BAA2B,CAAC;AASnC,MAAM,MAAM,4BAA4B,GAAG;IACzC,yBAAyB,EAAE,CACzB,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,KAChB,aAAa,GAAG,IAAI,GAAG,SAAS,CAAC;CACvC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,uBAAuB,GAAI,gCAErC,4BAA4B,eAMhB,MAAM,cACH,aAAa,cACb,aAAa,CAAC,mBAAmB,CAAC,EAAE,UACxC,UAAU,gBACJ,gBAAgB,SAkCnC,CAAC"}

package/dist/esm/hooks/use-on-request-create-table.js

@@ -6,9 +6,9 @@ import { useSetCreatingTable } from "../atoms/creating-table.atom";
 import { useSetCreateTableDialogValue } from "../atoms/dialog-state.atom";
 /**
  * Open Excel's "Create Table" dialog (Ctrl+T / Insert → Table / Format as Table): the range
- * pre-fills with the selection, else the data region around the active cell (Ctrl+A's region),
- * and "My table has headers" is guessed from that range's first row. The dialog submits through
- * `onCreateTable`.
+ * pre-fills with the selection, else — with none, or one collapsed on a single cell — the data
+ * region around the active cell (Ctrl+A's region), and "My table has headers" is guessed from
+ * that range's first row. The dialog submits through `onCreateTable`.
  */
 export const useOnRequestCreateTable = ({ getEffectiveExtendedValue, }) => {
     const setCreateDialog = useSetCreateTableDialogValue();
@@ -16,7 +16,11 @@ export const useOnRequestCreateTable = ({ getEffectiveExtendedValue, }) => {
     const getContentfulRangeAroundCell = useGetContentfulRangeAroundCell();
     return useCallback((sheetId, activeCell, selections, theme, bandedRange) => {
         const lastSelection = selections?.[selections.length - 1];
-        const range = lastSelection
+        const collapsed = !lastSelection ||
+            (lastSelection.range.startRowIndex === lastSelection.range.endRowIndex &&
+                lastSelection.range.startColumnIndex ===
+                    lastSelection.range.endColumnIndex);
+        const range = !collapsed
             ? lastSelection.range
             : getContentfulRangeAroundCell?.(sheetId, activeCell) ??
                 selectionFromActiveCell(activeCell)[0].range;

package/dist/esm/interface/delete-cells.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAK/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,gUAoB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SAgSpC,CAAC"}
+{"version":3,"file":"delete-cells.d.ts","sourceRoot":"","sources":["../../../interface/delete-cells.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,aAAa,EAEb,aAAa,EACd,MAAM,4BAA4B,CAAC;AAGpC,OAAO,KAAK,EACV,QAAQ,EAER,mBAAmB,EACnB,KAAK,EACN,MAAM,2BAA2B,CAAC;AAKnC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAM/D,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,UAAU,GAAG,SAAS,CAAC;AAEvD,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,gUAoB9D,gBAAgB,CAAC,CAAC,EAAE,CAAC,CAAC,MAErB,SAAS,MAAM,EACf,MAAM,aAAa,EACnB,YAAY,aAAa,CAAC,mBAAmB,CAAC,EAAE,EAChD,YAAW,SAAsB,SA+TpC,CAAC"}

package/dist/esm/interface/delete-cells.js

@@ -4,6 +4,7 @@ import { produceWithPatches } from "immer";
 import { createSheetCellIdentifier } from "../api/formula";
 import { createSheetDraft } from "../api/immer-utils";
 import { SheetCell } from "../sheet-cell";
+import { releaseTableFilterRows } from "./remove-table";
 import { getTableHeaderRefills } from "./table-utils";
 export const deleteCells = ({ locale, rowCount, cellXfsRegistry, sharedStringRegistry, getDataValidationById, getDependents, getDataRowCount, onChangeSheetData, onChangeSheets, onChangeTables, onChangeCitations, onChangeActiveCell, enqueueCalculation, enqueueStructureChange, createHistory, isArrayFormula, getRowHeight, tables, generatedNames, }) => {
     return (sheetId, cell, selections, clearType = "contents") => {
@@ -14,13 +15,24 @@ export const deleteCells = ({ locale, rowCount, cellXfsRegistry, sharedStringReg
             : selectionFromActiveCell(cell);
         const saveHistory = createHistory();
         const maxRowCount = getDataRowCount();
-        // A table survives Delete / Clear (Excel): only its cells empty out, and
-        // cleared header cells are re-named `ColumnN` so the header row and
-        // `columns[]` stay in step.
+        // Delete / Clear over a selection that covers a whole table deletes the table (Excel's
+        // "select the table and press Delete"): the table, its filter and its style go with the
+        // cells, and the structured references into it error as they do for any deleted table
+        // (Convert to Range is the path that rewrites them to A1).
+        const removedTables = shouldClearContents
+            ? (tables ?? []).filter((table) => table.sheetId === sheetId &&
+                finalSelections.some((selection) => areaInsideArea(table.range, selection.range)))
+            : [];
+        const keptTables = removedTables.length
+            ? tables?.filter((table) => !removedTables.includes(table))
+            : tables;
+        // A table survives Delete / Clear over part of it (Excel): only its cells
+        // empty out, and cleared header cells are re-named `ColumnN` so the header
+        // row and `columns[]` stay in step.
         const headerRefills = shouldClearContents
             ? getTableHeaderRefills({
                 sheetId,
-                tables,
+                tables: keptTables,
                 selections: finalSelections,
                 generatedNames,
             })
@@ -133,10 +145,16 @@ export const deleteCells = ({ locale, rowCount, cellXfsRegistry, sharedStringReg
             });
             return nextState;
         });
-        // Rename the columns whose header cells were cleared
-        if (headerRefills.length) {
+        // Drop the tables the selection covered, and rename the columns whose header cells were cleared
+        if (headerRefills.length || removedTables.length) {
             onChangeTables?.((prevTables) => {
                 const [nextState, patches, inversePatches] = produceWithPatches(prevTables, (draft) => {
+                    for (const removed of removedTables) {
+                        const tableIndex = draft.findIndex((table) => table.id == removed.id);
+                        if (tableIndex !== -1) {
+                            draft.splice(tableIndex, 1);
+                        }
+                    }
                     for (const { tableId, dimensionIndex, name } of headerRefills) {
                         const table = draft.find((table) => table.sheetId === sheetId && table.id === tableId);
                         if (!table)
@@ -156,6 +174,13 @@ export const deleteCells = ({ locale, rowCount, cellXfsRegistry, sharedStringReg
                 });
                 return nextState;
             });
+            releaseTableFilterRows({
+                tables: removedTables,
+                onChangeSheets,
+                saveHistory,
+                getDependents,
+                enqueueCalculation,
+            });
         }
         // Update citations - retire citations whose cited cells lose their contents.
         // Clearing formats only leaves the cited values in place, so the citations stay.

package/dist/esm/interface/remove-table.d.ts

@@ -1,4 +1,15 @@
 import type { CellData, Sheet, TableView } from "@rowsncolumns/spreadsheet";
 import type { UseOnTableProps } from "../hooks/use-on-table";
 export declare const removeTable: <T extends CellData, S extends Sheet>({ locale, cellXfsRegistry, sharedStringRegistry, getSheetName, onChangeTables, onChangeSheets, onChangeSheetData, createHistory, getDependents, getGraphNodes, getBandedCells, enqueueCalculation, enqueueGraphOperation, }: Pick<UseOnTableProps<T, S>, "locale" | "cellXfsRegistry" | "sharedStringRegistry" | "getSheetName" | "onChangeTables" | "onChangeSheets" | "onChangeSheetData" | "getDependents" | "getGraphNodes" | "getBandedCells" | "enqueueCalculation" | "enqueueGraphOperation" | "createHistory">) => (tableView: TableView) => void;
+/**
+ * A removed table's filter goes with it: the rows it hid become visible again, in the history step
+ * of the removal (`saveHistory`). Their values did not change, so only the formulas that read
+ * visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over those rows — are stale; dirtying the
+ * un-hidden cells recalculates exactly those dependents. Shared by Convert to Range
+ * (`removeTable`) and Delete over a whole table (`deleteCells`).
+ */
+export declare const releaseTableFilterRows: <T extends CellData, S extends Sheet>({ tables, onChangeSheets, saveHistory, getDependents, enqueueCalculation, }: Pick<UseOnTableProps<T, S>, "onChangeSheets" | "getDependents" | "enqueueCalculation"> & {
+    tables: TableView[];
+    saveHistory: ReturnType<UseOnTableProps<T, S>["createHistory"]>;
+}) => void;
 //# sourceMappingURL=remove-table.d.ts.map

package/dist/esm/interface/remove-table.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"remove-table.d.ts","sourceRoot":"","sources":["../../../interface/remove-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAK5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAI7D,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6NAc9D,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACnB,QAAQ,GACR,iBAAiB,GACjB,sBAAsB,GACtB,cAAc,GACd,gBAAgB,GAChB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,eAAe,GACf,gBAAgB,GAChB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,CAClB,MACS,WAAW,SAAS,SAwH7B,CAAC"}
+{"version":3,"file":"remove-table.d.ts","sourceRoot":"","sources":["../../../interface/remove-table.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAK5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAI7D,eAAO,MAAM,WAAW,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6NAc9D,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACnB,QAAQ,GACR,iBAAiB,GACjB,sBAAsB,GACtB,cAAc,GACd,gBAAgB,GAChB,gBAAgB,GAChB,mBAAmB,GACnB,eAAe,GACf,eAAe,GACf,gBAAgB,GAChB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,CAClB,MACS,WAAW,SAAS,SA0E7B,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,sBAAsB,GAAI,CAAC,SAAS,QAAQ,EAAE,CAAC,SAAS,KAAK,EAAE,6EAMzE,IAAI,CACL,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,EACrB,gBAAgB,GAAG,eAAe,GAAG,oBAAoB,CAC1D,GAAG;IACF,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,WAAW,EAAE,UAAU,CAAC,eAAe,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC;CACjE,SAoDA,CAAC"}

package/dist/esm/interface/remove-table.js

@@ -57,42 +57,58 @@ export const removeTable = ({ locale, cellXfsRegistry, sharedStringRegistry, get
             });
             return nextState;
         });
-        // The table's filter goes with it: rows it hid become visible again. Their values did not
-        // change, so only the formulas that read visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over
-        // those rows — are stale; dirtying the un-hidden cells recalculates exactly those dependents
-        // (the rewritten structured references were re-entered above with `force`).
-        const unhiddenRecalcCells = new Map();
-        onChangeSheets((prevSheets) => {
-            const [nextState, patches, inversePatches] = produceWithPatches(prevSheets, (draft) => {
+        releaseTableFilterRows({
+            tables: [tableView],
+            onChangeSheets,
+            saveHistory,
+            getDependents,
+            enqueueCalculation,
+        });
+    };
+};
+/**
+ * A removed table's filter goes with it: the rows it hid become visible again, in the history step
+ * of the removal (`saveHistory`). Their values did not change, so only the formulas that read
+ * visibility — `SUBTOTAL(101…111)` / `AGGREGATE` over those rows — are stale; dirtying the
+ * un-hidden cells recalculates exactly those dependents. Shared by Convert to Range
+ * (`removeTable`) and Delete over a whole table (`deleteCells`).
+ */
+export const releaseTableFilterRows = ({ tables, onChangeSheets, saveHistory, getDependents, enqueueCalculation, }) => {
+    if (!tables.length)
+        return;
+    const unhiddenRecalcCells = new Map();
+    onChangeSheets((prevSheets) => {
+        const [nextState, patches, inversePatches] = produceWithPatches(prevSheets, (draft) => {
+            for (const tableView of tables) {
                 const sheet = draft.find((sheet) => sheet.sheetId === tableView.sheetId);
-                if (sheet && tableView) {
-                    if (!sheet.rowMetadata) {
-                        sheet.rowMetadata = [];
-                    }
-                    const { range } = tableView;
-                    for (let rowIndex = range.startRowIndex; rowIndex <= range.endRowIndex; rowIndex++) {
-                        if (sheet.rowMetadata[rowIndex]?.hiddenByFilter) {
-                            sheet.rowMetadata[rowIndex].hiddenByFilter = false;
-                            for (const [key, position] of markTableAsDirty(tableView.sheetId, { ...range, startRowIndex: rowIndex, endRowIndex: rowIndex }, getDependents, enqueueCalculation)) {
-                                unhiddenRecalcCells.set(key, position);
-                            }
+                if (!sheet)
+                    continue;
+                if (!sheet.rowMetadata) {
+                    sheet.rowMetadata = [];
+                }
+                const { range } = tableView;
+                for (let rowIndex = range.startRowIndex; rowIndex <= range.endRowIndex; rowIndex++) {
+                    if (sheet.rowMetadata[rowIndex]?.hiddenByFilter) {
+                        sheet.rowMetadata[rowIndex].hiddenByFilter = false;
+                        for (const [key, position] of markTableAsDirty(tableView.sheetId, { ...range, startRowIndex: rowIndex, endRowIndex: rowIndex }, getDependents, enqueueCalculation)) {
+                            unhiddenRecalcCells.set(key, position);
                         }
                     }
                 }
-            });
-            // Save to history
-            saveHistory({
-                sheets: { patches, inversePatches },
-                ...(unhiddenRecalcCells.size
-                    ? {
-                        recalcCells: {
-                            undo: unhiddenRecalcCells,
-                            redo: unhiddenRecalcCells,
-                        },
-                    }
-                    : {}),
-            });
-            return nextState;
+            }
         });
-    };
+        // Save to history
+        saveHistory({
+            sheets: { patches, inversePatches },
+            ...(unhiddenRecalcCells.size
+                ? {
+                    recalcCells: {
+                        undo: unhiddenRecalcCells,
+                        redo: unhiddenRecalcCells,
+                    },
+                }
+                : {}),
+        });
+        return nextState;
+    });
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rowsncolumns/spreadsheet-state",
-  "version": "29.1.16",
+  "version": "29.1.17",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",
   "types": "./dist/esm/index.d.ts",