Merge pull request opencast#1495 from JulianKniephoff/forward-button

Make behavior of forward button more intuitive
UoM-Podcast · Dec 9, 2024 · 0b282ae · 0b282ae
2 parents 6d1dd7b + d204ae4
commit 0b282ae
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/redux/videoSlice.ts b/src/redux/videoSlice.ts
@@ -171,14 +171,17 @@ const videoSlice = createSlice({
       state.jumpTriggered = true;
     },
     jumpToNextSegment: state => {
-      let nextSegmentIndex = state.activeSegmentIndex + 1;
+      const nextSegmentIndex = state.activeSegmentIndex + 1;
+      let jumpTarget = 0;
 
       if (state.activeSegmentIndex + 1 >= state.segments.length) {
-        // Jump to start of last segment
-        nextSegmentIndex = state.activeSegmentIndex;
+        // Jump to end of last segment
+        jumpTarget = state.segments[state.activeSegmentIndex].end;
+      } else {
+        jumpTarget = state.segments[nextSegmentIndex].start;
       }
 
-      updateCurrentlyAt(state, state.segments[nextSegmentIndex].start);
+      updateCurrentlyAt(state, jumpTarget);
       state.jumpTriggered = true;
     },
     addSegment: (state, action: PayloadAction<video["segments"][0]>) => {