Jump to new sheet and fix delete sheet bug

Author: jstrieb

TODO.md

@@ -156,4 +156,3 @@
     vertical scrolling
   - Accidentally typing "RC" while typing another formula can cause everything
     to hang
-  - Deleting a sheet with a selection throws an exception

src/Tabs.svelte

@@ -86,8 +86,6 @@
 <script>
   import ContextMenu from "./ContextMenu.svelte";
 
-  import { Sheet } from "./classes.svelte.js";
-
   let { globals = $bindable(), value = $bindable() } = $props();
   let tabs = $derived(globals.sheets.map((s) => s.name));
 

src/classes.svelte.js

@@ -397,10 +397,13 @@ functions.crypto = async (ticker) => {
   addSheet(name, rows, cols, formula, initial, start = this.sheets.length) {
     const sheet = new Sheet(name, rows, cols, formula, initial);
     sheet.globals = this;
+    this.deselect();
     this.sheets.splice(start, 0, sheet);
+    this.currentSheetIndex = start;
   }
 
   deleteSheets(n, start = this.sheets.length - n) {
+    this.deselect();
     return this.sheets.splice(start, n);
   }
 }

src/llm.svelte.js

@@ -5,7 +5,6 @@ llmToolFunctions.newSheet = function (name, cells) {
   this.globals?.addSheet(name, cells.length, cols, (i, j) =>
     cells[i]?.[j]?.toString(),
   );
-  this.globals.currentSheetIndex = this.globals.sheets.length - 1;
 };
 llmToolFunctions.newSheet.description =
   "takes a 2D array of strings containing formulas";