From 32af52ccdadb46b9c1e7b397071059745f4d8fdf Mon Sep 17 00:00:00 2001 From: Yiannis Christodoulou Date: Mon, 28 Jul 2025 02:19:00 +0300 Subject: [PATCH] chore: Disable the unload page warning --- .../client/src/hooks/useVideoChapters.tsx | 24 +------------------ .../client/src/hooks/useVideoTrimmer.tsx | 24 +------------------ 2 files changed, 2 insertions(+), 46 deletions(-) diff --git a/frontend-tools/chapters-editor/client/src/hooks/useVideoChapters.tsx b/frontend-tools/chapters-editor/client/src/hooks/useVideoChapters.tsx index 3ef81892..6a7cef01 100644 --- a/frontend-tools/chapters-editor/client/src/hooks/useVideoChapters.tsx +++ b/frontend-tools/chapters-editor/client/src/hooks/useVideoChapters.tsx @@ -78,28 +78,6 @@ const useVideoChapters = () => { } }, [history, historyPosition]); - // Set up page unload warning - useEffect(() => { - // Event handler for beforeunload - const handleBeforeUnload = (e: BeforeUnloadEvent) => { - if (hasUnsavedChanges) { - // Standard way of showing a confirmation dialog before leaving - const message = 'Your edits will get lost if you leave the page. Do you want to continue?'; - e.preventDefault(); - e.returnValue = message; // Chrome requires returnValue to be set - return message; // For other browsers - } - }; - - // Add event listener - window.addEventListener('beforeunload', handleBeforeUnload); - - // Clean up - return () => { - window.removeEventListener('beforeunload', handleBeforeUnload); - }; - }, [hasUnsavedChanges]); - // Initialize video event listeners useEffect(() => { const video = videoRef.current; @@ -737,7 +715,7 @@ const useVideoChapters = () => { if (typeof window !== 'undefined' && window.lastSeekedPosition > 0) { // Only seek if the position is significantly different if (Math.abs(video.currentTime - window.lastSeekedPosition) > 0.1) { - console.log('handlePlay: Using lastSeekedPosition', window.lastSeekedPosition); + logger.debug('handlePlay: Using lastSeekedPosition', window.lastSeekedPosition); video.currentTime = window.lastSeekedPosition; } } diff --git a/frontend-tools/video-editor/client/src/hooks/useVideoTrimmer.tsx b/frontend-tools/video-editor/client/src/hooks/useVideoTrimmer.tsx index 612b5586..38d49418 100644 --- a/frontend-tools/video-editor/client/src/hooks/useVideoTrimmer.tsx +++ b/frontend-tools/video-editor/client/src/hooks/useVideoTrimmer.tsx @@ -63,28 +63,6 @@ const useVideoTrimmer = () => { } }, [history, historyPosition]); - // Set up page unload warning - useEffect(() => { - // Event handler for beforeunload - const handleBeforeUnload = (e: BeforeUnloadEvent) => { - if (hasUnsavedChanges) { - // Standard way of showing a confirmation dialog before leaving - const message = 'Your edits will get lost if you leave the page. Do you want to continue?'; - e.preventDefault(); - e.returnValue = message; // Chrome requires returnValue to be set - return message; // For other browsers - } - }; - - // Add event listener - window.addEventListener('beforeunload', handleBeforeUnload); - - // Clean up - return () => { - window.removeEventListener('beforeunload', handleBeforeUnload); - }; - }, [hasUnsavedChanges]); - // Initialize video event listeners useEffect(() => { const video = videoRef.current; @@ -665,7 +643,7 @@ const useVideoTrimmer = () => { if (typeof window !== 'undefined' && window.lastSeekedPosition > 0) { // Only seek if the position is significantly different if (Math.abs(video.currentTime - window.lastSeekedPosition) > 0.1) { - console.log('handlePlay: Using lastSeekedPosition', window.lastSeekedPosition); + logger.debug('handlePlay: Using lastSeekedPosition', window.lastSeekedPosition); video.currentTime = window.lastSeekedPosition; } }