options.requestAnimationFrame solves it

preactjs · Oct 5, 2024 · 29cb98d · 29cb98d
1 parent 2bc881b
commit 29cb98d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/preact/src/index.ts b/packages/preact/src/index.ts
@@ -381,7 +381,7 @@ function flushEffects() {
 
 function notifyEffects(this: Effect) {
 	if (effectsQueue.push(this) === 1) {
-		(options.debounceRendering || defer)(flushEffects);
+		(options.requestAnimationFrame || defer)(flushEffects);
 	}
 }
 
@@ -394,7 +394,7 @@ function flushComputeds() {
 
 function notifyComputeds(this: Effect) {
 	if (computedsQueue.push(this) === 1) {
-		(options.debounceRendering || defer)(flushComputeds);
+		(options.requestAnimationFrame || defer)(flushComputeds);
 	}
 }
 

diff --git a/packages/preact/test/index.test.tsx b/packages/preact/test/index.test.tsx
@@ -23,7 +23,7 @@ const afterFrame = () => {
 	});
 };
 
-describe.only("@preact/signals", () => {
+describe("@preact/signals", () => {
 	let scratch: HTMLDivElement;
 	let rerender: () => void;
 
@@ -756,7 +756,6 @@ describe.only("@preact/signals", () => {
 				render(<App />, scratch);
 			});
 
-			await afterFrame();
 			expect(cleanup).not.to.have.been.called;
 			expect(spy).to.have.been.calledOnceWith(
 				"foo",