ReadAloud ignores hyphens that span lines

src/js/plugins/tts/PageChunk.js

@@ -53,7 +53,8 @@ export default class PageChunk {
     static _fromTextWrapperResponse(leafIndex, chunksResponse) {
         return chunksResponse.map((c, i) => {
             const correctedLineRects = PageChunk._fixChunkRects(c.slice(1));
-            return new PageChunk(leafIndex, i, c[0], correctedLineRects);
+            const correctedText = PageChunk._removeDanglingHyphens(c[0]);
+            return new PageChunk(leafIndex, i, correctedText, correctedLineRects);
         });
     }
 
@@ -90,6 +91,15 @@ export default class PageChunk {
 
         return rects;
     }
+
+    /**
+     * Remove "dangling" hyphens from read aloud text to avoid TTS stuttering
+     * @param {string} text 
+     * @return {string}
+     */
+    static _removeDanglingHyphens(text) {
+        return text.replace(/-\s+/g, '');
+    }
 }
 
 /**

tests/plugins/tts/PageChunk.test.js

@@ -34,3 +34,24 @@ describe('_fromTextWrapperResponse', () => {
         expect(chunks).toEqual([new PageChunk(0, 0, 'Line', [[0,100,100,0]])]);
     });
 });
+
+describe('_removeDanglingHyphens', () => {
+    const { _removeDanglingHyphens } = PageChunk;
+
+    test('No change to empty string', () => {
+        expect(_removeDanglingHyphens('')).toEqual('');
+    });
+
+    test('No change when no hyphens string', () => {
+        expect(_removeDanglingHyphens('Hello world!')).toEqual('Hello world!');
+    });
+
+    test('No change to hyphens mid-word', () => {
+        expect(_removeDanglingHyphens('It is mid-word!')).toEqual('It is mid-word!');
+    });
+
+    test('Removes hyphens followed by spaces', () => {
+        expect(_removeDanglingHyphens('It is not mid- word!')).toEqual('It is not midword!');
+        expect(_removeDanglingHyphens('mid- word fid- word')).toEqual('midword fidword');
+    });
+});