commit 8b29fec247bac3f33b3b9179504de108ecd61171
author Randall Leeds <randall@apache.org> Sat Sep 28 16:30:42 2019 -0700
committer Randall Leeds <randall@apache.org> Sat Sep 28 16:30:42 2019 -0700
tree 5d884d3765c8486f95c6a6ccaff69ee29e84d95d
parent f7864a513d528e9c1cd750d5015d5f19d7a6df44

Improve handling of affixes at scope boundaries

packages/dom/src/text-quote.js

diff --git a/packages/dom/src/text-quote.js b/packages/dom/src/text-quote.js
index 9a7f28c..81bd0d8 100644
--- a/packages/dom/src/text-quote.js
+++ b/packages/dom/src/text-quote.js
@@ -135,7 +135,7 @@
       : seek(iter, startNode);
   const endIndex = startIndex + exact.length;
 
-  const affixLengthPairs = [[0, 0]];
+  const affixLengthPairs = [];
 
   for await (const match of selector(scope)) {
     const matchIter = createNodeIterator(root, SHOW_TEXT);
@@ -174,11 +174,11 @@
   if (affixLengthPairs.length) {
     const [prefixLength, suffixLength] = minimalSolution(affixLengthPairs);
 
-    if (prefixLength > 0) {
+    if (prefixLength > 0 && startIndex > 0) {
       result.prefix = text.substring(startIndex - prefixLength, startIndex);
     }
 
-    if (suffixLength > 0) {
+    if (suffixLength > 0 && endIndex < text.length) {
       result.suffix = text.substring(endIndex, endIndex + suffixLength);
     }
   }