Fix block start index, accounting for trailing \n

Soreine · Soreine · commit 16f7381fa354 · 2016-05-25T11:09:41.000+02:00
diff --git a/diff-area.js b/diff-area.js
@@ -161,10 +161,9 @@ function mapDiffsToBlocks(diffs, type, blockMap) {
     var blockStartIndex = 0;
     return blockMap.map(function (block) {
         var ranges = findRangesBetween(absoluteRanges, blockStartIndex, blockStartIndex + block.getLength());
-        blockStartIndex += block.getLength();
+        blockStartIndex += block.getLength() + 1; // Account for \n
         return {
             text: block.getText(),
-            key: block.getKey(),
             ranges: ranges
         };
     });
@@ -199,10 +198,6 @@ function findDiff(mappedRanges, type, contentBlock, callback) {
         mapping.ranges.forEach(function (range) {
             callback(range.start, range.end);
         });
-    } else {
-        if (mapping) {
-            console.log('Content changed', mapping.key, contentBlock.getKey());
-        }
     }
 }
 
diff --git a/index.html b/index.html
@@ -23,27 +23,23 @@
     <body>
         <div id="content"></div>
 
-        <p id="left-initial" hidden="true">
-Accusantium libero illum tempore provident doloribus voluptate. Blanditiis et recusandae unde non necessitatibus. Nulla qui libero optio doloremque dolorum accusamus.
+        <p id="left-initial" hidden="true">Accusantium libero illum tempore provident doloribus voluptate. Blanditiis et recusandae unde non necessitatibus. Nulla qui libero optio doloremque dolorum accusamus.
 
 Sapiente est, un mot par ci, et corporis. Porro magni sit similique quo. Molestias rerum quo ipsum ut. Aut sint id non mollitia.
 
 Voluptas hic quia dolores quo inventore. Quasi minima sit ut culpa consequuntur et. Inventore perferendis nihil aut ipsa odio. Possimus ratione temporibus et aut. Dicta incidunt magni voluptatem.
 
-Molestiae optio et quam at labore voluptatem animi. Sed libero odio repellendus iure omnis eos. Alias sapiente placeat est provident repellendus. Earum minima vero nesciunt voluptatem ipsa. Sint vitae et hic eveniet sunt. Doloremque ea esse consequatur.
-        </p>
+Molestiae optio et quam at labore voluptatem animi. Sed libero odio repellendus iure omnis eos. Alias sapiente placeat est provident repellendus. Earum minima vero nesciunt voluptatem ipsa. Sint vitae et hic eveniet sunt. Doloremque ea esse consequatur.</p>
 
-        <p id="right-initial" hidden="true">
-Accusantium libero illum tempore provident doloribus voluptate. Blanditiis et recusandae unde non necessitatibus. Nulla qui libero optio doloremque dolorum accusamus.
+        <p id="right-initial" hidden="true">Accusantium libero illum tempore provident doloribus voluptate. Blanditiis et recusandae unde non necessitatibus. Nulla qui libero optio doloremque dolorum accusamus.
 
 Sapiente est molestias et corporis. Porro magni sit similique quo. Molestias rerum quo ipsum ut. Aut sint id non mollitia.
 
 Voluptas hic quia dolores quo inventore. Quasi minima sit ut culpa consequuntur et. Inventore perferendis nihil aut ipsa odio. Possimus ratione temporibus et aut. Dicta incidunt magni voluptatem.
 
 Id est unde eum. Et voluptatem ut fugiat consequatur temporibus harum. Quasi quae ut accusantium. Et architecto rerum optio qui maxime molestiae libero.
 
-Molestiae optio et quam at labore voluptatem animi. Sed libero odio repellendus iure omnis eos. Alias sapiente placeat est provident repellendus. Earum minima vero nesciunt voluptatem ipsa. Sint vitae et hic eveniet sunt. Doloremque ea esse consequatur.
-        </p>
+Molestiae optio et quam at labore voluptatem animi. Sed libero odio repellendus iure omnis eos. Alias sapiente placeat est provident repellendus. Earum minima vero nesciunt voluptatem ipsa. Sint vitae et hic eveniet sunt. Doloremque ea esse consequatur.</p>
 
         <script type="text/javascript" src="lib/diff-word-mode.js"></script>
         <script type="text/babel" src="diff-area.js"></script>
