govspeak 10.7.0 → 10.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b1efcb5cc3a8435cb52e7eb9a87539a1cf9a8fc3a000adfe75e0321b2f37a98
-  data.tar.gz: 9360c67b0ff42e408f30a7613e9b6388dc2cff57d3e9eef41022ad85daf265ca
+  metadata.gz: '0920f169885278f664642da1cb6d8441a856b175779b4ad701aa966051644d25'
+  data.tar.gz: f0924a2e6574dc0b8c79ca819195368696b20b3c1e1943c3db283b98d3fada34
 SHA512:
-  metadata.gz: d2e4582880a1156af68977a09bb84a1f6b7846fa13b8b01501ae79f6fc6fbe1a3ca4f9284395801b4fc29899d04b3ebf7b2a12b91b2588e295d6ec60979503fe
-  data.tar.gz: 7a13a22a339c51c3847a43a4468ede4201ddb537bd0edabe8cd8641de025dfaa62e20a3bdfb92827ad9509ea5356950c3434c59d41168510b9a2c22ea95188cf
+  metadata.gz: 75028eeb6446acc724c97b76f8f1ccbc04447e21c2016b89abfdb4154e18bea563b5176cb5ba014ef7c60579e07311ecae50a669999f4502253ce4951c25e810
+  data.tar.gz: 884b2da2acb61aaa721201b80c67d40d57a2cc0ce517b1ff15114d8e530f37de35ab11473b4dca946ad34c8537e14eaaae9f7bc57d629228565f8f06cae4285c

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 10.8.0
+
+* feature: Allow callers to specify which headers will be auto-numbered [PR #451](https://github.com/alphagov/govspeak/pull/451)
+
 ## 10.7.0
 
 * feature: Add auto-numbered headers option [PR #449](https://github.com/alphagov/govspeak/pull/449)

data/README.md

@@ -22,7 +22,7 @@ then create a new document
 
 Some additional steps or considerations are needed to ensure changes to govspeak cascade across GOV.UK in a holistic way.
 
-Once govspeak has been updated and version incremented then: 
+Once govspeak has been updated and version incremented then:
 - [`govuk_publishing_components` govspeak](https://components.publishing.service.gov.uk/component-guide/govspeak) will also need updating to reflect your most recent change.
 - [Publishing apps](https://docs.publishing.service.gov.uk/apps.html) (including but not limited to [content-publisher](https://github.com/alphagov/content-publisher) & [whitehall](https://github.com/alphagov/whitehall)) also use govspeak, these apps will need to be released with the new govspeak version present.
 
@@ -30,7 +30,7 @@ Also, consider if:
 - [whitehall](https://github.com/alphagov/whitehall) needs updating (as custom govspeak changes are present)
 - [govspeak-preview](https://github.com/alphagov/govspeak-preview) needs updating
 
-Any pages that use govspeak to generate Content will need to *republished* in order for the new changes to be reflected.  
+Any pages that use govspeak to generate Content will need to be *republished* in order for the new changes to be reflected.
 
 - Data Labs can help identify which pages need updating by [submitting a request](https://gov-uk.atlassian.net/wiki/spaces/GOVUK/pages/1860075525/GOV.UK+Data+Labs#Submitting-a-data-science-request) and [#govuk-2ndline](https://docs.publishing.service.gov.uk/manual/2nd-line.html) can help with republishing
 
@@ -48,6 +48,34 @@ with understanding how the content transforms from source through:
 
 In addition to the [standard Markdown syntax](http://daringfireball.net/projects/markdown/syntax "Markdown syntax"), we have added our own extensions.
 
+## Autonumbered headers
+
+This extension is off by default, can be turned on with
+
+    doc = Govspeak::Document.new "## Header\n### Sub-header\n#### Sub-sub-header", auto_numbered_headers: true
+
+Creates automatically numbered headers
+
+```html
+<h2>1 Header</h2>
+<h3>1.1 Sub-header</h3>
+<h4>1.1.1 Sub-sub-header</h4>
+```
+
+By default the numbering scheme will work for all headers apart from H1s, if you want to
+restrict this you can specify the levels you want in an array (default value is [2,3,4,5,6])
+
+    doc = Govspeak::Document.new "## Header\n### Sub-header\n#### Sub-sub-header", auto_numbered_headers: true, auto_numbered_header_levels: [2,3]
+
+Creates automatically numbered headers
+
+```html
+<h2>1 Header</h2>
+<h3>1.1 Sub-header</h3>
+<h4>Sub-sub-header</h4>
+```
+
+
 ## Callouts
 
 ### Information callouts

data/lib/govspeak/post_processor.rb

@@ -164,7 +164,9 @@ module Govspeak
       if govspeak_document.auto_numbered_headers
         h2, h3, h4, h5, h6 = 0, 0, 0, 0, 0, 0
 
-        document.css("h2,h3,h4,h5,h6").map do |el|
+        selector = govspeak_document.auto_numbered_header_levels.map { |i| "h#{i}" }.join(",")
+
+        document.css(selector).map do |el|
           case el.name
           when "h2"
             h2 += 1

data/lib/govspeak/structured_header_extractor.rb

@@ -41,7 +41,7 @@ module Govspeak
         stack.push(header)
       end
 
-      add_auto_numbering(structured_headers) if doc.auto_numbered_headers
+      add_auto_numbering(structured_headers, doc.auto_numbered_header_levels) if doc.auto_numbered_headers
 
       structured_headers
     end
@@ -98,10 +98,10 @@ module Govspeak
       @stack = []
     end
 
-    def add_auto_numbering(structured_headers, prefix: "")
+    def add_auto_numbering(structured_headers, levels, prefix: "")
       structured_headers.each.with_index(1) do |header, index|
-        header[:text] = "#{prefix}#{index}#{'.' if prefix == ''} #{header[:text]}"
-        add_auto_numbering(header[:headers], prefix: "#{prefix}#{index}.")
+        header[:text] = "#{prefix}#{index}#{'.' if prefix == ''} #{header[:text]}" if levels.include?(header[:level])
+        add_auto_numbering(header[:headers], levels, prefix: "#{prefix}#{index}.")
       end
     end
 

data/lib/govspeak/version.rb

@@ -1,3 +1,3 @@
 module Govspeak
-  VERSION = "10.7.0".freeze
+  VERSION = "10.8.0".freeze
 end

data/lib/govspeak.rb

@@ -38,7 +38,7 @@ module Govspeak
     @extensions = []
 
     attr_accessor :images
-    attr_reader :attachments, :auto_numbered_headers, :contacts, :links, :locale, :log_snapshots
+    attr_reader :attachments, :auto_numbered_headers, :auto_numbered_header_levels, :contacts, :links, :locale, :log_snapshots
 
     def self.to_html(source, options = {})
       new(source, options).to_html
@@ -55,12 +55,12 @@ module Govspeak
       @log_snapshots = options.fetch(:log_snapshots, false)
       log_snapshot("options", options)
       log_snapshot("source", @source)
-
       @images = options.delete(:images) || []
       @allowed_elements = options.delete(:allowed_elements) || []
       @allowed_image_hosts = options.delete(:allowed_image_hosts) || []
       @attachments = Array.wrap(options.delete(:attachments))
       @auto_numbered_headers = options.fetch(:auto_numbered_headers, false)
+      @auto_numbered_header_levels = options.delete(:auto_numbered_header_levels) || [2, 3, 4, 5, 6]
       @links = Array.wrap(options.delete(:links))
       @contacts = Array.wrap(options.delete(:contacts))
       @locale = options.fetch(:locale, "en")

data/test/govspeak_structured_headers_test.rb

@@ -166,4 +166,25 @@ class GovspeakStructuredHeadersTest < Minitest::Test
     assert_equal "5. Heading 5", headers[4][:text]
     assert_equal "5.1 Sub heading 5.1", headers[4][:headers][0][:text]
   end
+
+  test "auto-numbered headers are restricted to appropriate levels when the option is set on the document" do
+    doc = Govspeak::Document.new(document_body, auto_numbered_headers: true, auto_numbered_header_levels: [2, 3])
+
+    headers = doc.structured_headers
+
+    assert_equal "1. Heading 1", headers[0][:text]
+    assert_equal "2. Heading 2", headers[1][:text]
+    assert_equal "2.1 Sub heading 2.1", headers[1][:headers][0][:text]
+    assert_equal "2.2 Sub heading 2.2", headers[1][:headers][1][:text]
+    assert_equal "Sub sub heading 2.2.1", headers[1][:headers][1][:headers][0][:text]
+    assert_equal "2.3 Sub heading 2.3", headers[1][:headers][2][:text]
+    assert_equal "3. Heading 3", headers[2][:text]
+    assert_equal "4. Heading 4", headers[3][:text]
+    assert_equal "4.1 Sub heading 4.1", headers[3][:headers][0][:text]
+    assert_equal "Sub sub heading 4.1.1", headers[3][:headers][0][:headers][0][:text]
+    assert_equal "Sub sub sub heading 4.1.1.1", headers[3][:headers][0][:headers][0][:headers][0][:text]
+    assert_equal "4.2 Sub heading 4.2", headers[3][:headers][1][:text]
+    assert_equal "5. Heading 5", headers[4][:text]
+    assert_equal "5.1 Sub heading 5.1", headers[4][:headers][0][:text]
+  end
 end

data/test/govspeak_test.rb

@@ -121,6 +121,28 @@ class GovspeakTest < Minitest::Test
     assert_equal %(\n<h2 id="h2-one">1. H2 One</h2>\n\n<h3 id="h3-one">1.1 H3 One</h3>\n\n<h3 id="h3-two">1.2 H3 Two</h3>\n\n<h2 id="h2-two">2. H2 Two</h2>\n\n<h3 id="h3-three">2.1 H3 Three</h3>\n\n<h4 id="h4-one">2.1.1 H4 One</h4>\n\n<h5 id="h5-one">2.1.1.1 H5 One</h5>\n\n<h6 id="h6-one">2.1.1.1.1 H6 One</h6>\n), doc.to_html
   end
 
+  test "restricts auto-numbered headings to specified levels when the auto_numbered_heading_levels option is set" do
+    input = %(
+## H2 One
+
+### H3 One
+
+### H3 Two
+
+## H2 Two
+
+### H3 Three
+
+#### H4 One
+
+##### H5 One
+
+###### H6 One
+)
+    doc = Govspeak::Document.new(input, auto_numbered_headers: true, auto_numbered_header_levels: [2, 3])
+    assert_equal %(\n<h2 id="h2-one">1. H2 One</h2>\n\n<h3 id="h3-one">1.1 H3 One</h3>\n\n<h3 id="h3-two">1.2 H3 Two</h3>\n\n<h2 id="h2-two">2. H2 Two</h2>\n\n<h3 id="h3-three">2.1 H3 Three</h3>\n\n<h4 id="h4-one">H4 One</h4>\n\n<h5 id="h5-one">H5 One</h5>\n\n<h6 id="h6-one">H6 One</h6>\n), doc.to_html
+  end
+
   test "trailing space after the address should not prevent parsing" do
     input = %($A
 123 Test Street

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: govspeak
 version: !ruby/object:Gem::Version
-  version: 10.7.0
+  version: 10.8.0
 platform: ruby
 authors:
 - GOV.UK Dev