Browse Source

remove unnecesary eslint ignore

pull/250/head
mrjvs 3 years ago
parent
commit
7a6af6c072
  1. 14
      src/video/components/actions/CaptionRendererAction.tsx

14
src/video/components/actions/CaptionRendererAction.tsx

@ -53,6 +53,11 @@ export function CaptionRendererAction({
const { captionSettings, setCaptionDelay } = useSettings(); const { captionSettings, setCaptionDelay } = useSettings();
const captions = useRef<ContentCaption[]>([]); const captions = useRef<ContentCaption[]>([]);
const captionSetRef = useRef<(delay: number) => void>(setCaptionDelay);
useEffect(() => {
captionSetRef.current = setCaptionDelay;
}, [setCaptionDelay]);
useAsync(async () => { useAsync(async () => {
const blobUrl = source?.caption?.url; const blobUrl = source?.caption?.url;
if (blobUrl) { if (blobUrl) {
@ -69,11 +74,12 @@ export function CaptionRendererAction({
captions.current = []; captions.current = [];
} }
}, [source?.caption?.url]); }, [source?.caption?.url]);
// reset delay when loading new source url
useEffect(() => { useEffect(() => {
// reset delay after video ends captionSetRef.current(0);
return () => setCaptionDelay(0); }, [source?.caption?.url]);
// eslint-disable-next-line react-hooks/exhaustive-deps
}, []);
const isVisible = useCallback( const isVisible = useCallback(
( (
start: number, start: number,

Loading…
Cancel
Save