Merge pull request #4290 from alphagov/serach-results-metadata

Reformat search results metadata to be more verbose
alphagov · Nov 25, 2024 · 3bf7774 · 3bf7774
2 parents 3b481bf + ca306c3
commit 3bf7774
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 13 deletions.
diff --git a/lib/metalsmith-lunr-index/index.js b/lib/metalsmith-lunr-index/index.js
@@ -5,10 +5,6 @@ const navigationConfig = require('../../config/navigation')
 module.exports = function lunrPlugin() {
   return (files, metalsmith, done) => {
     const outputPath = 'search-index.json'
-
-    const separator =
-      '<span class="app-site-search__separator" aria-hidden="true">›</span>'
-
     const includedSections = navigationConfig
       .filter((section) => !section.ignoreInSearch)
       .map((section) => section.label)
@@ -88,7 +84,7 @@ module.exports = function lunrPlugin() {
         store[doc.permalink] = {
           permalink: doc.permalink,
           title: doc.title,
-          section: doc.section,
+          section: `in ${doc.section}`,
           aliases: doc.aliases
         }
         this.add(doc)
@@ -98,7 +94,7 @@ module.exports = function lunrPlugin() {
         store[doc.permalink] = {
           permalink: doc.permalink,
           title: doc.title,
-          section: `${doc.section}${separator}${doc.page}`,
+          section: `part of ${doc.page} in ${doc.section}`,
           aliases: doc.aliases
         }
         this.add(doc)

diff --git a/lib/metalsmith-lunr-index/index.test.js b/lib/metalsmith-lunr-index/index.test.js
@@ -59,7 +59,7 @@ describe('metalsmith-lunr-index plugin', () => {
         ({ title }) => title === 'Checkboxes'
       )
 
-      expect(entry.section).toEqual('Components')
+      expect(entry.section).toEqual('in Components')
     })
 
     it('uses a custom permalink if in document metadata', () => {

diff --git a/src/stylesheets/components/_site-search.scss b/src/stylesheets/components/_site-search.scss
@@ -303,9 +303,3 @@ $icon-size: 40px;
     content: ")";
   }
 }
-
-.app-site-search__separator {
-  display: inline-block;
-  margin-right: govuk-spacing(1);
-  margin-left: govuk-spacing(1);
-}