Fixed mir3z#3

dchartash · Jan 1, 2014 · 19169bb · 19169bb
2 parents 1a54b6f + 5440eb4
commit 19169bb
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 3 deletions.
diff --git a/src/jquery.textHighlighter.js b/src/jquery.textHighlighter.js
@@ -412,7 +412,7 @@
 
                 hlDescriptors.push([
                     wrapper,
-                    highlight.innerText,
+                    $(highlight).text(),
                     hlPath.join(':'),
                     offset,
                     length

diff --git a/src/jquery.textHighlighter.min.js b/src/jquery.textHighlighter.min.js
diff --git a/src/jquery.textHighlighter.pack.min.js b/src/jquery.textHighlighter.pack.min.js
diff --git a/test/test.js b/test/test.js
@@ -1062,14 +1062,18 @@ $(document).ready(function() {
         var hl = $sandbox.getHighlighter();
         var highlights = hl.getAllHighlights($sandbox);
         var jsonStr = hl.serializeHighlights();
+        var json = JSON.parse(jsonStr);
         var expectedHtml = $sandbox.html();
 
         $sandbox.html(params.sandboxInitHtml);
         var deserializedHighlights = hl.deserializeHighlights(jsonStr);
         var actualHtml = $sandbox.html();
+        var serializedText = ($.map(json, function (val) { return val[1]; })).join('');
+        var expectedText = ($.map(params.highlights, function (val) { return val.rangeExpectedText; })).join('');
 
         equal(expectedHtml, actualHtml, 'Serialization and deserialization successful');
         equal(highlights.length, deserializedHighlights.length, 'All highlights deserialized');
+        equal(serializedText, expectedText, 'Highlights text ok');
     }
 
     test('Serialize one highlight', function() {