Merge pull request #43 from ConvivioTeam/remove-duplicate-search-resu…

…lt-snippets Remove duplicate search result snippets
alphagov · Aug 1, 2018 · 1def8cf · 1def8cf
2 parents 2b9e62f + fe6ebea
commit 1def8cf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/lib/assets/javascripts/_modules/collapsible-navigation.js b/lib/assets/javascripts/_modules/collapsible-navigation.js
@@ -31,7 +31,7 @@
         var $topLevelItem = $($topLevelItems[i]);
         var $heading = $topLevelItem.find('> a');
         var $listing = $topLevelItem.find('> ul');
-        var id = 'toc-' + $heading.text().toLowerCase().replace(' ', '-')
+        var id = 'toc-' + $heading.text().toLowerCase().replace(' ', '-');
         // Only add collapsible functionality if there are children.
         if ($listing.length == 0) {
           continue;

diff --git a/lib/assets/javascripts/_modules/search.js b/lib/assets/javascripts/_modules/search.js
@@ -149,9 +149,10 @@
           break;
         }
 
-        var containsMark = sentences[i].includes('mark>');
-        if (containsMark) {
-          selectedSentences.push(sentences[i].trim());
+        var sentence = sentences[i].trim();
+        var containsMark = sentence.includes('mark>');
+        if (containsMark && (selectedSentences.indexOf(sentence) == -1)) {
+          selectedSentences.push(sentence);
         }
       }
       if(selectedSentences.length > 0) {