Fix text selection issues

Author: OpenHypervideo

src/player/modules/InteractionController/module.js

@@ -16,13 +16,29 @@
 
 FrameTrail.defineModule('InteractionController', function(FrameTrail){
 
-    // Prevent accidental drag-selection of text (one-time global setup).
+    // Prevent accidental drag-selection of text within FrameTrail's own UI.
     // Uses pointer events to cover both mouse and touch input.
     // Double/triple-click word/line selection is still allowed via e.detail >= 2.
+    //
+    // Scope notes (important — this used to be a document-wide block):
+    //  - The block only applies inside this FrameTrail instance's root element,
+    //    so text on the surrounding host page (when FrameTrail is embedded) stays
+    //    selectable.
+    //  - Selectable content regions are exempted: the interactive transcript,
+    //    form fields, contenteditable areas, and anything opting in via the
+    //    .ft-selectable class. This lets users select transcript text while
+    //    overlay/timeline/slider dragging is still protected.
     document.addEventListener('pointerdown', function(e) {
         if (e.pointerType !== 'mouse') return; // touch/pen don't trigger selectstart
         if (e.detail >= 2) return;
-        if (e.target.closest('[contenteditable]')) return;
+
+        // Only guard within this instance's own UI; never touch host-page selection.
+        var rootSelector = FrameTrail.getState('target');
+        if (!rootSelector || !e.target.closest(rootSelector)) return;
+
+        // Leave genuinely selectable regions alone.
+        if (e.target.closest('[contenteditable], input, textarea, .transcriptContainer, .customhtmlContainer, .ft-selectable')) return;
+
         function onSelectStart(evt) {
             evt.preventDefault();
         }

src/player/types/ContentView/type.js

@@ -266,6 +266,12 @@ FrameTrail.defineType(
 
                             customhtmlContainer.addEventListener('click', function(evt) {
                                 if ( evt.target.classList.contains('timebased') ) {
+                                    // Don't seek when the click concludes a text selection,
+                                    // so users can select transcript text without the video jumping.
+                                    var selection = window.getSelection();
+                                    if (selection && !selection.isCollapsed && selection.toString().length > 0) {
+                                        return;
+                                    }
                                     FrameTrail.module('HypervideoController').currentTime = parseFloat(evt.target.getAttribute('data-start')) + 0.05;
                                 }
                             });
@@ -308,7 +314,15 @@ FrameTrail.defineType(
                             }
 
                             transcriptContainer.addEventListener('click', function(evt) {
-                                FrameTrail.module('HypervideoController').currentTime = parseFloat(evt.target.getAttribute('data-start')) + 0.05;
+                                // Don't seek when the click concludes a text selection,
+                                // so users can select transcript text without the video jumping.
+                                var selection = window.getSelection();
+                                if (selection && !selection.isCollapsed && selection.toString().length > 0) {
+                                    return;
+                                }
+                                var start = evt.target.getAttribute('data-start');
+                                if (start === null) return;
+                                FrameTrail.module('HypervideoController').currentTime = parseFloat(start) + 0.05;
                             });