fix pdf line break

Author: ttop32

doc/description.md

@@ -19,6 +19,8 @@ Mouseover Translate Any Language At Once
 English, Russian, Japanese, Chinese and so on
 
 # Change Log
+- 0.1.206
+  - fix pdf line break (requested by mrk67422)
 - 0.1.205
   - Add right mouse hold in pdf viewer now act as handtool (contributed by Witnull)
   - filter parenthesis text from tts (requested by Gobidobi)

src/event/mouseover.js

@@ -108,7 +108,7 @@ export async function getMouseoverText(x, y) {
   var mouseoverText = getTextFromRange(range, mouseoverType);
   // if fail detect using expand range use seg range
   if (
-    !isFirefox() &&
+    !shouldUseSegmentation() &&
     (mouseoverType === "word" || mouseoverType === "sentence") &&
     !mouseoverText["mouseoverText"] &&
     checkContainerDetectText(range) // check if container detect text
@@ -130,6 +130,10 @@ function getTextFromRange(range, mouseoverType, useSegmentation= false) {
   return output;
 }
 
+function shouldUseSegmentation() {
+  return isFirefox() || util.isPDFViewer();
+}
+
 function expandRange(range, type, useSegmentation) {
   try {
     if (!range) {
@@ -138,7 +142,7 @@ function expandRange(range, type, useSegmentation) {
     if (type == "container") {
       // get whole text paragraph
       range = getContainerRange(range);
-    } else if (isFirefox() || useSegmentation) {
+    } else if (shouldUseSegmentation() || useSegmentation) {
       // for firefox, use segmentation to extract word
       range = expandRangeWithSeg(range, type);
     } else {

src/pdf/pdfInject.js

@@ -88,8 +88,8 @@ function waitUntilPdfLoad() {
 async function addSpaceBetweenPdfText() {
   var prevY;
   var prevLine;
-  var newLineScale = 1.5;
-  var spaceScale = 1.0;
+  var newLineScale = 1.3;  // if line gap more than 1.3 times font size, add new line
+  var spaceScale = 1.0;  // if line gap more than 1.0 times font size, add space
 
   // remove all br
   $("br").remove();