From 7838342f9657d34ce8941ce2d63a8ac106173e08 Mon Sep 17 00:00:00 2001
From: isUnknown <adrien.payet@outlook.com>
Date: Fri, 20 Mar 2026 14:32:27 +0100
Subject: [PATCH] =?UTF-8?q?fix(expertise):=20attend=20seeked=20avant=20de?=
 =?UTF-8?q?=20switcher=20la=20vid=C3=A9o=20visible?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Le flash venait du fait que isReverse changeait immédiatement après
le seek (synchrone), mais le navigateur n'avait pas encore décodé
le frame cible. Maintenant on attend l'événement seeked du <video>
avant de changer isReverse, garantissant que le bon frame est affiché
au moment du switch.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/views/Expertise.svelte | 55 ++++++++++++++++++--------------------
 1 file changed, 26 insertions(+), 29 deletions(-)

diff --git a/src/views/Expertise.svelte b/src/views/Expertise.svelte
index fbd6bbf..19b286d 100644
--- a/src/views/Expertise.svelte
+++ b/src/views/Expertise.svelte
@@ -84,43 +84,41 @@
     }
   }
 
-  /**
-   * Switch visuel entre les deux vidéos.
-   * Positionne la vidéo cible au bon frame AVANT de l'afficher
-   * pour éliminer les flashes.
-   */
-  function switchToForward() {
-    if (!isReverse) return
-    // Positionner forward au bon frame avant de l'afficher
-    if (videoFwd) videoFwd.currentTime = currentFwdTime
-    isReverse = false
-  }
-
-  function switchToReverse() {
-    if (isReverse) return
-    // Positionner reverse au bon frame avant de l'afficher
-    const revPos = videoDuration - currentFwdTime
-    if (videoRev) videoRev.currentTime = revPos
-    isReverse = true
-  }
-
   function playForward(targetTime) {
     fwdTarget = targetTime
+
+    if (!isReverse) {
+      // Déjà sur forward : seek + play directement
+      if (videoFwd) videoFwd.currentTime = currentFwdTime
+      videoFwd?.play().catch(() => {})
+      return
+    }
+
+    // Changement de direction : seek forward, attendre seeked, puis switcher
     if (videoFwd) videoFwd.currentTime = currentFwdTime
-    // Switcher visuellement, puis lancer la lecture
-    switchToForward()
-    videoFwd?.play().catch(() => {})
+    videoFwd?.addEventListener('seeked', () => {
+      isReverse = false       // maintenant forward est au bon frame → on l'affiche
+      videoFwd?.play().catch(() => {})
+    }, { once: true })
   }
 
   function playReverse(targetFwdPos) {
-    // targetFwdPos = la position forward-équivalente où on veut arriver
-    // On la convertit en position dans la vidéo reverse
     revTarget = videoDuration - Math.max(targetFwdPos, 0)
     const revStart = videoDuration - currentFwdTime
+
+    if (isReverse) {
+      // Déjà sur reverse : seek + play directement
+      if (videoRev) videoRev.currentTime = revStart
+      videoRev?.play().catch(() => {})
+      return
+    }
+
+    // Changement de direction : seek reverse, attendre seeked, puis switcher
     if (videoRev) videoRev.currentTime = revStart
-    // Switcher visuellement, puis lancer la lecture
-    switchToReverse()
-    videoRev?.play().catch(() => {})
+    videoRev?.addEventListener('seeked', () => {
+      isReverse = true        // maintenant reverse est au bon frame → on l'affiche
+      videoRev?.play().catch(() => {})
+    }, { once: true })
   }
 
   // --- Navigate: move one item up or down (called by composable) ---
@@ -322,7 +320,6 @@
     height: 100%;
     object-fit: cover;
     opacity: 0;
-    transition: opacity 0.15s ease;
   }
 
   .expertise-bg video.active {