cpvalente/ontime
1
0
Fork 0
mirror of https://github.com/cpvalente/ontime.git synced 2026-09-01 04:19:12 +00:00
Code Issues Packages Projects Releases 10 Wiki Activity

refactor(hooks): drop unused parameters from follow callback

Browse Source 
Both consumers of useFollowComponent call the returned callback with no
arguments, so the ref and offset overrides were dead. Removing them also
drops the guard on refs that can never be falsy.
This commit is contained in:
Claude
2026-08-30 10:52:33 +00:00
parent 417773e7a9
commit ac927a97cf
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/client/src/common/hooks/useFollowComponent.ts
+3 -8
View File
@@ -57,14 +57,9 @@ export default function useFollowComponent({
}
}, [followTrigger, doFollow, followRef, scrollRef, setScrollFlag, topOffset]);
const scrollToRefComponent = useCallback(
(componentRef = followRef, containerRef = scrollRef, offset?: number) => {
if (componentRef && containerRef) {
scrollToComponent(componentRef, containerRef, offset ?? resolveTopOffset(topOffset));
}
},
[followRef, scrollRef, topOffset],
);
const scrollToRefComponent = useCallback(() => {
scrollToComponent(followRef, scrollRef, resolveTopOffset(topOffset));
}, [followRef, scrollRef, topOffset]);
return scrollToRefComponent;
}