user more portable type for timeout and clear timeout in finish_change (AI suggestions)

cameront · cameront · commit 3f50fc272b03 · 2025-06-17T09:36:58.000-07:00
diff --git a/src/lib/utils/Popper.svelte b/src/lib/utils/Popper.svelte
@@ -189,7 +189,7 @@
     interactive: boolean;
   }
   let context: ChangeContext | undefined = $state(undefined);
-  let timeout: NodeJS.Timeout | undefined = $state(undefined);
+  let timeout: ReturnType<typeof setTimeout> | undefined = $state(undefined);
 
   // start_change debounces calls that attempt to open or close the popover.
   // callers must specify whether this request is on behalf of ui interactivity
@@ -220,6 +220,7 @@
     if (!context) return;
     isOpen = context.nextOpen;
     context = undefined;
+    timeout = undefined;
   }
 </script>
 

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,7 @@`
`189`	`189`	`interactive: boolean;`
`190`	`190`	`}`
`191`	`191`	`let context: ChangeContext \| undefined = $state(undefined);`
`192`		`- let timeout: NodeJS.Timeout \| undefined = $state(undefined);`
	`192`	`+ let timeout: ReturnType<typeof setTimeout> \| undefined = $state(undefined);`
`193`	`193`
`194`	`194`	`// start_change debounces calls that attempt to open or close the popover.`
`195`	`195`	`// callers must specify whether this request is on behalf of ui interactivity`
`@@ -220,6 +220,7 @@`
`220`	`220`	`if (!context) return;`
`221`	`221`	`isOpen = context.nextOpen;`
`222`	`222`	`context = undefined;`
	`223`	`+ timeout = undefined;`
`223`	`224`	`}`
`224`	`225`	`</script>`
`225`	`226`