chore(media): solve linter warnings

This commit is contained in:
Elian Doran 2026-03-11 20:38:53 +02:00
parent 7777f72893
commit 3edd8f6c5a
No known key found for this signature in database
3 changed files with 8 additions and 8 deletions

View File

@ -108,5 +108,5 @@ function useKeyboardShortcuts(audioRef: MutableRef<HTMLAudioElement | null>, wra
audio.currentTime = audio.duration;
break;
}
}, [togglePlayback]);
}, [ audioRef, togglePlayback ]);
}

View File

@ -25,7 +25,7 @@ export function SeekBar({ mediaRef }: { mediaRef: RefObject<HTMLVideoElement | H
media.removeEventListener("timeupdate", onTimeUpdate);
media.removeEventListener("durationchange", onDurationChange);
};
}, []);
}, [ mediaRef ]);
const onSeek = (e: Event) => {
const media = mediaRef.current;
@ -96,7 +96,7 @@ export function VolumeControl({ mediaRef }: { mediaRef: RefObject<HTMLVideoEleme
};
media.addEventListener("volumechange", onVolumeChange);
return () => media.removeEventListener("volumechange", onVolumeChange);
}, []);
}, [ mediaRef ]);
const onVolumeChange = (e: Event) => {
const media = mediaRef.current;
@ -160,7 +160,7 @@ export function LoopButton({ mediaRef }: { mediaRef: RefObject<HTMLVideoElement
const observer = new MutationObserver(() => setLoop(media.loop));
observer.observe(media, { attributes: true, attributeFilter: ["loop"] });
return () => observer.disconnect();
}, []);
}, [ mediaRef ]);
const toggle = () => {
const media = mediaRef.current;
@ -193,7 +193,7 @@ export function PlaybackSpeed({ mediaRef }: { mediaRef: RefObject<HTMLVideoEleme
const onRateChange = () => setSpeed(media.playbackRate);
media.addEventListener("ratechange", onRateChange);
return () => media.removeEventListener("ratechange", onRateChange);
}, []);
}, [ mediaRef ]);
const selectSpeed = (rate: number) => {
const media = mediaRef.current;

View File

@ -138,7 +138,7 @@ function useKeyboardShortcuts(videoRef: MutableRef<HTMLVideoElement | null>, wra
flashControls();
break;
}
}, [togglePlayback, flashControls]);
}, [ wrapperRef, videoRef, togglePlayback, flashControls ]);
}
function useAutoHideControls(videoRef: RefObject<HTMLVideoElement>, playing: boolean) {
@ -150,7 +150,7 @@ function useAutoHideControls(videoRef: RefObject<HTMLVideoElement>, playing: boo
if (videoRef.current && !videoRef.current.paused) {
hideTimerRef.current = setTimeout(() => setVisible(false), AUTO_HIDE_DELAY);
}
}, []);
}, [ videoRef]);
const onMouseMove = useCallback(() => {
setVisible(true);
@ -244,7 +244,7 @@ function PictureInPictureButton({ videoRef }: { videoRef: RefObject<HTMLVideoEle
video.removeEventListener("enterpictureinpicture", onEnter);
video.removeEventListener("leavepictureinpicture", onLeave);
};
}, [supported]);
}, [ videoRef, supported ]);
if (!supported) return null;