Update srt_utils.py

srt_utils.py

@@ -38,11 +38,14 @@ def format_text_lines(text, max_chars=42):
     if not words:
         return ""
     
-    # If fits in one line
-    if len(text) <= max_chars:
+    # If fits in one line, but we might WANT to split if it's long (> 30 chars) for better reading (pyramid shape)
+    # The user complained about 42 chars being too long for one line.
+    FORCE_SPLIT_THRESHOLD = 30
+    
+    if len(text) <= max_chars and len(text) <= FORCE_SPLIT_THRESHOLD:
         return text
     
-    # Needs splitting
+    # Needs splitting (or we want to try splitting)
     # Simple split strategy: find middle space
     best_split_idx = -1
     best_balance = float('inf')
@@ -57,24 +60,36 @@ def format_text_lines(text, max_chars=42):
         len2 = len(line2)
         
         # Valid split? Only if both fit max_chars 
-        # (Or if single line is impossible, pick best fit)
         if len1 <= max_chars and len2 <= max_chars:
             balance = abs(len2 - len1)
-            # Bonus for bottom heavy (line2 >= line1)
+            # Bonus for bottom heavy (line2 >= line1) which looks better often (pyramid)
             if len2 >= len1:
                 balance -= 5 
             
             if balance < best_balance:
                 best_balance = balance
                 best_split_idx = i
-    
+                
+    # If we found a valid split
     if best_split_idx != -1:
+        # If the original text fit in one line (< max_chars), only use the split if it's reasonably balanced.
+        # If the split results in a tiny orphan like "I\nam going", stick to 1 line if possible.
+        if len(text) <= max_chars:
+             line1 = " ".join(words[:best_split_idx])
+             line2 = " ".join(words[best_split_idx:])
+             # If one line is very short relative to the other, maybe don't split?
+             # But user wants "Netflix style", usually balanced.  
+             pass 
+
         line1 = " ".join(words[:best_split_idx])
         line2 = " ".join(words[best_split_idx:])
         return f"{line1}\n{line2}"
     
-    # Fallback: if no valid split found (e.g. one word is super long or total > 84)
-    # Just try to split in half by words regardless of limit (player will wrap or clip)
+    # Fallback: if no valid split found (e.g. words too long), but whole text fits in max_chars
+    if len(text) <= max_chars:
+        return text
+        
+    # Fallback 2: Really long text, just split in middle
     mid = len(words) // 2
     return " ".join(words[:mid]) + "\n" + " ".join(words[mid:])