refactor: address GitHub Copilot review comments

- Remove unnecessary currentSegment from skipSegment dependency array
- Remove redundant wrappedSeek wrapper (ref guard prevents issues)
- Document 200ms setTimeout delay for seek operations
- Improve code clarity and reduce unnecessary re-renders
This commit is contained in:
Uruk
2026-01-14 14:07:14 +01:00
parent 5f2d183459
commit 27e1dce1ca
2 changed files with 6 additions and 11 deletions

View File

@@ -313,9 +313,12 @@ export const Controls: FC<Props> = ({
const maxSeconds = maxMs ? msToSeconds(maxMs) : undefined;
// Wrapper to convert segment skip from seconds to milliseconds
// Includes 200ms delay to allow seek operation to complete before resuming playback
const seekMs = useCallback(
(timeInSeconds: number) => {
seek(timeInSeconds * 1000);
// Brief delay ensures the seek operation completes before resuming playback
// Without this, playback may resume from the old position
setTimeout(() => {
play();
}, 200);