From 7f0b37cda5f068cf0a51b5ddcaa27388bf4950d6 Mon Sep 17 00:00:00 2001 From: kuekhaoyang Date: Mon, 29 Dec 2025 10:50:15 +0800 Subject: [PATCH] feat: Add `onSpeedMenuMouseEnter` prop to `DesktopOverlayWrapper` and expose `isOutroActive` from `useAutoSkip`. --- components/player/desktop/DesktopOverlayWrapper.tsx | 4 +--- components/player/hooks/useAutoSkip.ts | 1 + 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/components/player/desktop/DesktopOverlayWrapper.tsx b/components/player/desktop/DesktopOverlayWrapper.tsx index 2af0dfb..f41aec7 100644 --- a/components/player/desktop/DesktopOverlayWrapper.tsx +++ b/components/player/desktop/DesktopOverlayWrapper.tsx @@ -19,9 +19,9 @@ interface DesktopOverlayWrapperProps { playbackRate: number; showSpeedMenu: boolean; speeds: number[]; - onToggleMoreMenu?: () => void; // Unused but kept for type safety if needed onToggleSpeedMenu: () => void; onSpeedChange: (speed: number) => void; + onSpeedMenuMouseEnter: () => void; onSpeedMenuMouseLeave: () => void; containerRef: React.RefObject; } @@ -36,8 +36,6 @@ export function DesktopOverlayWrapper({ showMoreMenu, isProxied, onToggleMoreMenu, - onMouseEnter, // Note: The prop name was actually missing in destructuring or renamed? Let me check previous view_file. - // Wait, the previous view_file of DesktopOverlayWrapper had these: onMoreMenuMouseEnter, onMoreMenuMouseLeave, onCopyLink, diff --git a/components/player/hooks/useAutoSkip.ts b/components/player/hooks/useAutoSkip.ts index a358977..eb9e184 100644 --- a/components/player/hooks/useAutoSkip.ts +++ b/components/player/hooks/useAutoSkip.ts @@ -202,5 +202,6 @@ export function useAutoSkip({ return { hasSkippedIntro: hasSkippedIntroRef.current, hasTriggeredOutroSkip: hasTriggeredOutroSkipRef.current, + isOutroActive, }; }