From ae54ab41a8549730ccccbd7c9a2301de10a9efad Mon Sep 17 00:00:00 2001 From: CodeDevMLH <145071728+CodeDevMLH@users.noreply.github.com> Date: Fri, 13 Feb 2026 03:40:23 +0100 Subject: [PATCH] Refactor video playback logic to simplify conditions for active slides --- .../Web/mediaBarEnhanced.js | 26 +++++++++++-------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/Jellyfin.Plugin.MediaBarEnhanced/Web/mediaBarEnhanced.js b/Jellyfin.Plugin.MediaBarEnhanced/Web/mediaBarEnhanced.js index 79ae254..19519fc 100644 --- a/Jellyfin.Plugin.MediaBarEnhanced/Web/mediaBarEnhanced.js +++ b/Jellyfin.Plugin.MediaBarEnhanced/Web/mediaBarEnhanced.js @@ -1743,14 +1743,15 @@ const SlideCreator = { const isVideoPlayerOpen = document.querySelector('.videoPlayerContainer') || document.querySelector('.youtubePlayerContainer'); if (slide && slide.classList.contains('active') && !document.hidden && (!isVideoPlayerOpen || isVideoPlayerOpen.classList.contains('hide'))) { - const currentIndex = STATE.slideshow.currentSlideIndex; - const currentItemId = STATE.slideshow.itemIds[currentIndex]; - if (currentItemId !== itemId) { - console.log(`Slide ${itemId} is no longer active (current: ${currentItemId}), aborting playback.`); - event.target.mute(); // Mute just in case - return; - } - event.target.playVideo(); + // MARK: mark + // const currentIndex = STATE.slideshow.currentSlideIndex; + // const currentItemId = STATE.slideshow.itemIds[currentIndex]; + // if (currentItemId !== itemId) { + // console.log(`Slide ${itemId} is no longer active (current: ${currentItemId}), aborting playback.`); + // event.target.mute(); // Mute just in case + // return; + // } + event.target.playVideo(); // Check if it actually started playing after a short delay (handling autoplay blocks) const timeoutId = setTimeout(() => { @@ -1826,10 +1827,13 @@ const SlideCreator = { backdrop.addEventListener('play', (event) => { const slide = document.querySelector(`.slide[data-item-id="${itemId}"]`); - const currentIndex = STATE.slideshow.currentSlideIndex; - const currentItemId = STATE.slideshow.itemIds[currentIndex]; + // MARK: mark + // const currentIndex = STATE.slideshow.currentSlideIndex; + // const currentItemId = STATE.slideshow.itemIds[currentIndex]; - if (!slide || !slide.classList.contains('active') || currentItemId !== itemId) { + // if (!slide || !slide.classList.contains('active') || currentItemId !== itemId) { + + if (!slide || !slide.classList.contains('active')) { console.log(`Local video ${itemId} started playing but is not active, pausing.`); event.target.pause(); event.target.currentTime = 0;