refactor segment management logic for video duration and playback position, fixed control button height, set default clip selection to full length of video

Author: geek-at

index.html

@@ -87,8 +87,8 @@
       display: flex;
       align-items: center;
       gap: 10px;
+      height: 100%;
       -webkit-app-region: no-drag;
-      margin-right: 4px;
     }
 
     .gpu-badge {

js/controller.js

@@ -201,8 +201,8 @@ function loadVideo(filePath) {
       el.fps30Toggle.disabled = false;
     }
 
-    // Auto-add first segment (first 10s or full duration)
-    addSegment(0, Math.min(10, appState.videoDuration));
+    // Auto-add first segment covering the full video duration
+    addSegment(0, appState.videoDuration);
   };
 
   el.videoPlayer.ontimeupdate = () => {
@@ -262,10 +262,15 @@ function setupVideoControls() {
 function setupSegmentControls() {
   el.addSegmentBtn.addEventListener('click', () => {
     if (!appState.videoPath) return;
-    // Default new segment starts at current playback position
-    const cur = el.videoPlayer.currentTime || 0;
-    const end = Math.min(cur + 10, appState.videoDuration);
-    addSegment(cur, end, 1);
+    // If segments exist, start new segment at the end of the last one
+    let start;
+    if (appState.segments.length > 0) {
+      start = appState.segments[appState.segments.length - 1].endTime;
+    } else {
+      start = el.videoPlayer.currentTime || 0;
+    }
+    const end = Math.min(start + 10, appState.videoDuration);
+    addSegment(start, end, 1);
   });
 
   // "From Here" — sets start of last segment (or creates one) to current time

package.json

@@ -1,6 +1,6 @@
 {
   "name": "dta-cutter",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "description": "A simple video cutter for gaming clips with multi-segment support",
   "main": "main.js",
   "scripts": {