Delete empty cells

src/grid/actions/DeleteCells.ts

@@ -18,19 +18,32 @@ export const DeleteCells = async (args: DeleteCellsArgs) => {
 
   if (create_transaction ?? true) sheetController.start_transaction();
 
+  // cancel transaction if there are no cells to delete
+  let deletingCells = false;
+
   // delete cells row by row
   for (var current_row = y0; current_row <= y1; current_row++) {
     const cells_to_delete = sheetController.sheet.grid.getNakedCells(x0, current_row, x1, current_row);
+
     // delete cells
-    await updateCellAndDCells({
-      starting_cells: cells_to_delete,
-      sheetController,
-      pyodide,
-      delete_starting_cells: true,
-      app,
-      create_transaction: false,
-    });
+    if (cells_to_delete.length !== 0) {
+      deletingCells = true;
+      await updateCellAndDCells({
+        starting_cells: cells_to_delete,
+        sheetController,
+        pyodide,
+        delete_starting_cells: true,
+        app,
+        create_transaction: false,
+      });
+    }
   }
 
-  if (create_transaction ?? true) sheetController.end_transaction();
+  if (create_transaction ?? true) {
+    if (!deletingCells) {
+      sheetController.cancel_transaction();
+    } else {
+      sheetController.end_transaction();
+    }
+  }
 };

src/grid/controller/sheetController.ts

@@ -98,6 +98,14 @@ export class SheetController {
     return reverse_transaction;
   }
 
+  public cancel_transaction(): void {
+    if (!this.transaction_in_progress || !this.transaction_in_progress_reverse) {
+      throw new Error('No transaction in progress.');
+    }
+    this.transaction_in_progress = undefined;
+    this.transaction_in_progress_reverse = undefined;
+  }
+
   public predefined_transaction(statements: Statement[]): Transaction {
     // Starts a transaction, executes all statements, and ends the transaction.
     // Returns the transaction.