added loading indicator

frontend/js/favorites.js

@@ -111,6 +111,9 @@ App.favorites = App.favorites || {};
                     <button class="video-menu-item" type="button" data-action="download" role="menuitem">Download</button>
                 </div>
                 <img src="${item.thumb}" alt="${item.title}">
+                <div class="video-loading" aria-hidden="true">
+                    <div class="video-loading-spinner"></div>
+                </div>
                 <div class="favorite-info">
                     <h4>${item.title}</h4>
                     ${uploaderText ? `<p><button class="uploader-link" type="button" data-uploader="${uploaderText}">${uploaderText}</button></p>` : ''}
@@ -120,7 +123,11 @@ App.favorites = App.favorites || {};
             if (App.videos && typeof App.videos.attachNoReferrerRetry === 'function') {
                 App.videos.attachNoReferrerRetry(thumb);
             }
-            card.onclick = () => App.player.open(item.meta || item);
+            card.onclick = () => {
+                if (card.classList.contains('is-loading')) return;
+                card.classList.add('is-loading');
+                App.player.open(item.meta || item, { originEl: card });
+            };
             const favoriteBtn = card.querySelector('.favorite-btn');
             if (favoriteBtn) {
                 favoriteBtn.onclick = (event) => {

frontend/js/player.js

@@ -29,10 +29,22 @@ App.player = App.player || {};
         return host;
     }
 
-    App.player.open = async function(source) {
+    App.player.open = async function(source, opts) {
         const modal = document.getElementById('video-modal');
         const video = document.getElementById('player');
-        if (!modal || !video) return;
+        const originEl = opts && opts.originEl ? opts.originEl : null;
+        const clearLoading = () => {
+            if (originEl) {
+                originEl.classList.remove('is-loading');
+            }
+        };
+        if (originEl) {
+            originEl.classList.add('is-loading');
+        }
+        if (!modal || !video) {
+            clearLoading();
+            return;
+        }
         const useMobileFullscreen = isMobilePlayback() || isTvPlayback();
         let playbackStarted = false;
 
@@ -60,6 +72,7 @@ App.player = App.player || {};
             if (App.ui && App.ui.showError) {
                 App.ui.showError('Unable to play this stream.');
             }
+            clearLoading();
             return;
         }
         const refererParam = resolved.referer ? `&referer=${encodeURIComponent(resolved.referer)}` : '';
@@ -128,6 +141,7 @@ App.player = App.player || {};
         const startPlayback = () => {
             if (playbackStarted) return;
             playbackStarted = true;
+            clearLoading();
             const playPromise = video.play();
             if (playPromise && typeof playPromise.catch === 'function') {
                 playPromise.catch(() => {});
@@ -152,6 +166,7 @@ App.player = App.player || {};
                 startPlayback();
                 state.hlsPlayer.on(window.Hls.Events.ERROR, function(event, data) {
                     if (data && data.fatal) {
+                        clearLoading();
                         if (App.ui && App.ui.showError) {
                             App.ui.showError('Unable to play this stream.');
                         }
@@ -166,6 +181,7 @@ App.player = App.player || {};
                 if (App.ui && App.ui.showError) {
                     App.ui.showError('HLS is not supported in this browser.');
                 }
+                clearLoading();
                 return;
             }
         } else {
@@ -174,6 +190,7 @@ App.player = App.player || {};
         }
 
         video.onerror = () => {
+            clearLoading();
             if (App.ui && App.ui.showError) {
                 App.ui.showError('Video failed to load.');
             }

frontend/js/videos.js

@@ -134,6 +134,9 @@ App.videos = App.videos || {};
                     <button class="video-menu-item" type="button" data-action="download" role="menuitem">Download</button>
                 </div>
                 <img src="${v.thumb}" alt="${v.title}">
+                <div class="video-loading" aria-hidden="true">
+                    <div class="video-loading-spinner"></div>
+                </div>
                 <h4>${v.title}</h4>
                 ${uploaderText ? `<p class="video-meta"><button class="uploader-link" type="button" data-uploader="${uploaderText}">${uploaderText}</button></p>` : ''}
                 ${durationText ? `<p class="video-duration">${durationText}</p>` : ''}
@@ -183,7 +186,11 @@ App.videos = App.videos || {};
                     App.videos.closeAllMenus();
                 };
             }
-            card.onclick = () => App.player.open(v);
+            card.onclick = () => {
+                if (card.classList.contains('is-loading')) return;
+                card.classList.add('is-loading');
+                App.player.open(v, { originEl: card });
+            };
             grid.appendChild(card);
             state.renderedVideoIds.add(v.id);
         });