govuk_publishing_components 9.13.0 → 9.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a8b61958875082d4c07fbe597644029b2aa8430fa88bc7988e87db41880b8f9
-  data.tar.gz: 6de83fa7c13a30edcad268f77f9fb2d29a4f69ab8a47c137f488f9514c676b92
+  metadata.gz: '0728a64f2f0fc18019ddf15ee6eae25ce7e1372783163cc99fa82e70653f1dca'
+  data.tar.gz: 5f335f94637469efe80af8c17cd63fba0e5de6a04c919fa63bea08c8cf9e40f0
 SHA512:
-  metadata.gz: f73daf1fd6fdc7135e268b9ff363cf6e07555937fb753735075123028392c5680d322470d003fe4d062cf14394b6a938848a36bafb1fc7c36adcbcc0cb93e5c0
-  data.tar.gz: b56d58373dd2688ca846fbbad203206080a64476933baccd5a53e5edf6479f3786b9ff09ac14a4d58729d34fe55bc0d5aae4efd25beeef8fdd079d18b6e6365a
+  metadata.gz: a1a5ce40351d6682df52aaa4b38edc225bb29ce13cad65b04f9a085303c8892cba201c27e786505e63e8c53c7a3bdf2abf925f83bbe8d54154d7c9e1ed773ead
+  data.tar.gz: 8c0b59628431aeea2f132a47b68cfb8e782250620fb3c8d3ab9ab40d7c8d0e568fc1b91dc0d161dd5cbc4d6e8bbe1845117f8cc602f8bdc893c47879de9a00fc

data/app/assets/javascripts/govuk_publishing_components/components/step-by-step-nav.js

@@ -232,32 +232,22 @@
           return;
         }
 
-        var lastClicked = loadFromSessionStorage(sessionStoreLink);
+        var lastClicked = loadFromSessionStorage(sessionStoreLink) || $element.find('.' + activeLinkClass).first().attr('data-position');
 
-        if (lastClicked) {
-          // it's possible for the saved link position value to not match any of the currently duplicate highlighted links
-          // so check this otherwise it'll take the highlighting off all of them
-          if (!$element.find('.js-link[data-position="' + lastClicked + '"]').parent().hasClass(activeLinkClass)) {
-            lastClicked = $element.find('.' + activeLinkClass).first().find('.js-link').attr('data-position');
-          }
-          removeActiveStateFromAllButCurrent($activeLinks, lastClicked);
-          setActiveStepClass();
-        } else {
-          var activeLinkInActiveStep = $element.find('.' + activeStepClass).find('.' + activeLinkClass).first();
-
-          if (activeLinkInActiveStep.length) {
-            $activeLinks.removeClass(activeLinkClass);
-            activeLinkInActiveStep.addClass(activeLinkClass);
-          } else {
-            $activeLinks.slice(1).removeClass(activeLinkClass);
-          }
+        // it's possible for the saved link position value to not match any of the currently duplicate highlighted links
+        // so check this otherwise it'll take the highlighting off all of them
+        if (!$element.find('.js-link[data-position="' + lastClicked + '"]').parent().hasClass(activeLinkClass)) {
+          lastClicked = $element.find('.' + activeLinkClass).first().find('.js-link').attr('data-position');
         }
+        removeActiveStateFromAllButCurrent($activeLinks, lastClicked);
+        setActiveStepClass();
       }
 
       function removeActiveStateFromAllButCurrent($activeLinks, current) {
         $activeLinks.each(function() {
           if ($(this).find('.js-link').attr('data-position').toString() !== current.toString()) {
             $(this).removeClass(activeLinkClass);
+            $(this).find('.visuallyhidden').remove();
           }
         });
       }

data/app/views/govuk_publishing_components/components/docs/step_by_step_nav.yml

@@ -215,19 +215,16 @@ examples:
               contents: [
                 {
                   href: '/component-guide/step_by_step_navigation/with_links/preview',
-                  text: 'Obtain a driving licence prior to driving a car',
-                  active: true
+                  text: 'Obtain a driving licence prior to driving a car'
                 },
                 {
                   href: '/component-guide/step_by_step_navigation/with_links/preview',
                   text: 'Acquire food before attempting to cook',
-                  context: '1p to £20',
-                  active: true
+                  context: '1p to £20'
                 },
                 {
                   href: '/component-guide/step_by_step_navigation/with_links/preview',
-                  text: 'Maintain contact with the ground at all times',
-                  active: true
+                  text: 'Maintain contact with the ground at all times'
                 }
               ]
             },
@@ -241,14 +238,12 @@ examples:
               contents: [
                 {
                   href: '/component-guide/step_by_step_navigation/',
-                  text: 'Leave school at sixteen',
-                  active: true
+                  text: 'Leave school at sixteen'
                 },
                 {
                   href: '/component-guide/step_by_step_navigation/',
                   text: 'Continue into higher education',
-                  context: '£9,500',
-                  active: true
+                  context: '£9,500'
                 }
               ]
             }
@@ -272,8 +267,7 @@ examples:
                 },
                 {
                   href: '/component-guide/step_by_step_navigation/with_links/preview',
-                  text: 'Do the things',
-                  active: true
+                  text: 'Do the things'
                 }
               ]
             },
@@ -340,22 +334,24 @@ examples:
       ]
   solve_the_double_dot_problem:
     description: |
-      If a page is in a step by step navigation more than once and a user is viewing that URL, both links to it will be highlighted as the backend has no way to know which link the user is currently viewing (links should only be highlighted when the step by step navigation is in the sidebar).
+      As shown, options can be passed to the component to highlight one step, expand one step by default, and highlight multiple links. These options should only be used when the component is in the sidebar. The step that is highlighted and expanded will be referred to as the active step (even though they are separate options, it is assumed that they will normally be applied to the same step).
 
-      JavaScript is included in the component to solve this. It uses sessionStorage to capture the data-position attribute of non-external links when clicked, and then uses this value to decide which link (and parent step) to highlight and expand when the new page loads. Note that it uses the tracking_id attribute to uniquely identify the current step nav. If tracking_id is not set this may result in other step navs having the wrong link highlighted.
+      If a link is in a step by step navigation more than once and the user is on that page, the backend doesn't know which link to highlight, so it highlights both of them.
 
-      If a user has not clicked a link (i.e. has visited the page without first clicking on a step by step navigation) the first active link in the first active step will be highlighted. If there is no active step, the first active link will be highlighted (but there should always be an active step).
+      JavaScript is included in the component to solve this. It uses sessionStorage to capture the data-position attribute of non-external links when clicked, and then uses this value to decide which link to highlight after the new page loads. It uses the value of the tracking_id option to uniquely identify the current step nav in the session (if this is not passed to the component this may result in other step navs having the wrong link highlighted).
 
-      The current page in the step by step navigation is an anchor link to the top of the page. If there are more than one of these and the user clicks one that is not currently highlighted, that one will be highlighted.
+      If a user has not clicked a link (i.e. has visited the page without first clicking on a step by step navigation) only the first duplicate link will be highlighted. If that link is not in the active step, the JS makes the highlighted link's parent the active step. If there is no active step, the first active link will be highlighted (but there should always be an active step).
 
       The example below will show all non-external links highlighted if JS is disabled. In the real world no more than two or three links are likely to be highlighted at once.
+
+      Changes to the active step and highlighted link are also applied on click, if the user clicks one of the other links that is to the same page (they are amended by the component to be jump links to the top of the page). Open this example using the 'preview' link and try clicking on the 'internal' links to see how it behaves.
     data:
       highlight_step: 2
       show_step: 2
       tracking_id: "example-id"
       steps: [
         {
-          title: "Not the active step",
+          title: "The active step",
           contents: [
             {
               type: 'list',
@@ -383,7 +379,7 @@ examples:
           ]
         },
         {
-          title: "The active step",
+          title: "Not the active step",
           contents: [
             {
               type: 'list',

data/config/routes.rb

@@ -1,5 +1,5 @@
 GovukPublishingComponents::Engine.routes.draw do
-  root :to => 'component_guide#index', as: :component_guide
+  root to: 'component_guide#index', as: :component_guide
   get ':component/preview' => 'component_guide#preview', as: :component_preview_all
   get ':component/:example/preview' => 'component_guide#preview', as: :component_preview
   get ':component' => 'component_guide#show', as: :component_doc

data/lib/govuk_publishing_components/presenters/machine_readable/article_schema.rb

@@ -29,7 +29,7 @@ module GovukPublishingComponents
               "url" => page.logo_url,
             }
           }
-        }.merge(image_schema).merge(author_schema).merge(body).merge(is_part_of)
+        }.merge(image_schema).merge(author_schema).merge(body).merge(is_part_of).merge(about)
       end
 
     private
@@ -98,6 +98,29 @@ module GovukPublishingComponents
           image_placeholders: page.image_placeholders
         )
       end
+
+      def about
+        return {} unless live_taxons.any?
+        {
+            "about" => linked_taxons
+        }
+      end
+
+      def live_taxons
+        taxons = page.content_item.dig("links", "taxons")
+        return [] unless taxons
+        taxons.select { |taxon| taxon["phase"] == "live" }
+      end
+
+      def linked_taxons
+        live_taxons.map do |taxon|
+          {
+              "@context" => "http://schema.org",
+              "@type" => "CreativeWork",
+              "sameAs" => taxon["web_url"]
+          }
+        end
+      end
     end
   end
 end

data/lib/govuk_publishing_components/version.rb

@@ -1,3 +1,3 @@
 module GovukPublishingComponents
-  VERSION = '9.13.0'.freeze
+  VERSION = '9.14.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: govuk_publishing_components
 version: !ruby/object:Gem::Version
-  version: 9.13.0
+  version: 9.14.0
 platform: ruby
 authors:
 - GOV.UK Dev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-14 00:00:00.000000000 Z
+date: 2018-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: govuk_app_config