public/js/app_checklist.js

@@ -1,5 +1,35 @@
 window.onload = async function () {
     const token = localStorage.getItem('auth_token');
+    const movieTableBody = document.getElementById('movie-table-body');
+
+    function showTableLoading() {
+      if (!movieTableBody) return;
+      movieTableBody.innerHTML = `
+        <tr>
+          <td colspan="3" class="text-center py-4" aria-live="polite" aria-busy="true">
+            <div class="d-inline-flex align-items-center gap-2">
+              <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+                <span class="visually-hidden">Chargement...</span>
+              </div>
+              <span>Chargement…</span>
+            </div>
+          </td>
+        </tr>
+      `;
+    }
+
+    function showTableError(message) {
+      if (!movieTableBody) return;
+      movieTableBody.innerHTML = `
+        <tr>
+          <td colspan="3">
+            <div class="alert alert-danger my-2" role="alert">
+              ${message || 'Erreur lors du chargement.'}
+            </div>
+          </td>
+        </tr>
+      `;
+    }
 
     const DEFAULT_COMPLETION_MODAL = Object.freeze({
       title: 'Félicitations! very nice 🎉🎉🎉',
@@ -155,39 +185,49 @@ window.onload = async function () {
       window.location.href = '/';
     }
 
-    // Fetch remaining data in parallel (Render latency is high).
-    const [completionModalContent, moviesRes, watchedRes] = await Promise.all([
-      fetchCompletionModalContent(),
-      fetch(`/api/movies?year=${encodeURIComponent(String(activeYear))}&view=checklist`, {
-        method: 'GET',
-        headers: { 'Authorization': `Bearer ${token}` }
-      }),
-      fetch('/api/movies/watchedMovies', {
-        method: 'GET',
-        headers: { 'Authorization': `Bearer ${token}` }
-      })
-    ]);
+    showTableLoading();
 
-    applyCompletionModalContent(completionModalContent);
+    let movies = [];
+    let watchedMovies = [];
+    let watchedMoviesInYear = [];
+    let movieImdbIds = new Set();
+    try {
+      // Fetch remaining data in parallel (Render latency is high).
+      const [completionModalContent, moviesRes, watchedRes] = await Promise.all([
+        fetchCompletionModalContent(),
+        fetch(`/api/movies?year=${encodeURIComponent(String(activeYear))}&view=checklist`, {
+          method: 'GET',
+          headers: { 'Authorization': `Bearer ${token}` }
+        }),
+        fetch('/api/movies/watchedMovies', {
+          method: 'GET',
+          headers: { 'Authorization': `Bearer ${token}` }
+        })
+      ]);
 
-    if (!moviesRes.ok) {
-      localStorage.removeItem('auth_token');
-      window.location.href = '/';
+      applyCompletionModalContent(completionModalContent);
+
+      if (!moviesRes.ok) {
+        localStorage.removeItem('auth_token');
+        window.location.href = '/';
+        return;
+      }
+
+      movies = await moviesRes.json();
+
+      // Sort movies alphabetically by title
+      movies.sort((a, b) => a.title.localeCompare(b.title, 'fr', { sensitivity: 'base' }));
+
+      if (watchedRes.ok) {
+        watchedMovies = await watchedRes.json();
+      }
+    } catch (e) {
+      showTableError('Impossible de charger la checklist. Réessaie dans quelques instants.');
       return;
     }
 
-    const movies = await moviesRes.json();
-
-    // Sort movies alphabetically by title
-    movies.sort((a, b) => a.title.localeCompare(b.title, 'fr', { sensitivity: 'base' }));
-
-    let watchedMovies = [];
-    if (watchedRes.ok) {
-      watchedMovies = await watchedRes.json();
-    }
-
-    const movieImdbIds = new Set(movies.map((m) => m.imdb_id).filter(Boolean));
-    let watchedMoviesInYear = watchedMovies.filter((wm) => movieImdbIds.has(wm.imdb_id));
+    movieImdbIds = new Set(movies.map((m) => m.imdb_id).filter(Boolean));
+    watchedMoviesInYear = watchedMovies.filter((wm) => movieImdbIds.has(wm.imdb_id));
 
     const totalMoviesCount = movies.length;
     const watchedMoviesCount = watchedMoviesInYear.length;
@@ -248,7 +288,7 @@ window.onload = async function () {
 
     updateProgressBar(watchedMoviesCount, totalMoviesCount);
 
-    const movieTableBody = document.getElementById('movie-table-body');
+    if (movieTableBody) movieTableBody.innerHTML = '';
     movies.forEach(movie => {
       const watchedMovie = watchedMoviesInYear.find(wm => wm.imdb_id === movie.imdb_id);
       const watchedDate = watchedMovie ? new Date(watchedMovie.watchedDate).toLocaleString() : '';

public/js/app_movies.js

@@ -1,4 +1,24 @@
 window.onload = async function () {
+    function setLoading(el) {
+      if (!el) return;
+      el.innerHTML = `
+        <div class="d-flex justify-content-center align-items-center my-5" aria-live="polite" aria-busy="true">
+          <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+            <span class="visually-hidden">Chargement...</span>
+          </div>
+        </div>
+      `;
+    }
+
+    function setLoadError(el, message) {
+      if (!el) return;
+      el.innerHTML = `
+        <div class="alert alert-danger my-3" role="alert">
+          ${message || 'Erreur lors du chargement.'}
+        </div>
+      `;
+    }
+
     async function fetchActiveYear() {
       try {
         const res = await fetch('/api/settings/year', { method: 'GET' });
@@ -105,83 +125,91 @@ window.onload = async function () {
       window.location.href = '/';
     }
 
+    const moviesList = document.getElementById('movies-list');
+    setLoading(moviesList);
+
     // Populate "Dernière mise à jour des films" banner (per active year)
     await fetchMoviesLastUpdated(activeYear, token);
 
-    // Fetch movies + watched list in parallel (Render latency is high).
-    const [res, watchedRes] = await Promise.all([
-      fetch(`/api/movies?year=${encodeURIComponent(String(activeYear))}`, {
-        method: 'GET',
-        headers: { 'Authorization': `Bearer ${token}` },
-      }),
-      fetch('/api/movies/watchedMovies', {
-        method: 'GET',
-        headers: { 'Authorization': `Bearer ${token}` },
-      })
-    ]);
+    try {
+      // Fetch movies + watched list in parallel (Render latency is high).
+      const [res, watchedRes] = await Promise.all([
+        fetch(`/api/movies?year=${encodeURIComponent(String(activeYear))}`, {
+          method: 'GET',
+          headers: { 'Authorization': `Bearer ${token}` },
+        }),
+        fetch('/api/movies/watchedMovies', {
+          method: 'GET',
+          headers: { 'Authorization': `Bearer ${token}` },
+        })
+      ]);
 
-    if (!res.ok) {
-      localStorage.removeItem('auth_token');
-      window.location.href = '/';
-      return;
+      if (!res.ok) {
+        localStorage.removeItem('auth_token');
+        window.location.href = '/';
+        return;
+      }
+
+      const movies = await res.json();
+
+      // Sort movies alphabetically by title (server also sorts, but keep client as a fallback)
+      movies.sort((a, b) => a.title.localeCompare(b.title, 'fr', { sensitivity: 'base' }));
+
+      let watchedMovies = [];
+      if (watchedRes.ok) {
+        watchedMovies = await watchedRes.json();
+      }
+
+      const movieImdbIds = new Set(movies.map((m) => m.imdb_id).filter(Boolean));
+      const watchedMoviesInYear = watchedMovies.filter((wm) => movieImdbIds.has(wm.imdb_id));
+
+      const totalMoviesCount = movies.length;
+      const watchedMoviesCount = watchedMoviesInYear.length;
+      const ratioPct = totalMoviesCount > 0 ? ((watchedMoviesCount / totalMoviesCount) * 100).toFixed(1) : '0.0';
+      const ratioText = `Vu: ${watchedMoviesCount} / ${totalMoviesCount} (${ratioPct}%)`;
+      document.getElementById('watched-ratio').innerText = ratioText;
+
+      if (moviesList) moviesList.innerHTML = '';
+
+      movies.forEach(movie => {
+        const isChecked = watchedMoviesInYear.some(watchedMovie => watchedMovie.imdb_id === movie.imdb_id);
+        const watchedMovie = watchedMoviesInYear.find(wm => wm.imdb_id === movie.imdb_id);
+        const watchedDate = watchedMovie ? new Date(watchedMovie.watchedDate).toLocaleString() : '';
+        const hasVideoSrc = !!(movie && typeof movie.video_src === 'string' && movie.video_src.trim());
+        const hasEmbedSrc = !!(movie && typeof movie.embed_src === 'string' && movie.embed_src.trim());
+        const hasNewPlayerSource = hasVideoSrc || hasEmbedSrc;
+        const hasLegacy = !!(movie && typeof movie.vod_link === 'string' && movie.vod_link.trim());
+
+        // If the movie only has legacy `vod_link`, skip the player entirely and open the original URL in a new tab.
+        const playerUrl = (movie && movie._id && hasNewPlayerSource)
+          ? `/player.html?id=${encodeURIComponent(movie._id)}`
+          : (hasLegacy ? movie.vod_link : (movie && movie._id ? `/player.html?id=${encodeURIComponent(movie._id)}` : '#'));
+
+        const movieDiv = document.createElement('div');
+        movieDiv.classList.add('col-md-4', 'mb-4', 'movie-card');
+        movieDiv.setAttribute('data-imdb-id', movie.imdb_id);
+        movieDiv.innerHTML = `
+          <div class="card">
+            ${isChecked ? '<div class="watched-banner">VISIONNÉ</div>' : ''}
+            <a href="${playerUrl}" target="_self" rel="noopener noreferrer">
+              <img src="${movie.poster}" class="card-img-top" alt="${movie.title}" style="width: 75%; display: block; margin: 0 auto; padding: 10px;">
+            </a>
+            <div class="card-body">
+              <h5 class="card-title d-flex justify-content-between align-items-center">
+                <a href="${playerUrl}" target="_self" rel="noopener noreferrer" class="text-decoration-none text-dark">${movie.title}</a>
+                <span>⭐ ${movie.rating} <a href="https://www.imdb.com/title/${movie.imdb_id}/"><img src="/img/imdb.png"></a></span>
+              </h5>
+              <p class="fw-bold fst-italic card-text" style="font-size: 0.75rem;">${movie.category}</p>
+              <p class="card-text">${movie.description}</p>
+              ${isChecked ? `<p class="card-text"><strong>Regardé le:</strong> ${watchedDate}</p>` : ''}
+            </div>
+          </div>`;
+        moviesList.appendChild(movieDiv);
+      });
+    } catch (e) {
+      setLoadError(moviesList, 'Impossible de charger la liste des films. Réessaie dans quelques instants.');
     }
 
-    const movies = await res.json();
-
-    // Sort movies alphabetically by title (server also sorts, but keep client as a fallback)
-    movies.sort((a, b) => a.title.localeCompare(b.title, 'fr', { sensitivity: 'base' }));
-
-    let watchedMovies = [];
-    if (watchedRes.ok) {
-      watchedMovies = await watchedRes.json();
-    }
-
-    const moviesList = document.getElementById('movies-list');
-    const movieImdbIds = new Set(movies.map((m) => m.imdb_id).filter(Boolean));
-    const watchedMoviesInYear = watchedMovies.filter((wm) => movieImdbIds.has(wm.imdb_id));
-
-    const totalMoviesCount = movies.length;
-    const watchedMoviesCount = watchedMoviesInYear.length;
-    const ratioPct = totalMoviesCount > 0 ? ((watchedMoviesCount / totalMoviesCount) * 100).toFixed(1) : '0.0';
-    const ratioText = `Vu: ${watchedMoviesCount} / ${totalMoviesCount} (${ratioPct}%)`;
-    document.getElementById('watched-ratio').innerText = ratioText;
-
-    movies.forEach(movie => {
-      const isChecked = watchedMoviesInYear.some(watchedMovie => watchedMovie.imdb_id === movie.imdb_id);
-      const watchedMovie = watchedMoviesInYear.find(wm => wm.imdb_id === movie.imdb_id);
-      const watchedDate = watchedMovie ? new Date(watchedMovie.watchedDate).toLocaleString() : '';
-      const hasVideoSrc = !!(movie && typeof movie.video_src === 'string' && movie.video_src.trim());
-      const hasEmbedSrc = !!(movie && typeof movie.embed_src === 'string' && movie.embed_src.trim());
-      const hasNewPlayerSource = hasVideoSrc || hasEmbedSrc;
-      const hasLegacy = !!(movie && typeof movie.vod_link === 'string' && movie.vod_link.trim());
-
-      // If the movie only has legacy `vod_link`, skip the player entirely and open the original URL in a new tab.
-      const playerUrl = (movie && movie._id && hasNewPlayerSource)
-        ? `/player.html?id=${encodeURIComponent(movie._id)}`
-        : (hasLegacy ? movie.vod_link : (movie && movie._id ? `/player.html?id=${encodeURIComponent(movie._id)}` : '#'));
-
-      const movieDiv = document.createElement('div');
-      movieDiv.classList.add('col-md-4', 'mb-4', 'movie-card');
-      movieDiv.setAttribute('data-imdb-id', movie.imdb_id);
-      movieDiv.innerHTML = `
-        <div class="card">
-          ${isChecked ? '<div class="watched-banner">VISIONNÉ</div>' : ''}
-          <a href="${playerUrl}" target="_self" rel="noopener noreferrer">
-            <img src="${movie.poster}" class="card-img-top" alt="${movie.title}" style="width: 75%; display: block; margin: 0 auto; padding: 10px;">
-          </a>
-          <div class="card-body">
-            <h5 class="card-title d-flex justify-content-between align-items-center">
-              <a href="${playerUrl}" target="_self" rel="noopener noreferrer" class="text-decoration-none text-dark">${movie.title}</a>
-              <span>⭐ ${movie.rating} <a href="https://www.imdb.com/title/${movie.imdb_id}/"><img src="/img/imdb.png"></a></span>
-            </h5>
-            <p class="fw-bold fst-italic card-text" style="font-size: 0.75rem;">${movie.category}</p>
-            <p class="card-text">${movie.description}</p>
-            ${isChecked ? `<p class="card-text"><strong>Regardé le:</strong> ${watchedDate}</p>` : ''}
-          </div>
-        </div>`;
-      moviesList.appendChild(movieDiv);
-    });
-
     document.getElementById('log-off').addEventListener('click', function () {
       localStorage.removeItem('auth_token');
       window.location.href = '/';

public/js/app_user-stat.js

@@ -3,6 +3,24 @@ window.onload = async function () {
     let winnersCache = null;
     let completionsCache = null;
 
+    function setTableBodyLoading(tbodyId, colspan) {
+      const tbody = document.getElementById(tbodyId);
+      if (!tbody) return;
+      const span = Number.isFinite(Number(colspan)) ? Number(colspan) : 1;
+      tbody.innerHTML = `
+        <tr>
+          <td colspan="${span}" class="text-center py-4" aria-live="polite" aria-busy="true">
+            <div class="d-inline-flex align-items-center gap-2">
+              <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+                <span class="visually-hidden">Chargement...</span>
+              </div>
+              <span class="text-muted">Chargement…</span>
+            </div>
+          </td>
+        </tr>
+      `;
+    }
+
     async function fetchActiveYear() {
       try {
         const res = await fetch('/api/settings/year', { method: 'GET' });
@@ -229,6 +247,16 @@ window.onload = async function () {
     }
 
     try {
+      // Show loading states immediately (avoid "blank" panels)
+      const table = document.querySelector('.user-table');
+      const spinner = document.getElementById('loading-spinner');
+      const statsError = document.getElementById('stats-error');
+      if (spinner) spinner.classList.remove('d-none');
+      if (table) table.classList.add('d-none');
+      if (statsError) statsError.classList.add('d-none');
+      setTableBodyLoading('winners-table-body', 2);
+      setTableBodyLoading('completers-table-body', 4);
+
       const activeYear = await fetchActiveYear();
       if (activeYear) {
         document.title = `Pool Oscars ${activeYear} - Statistiques des utilisateurs`;
@@ -257,9 +285,6 @@ window.onload = async function () {
 
       const stats = await statsRes.json();
       const userTableBody = document.getElementById('user-table-body');
-      const table = document.querySelector('.user-table');
-      const spinner = document.getElementById('loading-spinner');
-      const statsError = document.getElementById('stats-error');
 
       // Ensure watchedRatio is treated as a number and sort in descending order
       stats.sort((a, b) => parseFloat(b.watchedRatio) - parseFloat(a.watchedRatio));

public/user-stat.html

@@ -91,8 +91,10 @@
       <div id="stats-error" class="alert alert-danger d-none" role="alert"></div>
 
       <!-- Spinner shown while loading -->
-      <div id="loading-spinner" class="spinner-border text-primary" role="status">
-        <span class="visually-hidden">Loading...</span>
+      <div id="loading-spinner" class="d-flex justify-content-center align-items-center my-5" aria-live="polite" aria-busy="true">
+        <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+          <span class="visually-hidden">Chargement...</span>
+        </div>
       </div>
 
       <!-- Table for users and movies watched -->
@@ -123,7 +125,14 @@
           </thead>
           <tbody id="winners-table-body">
             <tr>
-              <td colspan="2" class="text-muted">Chargement…</td>
+              <td colspan="2" class="text-center py-4" aria-live="polite" aria-busy="true">
+                <div class="d-inline-flex align-items-center gap-2">
+                  <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+                    <span class="visually-hidden">Chargement...</span>
+                  </div>
+                  <span class="text-muted">Chargement…</span>
+                </div>
+              </td>
             </tr>
           </tbody>
         </table>
@@ -146,7 +155,14 @@
           </thead>
           <tbody id="completers-table-body">
             <tr>
-              <td colspan="4" class="text-muted">Chargement…</td>
+              <td colspan="4" class="text-center py-4" aria-live="polite" aria-busy="true">
+                <div class="d-inline-flex align-items-center gap-2">
+                  <div class="spinner-border text-secondary" role="status" aria-label="Chargement">
+                    <span class="visually-hidden">Chargement...</span>
+                  </div>
+                  <span class="text-muted">Chargement…</span>
+                </div>
+              </td>
             </tr>
           </tbody>
         </table>