gaschz/trilium
1
0
Fork 0
mirror of https://github.com/zadam/trilium.git synced 2025-10-21 07:38:53 +02:00
Code Issues Actions 17 Packages Projects Releases Wiki Activity

fix(react/bulk_actions): spaced update triggering too fast

Browse Source
This commit is contained in:
Elian Doran 2025-08-09 09:15:54 +03:00
parent 6e1951b356
commit acf204d0e3
No known key found for this signature in database
1 changed files with 22 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
apps/client/src/widgets/react/hooks.tsx
View File

@ -1,4 +1,4 @@
import { useContext, useEffect, useMemo, useState } from "preact/hooks";
import { useCallback, useContext, useEffect, useMemo, useRef, useState } from "preact/hooks";
import { EventData, EventNames } from "../../components/app_context";
import { ParentComponent } from "./ReactBasicWidget";
import SpacedUpdate from "../../services/spaced_update";
@ -33,9 +33,26 @@ export default function useTriliumEvent<T extends EventNames>(eventName: T, hand
}
export function useSpacedUpdate(callback: () => Promise<void>, interval = 1000) {
const spacedUpdate = useMemo(() => {
return new SpacedUpdate(callback, interval);
}, [callback, interval]);
const callbackRef = useRef(callback);
const spacedUpdateRef = useRef<SpacedUpdate>();
return spacedUpdate;
// Update callback ref when it changes
useEffect(() => {
callbackRef.current = callback;
});
// Create SpacedUpdate instance only once
if (!spacedUpdateRef.current) {
spacedUpdateRef.current = new SpacedUpdate(
() => callbackRef.current(),
interval
);
}
// Update interval if it changes
useEffect(() => {
spacedUpdateRef.current?.setUpdateInterval(interval);
}, [interval]);
return spacedUpdateRef.current;
}