govuk_design_system_formbuilder 1.2.5 → 2.0.0b3

data/lib/govuk_design_system_formbuilder/containers/character_count.rb

@@ -14,7 +14,7 @@ module GOVUKDesignSystemFormBuilder
       def html
         return yield unless limit?
 
-        content_tag('div', **options) { yield }
+        tag.div(**options) { yield }
       end
 
     private

data/lib/govuk_design_system_formbuilder/containers/check_boxes.rb

@@ -8,7 +8,7 @@ module GOVUKDesignSystemFormBuilder
       end
 
       def html
-        content_tag('div', **options) { yield }
+        tag.div(**options) { yield }
       end
 
     private

data/lib/govuk_design_system_formbuilder/containers/check_boxes_fieldset.rb

@@ -4,20 +4,20 @@ module GOVUKDesignSystemFormBuilder
       include Traits::Error
       include Traits::Hint
 
-      def initialize(builder, object_name, attribute_name, hint_text:, legend:, caption:, small:, classes:, form_group_classes:, &block)
+      def initialize(builder, object_name, attribute_name, hint:, legend:, caption:, small:, classes:, form_group:, &block)
         super(builder, object_name, attribute_name, &block)
 
-        @legend             = legend
-        @caption            = caption
-        @hint_text          = hint_text
-        @small              = small
-        @classes            = classes
-        @form_group_classes = form_group_classes
-        @block_content      = capture { block.call }
+        @legend        = legend
+        @caption       = caption
+        @hint          = hint
+        @small         = small
+        @classes       = classes
+        @form_group    = form_group
+        @block_content = capture { block.call }
       end
 
       def html
-        Containers::FormGroup.new(@builder, @object_name, @attribute_name, classes: @form_group_classes).html do
+        Containers::FormGroup.new(@builder, @object_name, @attribute_name, **@form_group).html do
           Containers::Fieldset.new(@builder, @object_name, @attribute_name, **fieldset_options).html do
             safe_join([hint_element, error_element, checkboxes])
           end

data/lib/govuk_design_system_formbuilder/containers/fieldset.rb

@@ -11,7 +11,7 @@ module GOVUKDesignSystemFormBuilder
       end
 
       def html
-        content_tag('fieldset', **options) do
+        tag.fieldset(**options) do
           safe_join([legend_element, (@block_content || yield)])
         end
       end

data/lib/govuk_design_system_formbuilder/containers/form_group.rb

@@ -1,14 +1,15 @@
 module GOVUKDesignSystemFormBuilder
   module Containers
     class FormGroup < Base
-      def initialize(builder, object_name, attribute_name, classes: nil)
+      def initialize(builder, object_name, attribute_name, classes: nil, **kwargs)
         super(builder, object_name, attribute_name)
 
-        @classes = classes
+        @classes         = classes
+        @html_attributes = kwargs
       end
 
       def html
-        content_tag('div', class: classes) { yield }
+        tag.div(class: classes, **@html_attributes) { yield }
       end
 
     private

data/lib/govuk_design_system_formbuilder/containers/radio_buttons_fieldset.rb

@@ -4,21 +4,21 @@ module GOVUKDesignSystemFormBuilder
       include Traits::Hint
       include Traits::Error
 
-      def initialize(builder, object_name, attribute_name, hint_text:, legend:, caption:, inline:, small:, classes:, form_group_classes:, &block)
+      def initialize(builder, object_name, attribute_name, hint:, legend:, caption:, inline:, small:, classes:, form_group:, &block)
         super(builder, object_name, attribute_name)
 
-        @inline             = inline
-        @small              = small
-        @legend             = legend
-        @caption            = caption
-        @hint_text          = hint_text
-        @classes            = classes
-        @form_group_classes = form_group_classes
-        @block_content      = capture { block.call }
+        @inline        = inline
+        @small         = small
+        @legend        = legend
+        @caption       = caption
+        @hint          = hint
+        @classes       = classes
+        @form_group    = form_group
+        @block_content = capture { block.call }
       end
 
       def html
-        Containers::FormGroup.new(@builder, @object_name, @attribute_name, classes: @form_group_classes).html do
+        Containers::FormGroup.new(@builder, @object_name, @attribute_name, **@form_group).html do
           Containers::Fieldset.new(@builder, @object_name, @attribute_name, **fieldset_options).html do
             safe_join([hint_element, error_element, radios])
           end

data/lib/govuk_design_system_formbuilder/containers/radios.rb

@@ -11,7 +11,7 @@ module GOVUKDesignSystemFormBuilder
       end
 
       def html
-        content_tag('div', **options) { yield }
+        tag.div(**options) { yield }
       end
 
     private

data/lib/govuk_design_system_formbuilder/containers/supplemental.rb

@@ -10,7 +10,7 @@ module GOVUKDesignSystemFormBuilder
       def html
         return nil if @content.blank?
 
-        content_tag('div', id: supplemental_id) { @content }
+        tag.div(id: supplemental_id) { @content }
       end
 
     private

data/lib/govuk_design_system_formbuilder/elements/caption.rb

@@ -3,21 +3,26 @@ module GOVUKDesignSystemFormBuilder
     class Caption < Base
       include Traits::Localisation
 
-      def initialize(builder, object_name, attribute_name, text:, size: nil)
+      def initialize(builder, object_name, attribute_name, text: nil, size: nil, **kwargs)
         super(builder, object_name, attribute_name)
 
-        @text       = text(text)
-        @size_class = size_class(size)
+        @text            = text(text)
+        @size_class      = size_class(size)
+        @html_attributes = kwargs
       end
 
       def html
-        return nil if @text.blank?
+        return nil unless active?
 
-        tag.span(@text, class: @size_class)
+        tag.span(@text, class: @size_class, **@html_attributes)
       end
 
     private
 
+      def active?
+        @text.present?
+      end
+
       def text(override)
         override || localised_text(:caption)
       end

data/lib/govuk_design_system_formbuilder/elements/check_boxes/collection.rb

@@ -6,23 +6,23 @@ module GOVUKDesignSystemFormBuilder
         include Traits::Hint
         include Traits::Supplemental
 
-        def initialize(builder, object_name, attribute_name, collection, value_method:, text_method:, hint_method: nil, hint_text:, legend:, caption:, small:, classes:, form_group_classes:, &block)
+        def initialize(builder, object_name, attribute_name, collection, value_method:, text_method:, hint_method: nil, hint:, legend:, caption:, small:, classes:, form_group:, &block)
           super(builder, object_name, attribute_name, &block)
 
-          @collection         = collection
-          @value_method       = value_method
-          @text_method        = text_method
-          @hint_method        = hint_method
-          @small              = small
-          @legend             = legend
-          @caption            = caption
-          @hint_text          = hint_text
-          @classes            = classes
-          @form_group_classes = form_group_classes
+          @collection   = collection
+          @value_method = value_method
+          @text_method  = text_method
+          @hint_method  = hint_method
+          @small        = small
+          @legend       = legend
+          @caption      = caption
+          @hint         = hint
+          @classes      = classes
+          @form_group   = form_group
         end
 
         def html
-          Containers::FormGroup.new(@builder, @object_name, @attribute_name, classes: @form_group_classes).html do
+          Containers::FormGroup.new(@builder, @object_name, @attribute_name, **@form_group).html do
             Containers::Fieldset.new(@builder, @object_name, @attribute_name, **fieldset_options).html do
               safe_join([supplemental_content, hint_element, error_element, check_boxes])
             end

data/lib/govuk_design_system_formbuilder/elements/check_boxes/collection_check_box.rb

@@ -2,8 +2,6 @@ module GOVUKDesignSystemFormBuilder
   module Elements
     module CheckBoxes
       class CollectionCheckBox < Base
-        using PrefixableArray
-
         include Traits::CollectionItem
         include Traits::Hint
 
@@ -13,12 +11,12 @@ module GOVUKDesignSystemFormBuilder
           @checkbox    = checkbox
           @item        = checkbox.object
           @value       = checkbox.value
-          @hint_text   = retrieve(@item, hint_method)
+          @hint        = { text: retrieve(@item, hint_method) }
           @link_errors = link_errors
         end
 
         def html
-          content_tag('div', class: %(#{brand}-checkboxes__item)) do
+          tag.div(class: %(#{brand}-checkboxes__item)) do
             safe_join([check_box, label_element, hint_element])
           end
         end
@@ -42,7 +40,11 @@ module GOVUKDesignSystemFormBuilder
         end
 
         def hint_element
-          @hint_element ||= Elements::Hint.new(@builder, @object_name, @attribute_name, @hint_text, @value, checkbox: true)
+          @hint_element ||= Elements::Hint.new(@builder, @object_name, @attribute_name, **hint_options, **hint_content)
+        end
+
+        def hint_options
+          { value: @value, checkbox: true }
         end
       end
     end

data/lib/govuk_design_system_formbuilder/elements/check_boxes/fieldset_check_box.rb

@@ -8,12 +8,12 @@ module GOVUKDesignSystemFormBuilder
         include Traits::Hint
         include Traits::Conditional
 
-        def initialize(builder, object_name, attribute_name, value, label:, hint_text:, link_errors:, multiple:, &block)
+        def initialize(builder, object_name, attribute_name, value, label:, hint:, link_errors:, multiple:, &block)
           super(builder, object_name, attribute_name)
 
           @value       = value
           @label       = label
-          @hint_text   = hint_text
+          @hint        = hint
           @multiple    = multiple
           @link_errors = link_errors
 
@@ -30,7 +30,7 @@ module GOVUKDesignSystemFormBuilder
       private
 
         def item
-          content_tag('div', class: %(#{brand}-checkboxes__item)) do
+          tag.div(class: %(#{brand}-checkboxes__item)) do
             safe_join([check_box, label_element, hint_element])
           end
         end
@@ -62,7 +62,11 @@ module GOVUKDesignSystemFormBuilder
         end
 
         def hint_element
-          @hint_element ||= Elements::Hint.new(@builder, @object_name, @attribute_name, @hint_text, @value, checkbox: true)
+          @hint_element ||= Elements::Hint.new(@builder, @object_name, @attribute_name, **hint_options, **hint_content)
+        end
+
+        def hint_options
+          { checkbox: true }
         end
 
         def conditional_classes

data/lib/govuk_design_system_formbuilder/elements/date.rb

@@ -9,19 +9,19 @@ module GOVUKDesignSystemFormBuilder
 
       SEGMENTS = { day: '3i', month: '2i', year: '1i' }.freeze
 
-      def initialize(builder, object_name, attribute_name, legend:, caption:, hint_text:, date_of_birth: false, omit_day:, form_group_classes:, &block)
+      def initialize(builder, object_name, attribute_name, legend:, caption:, hint:, date_of_birth: false, omit_day:, form_group:, &block)
         super(builder, object_name, attribute_name, &block)
 
-        @legend             = legend
-        @caption            = caption
-        @hint_text          = hint_text
-        @date_of_birth      = date_of_birth
-        @omit_day           = omit_day
-        @form_group_classes = form_group_classes
+        @legend        = legend
+        @caption       = caption
+        @hint          = hint
+        @date_of_birth = date_of_birth
+        @omit_day      = omit_day
+        @form_group    = form_group
       end
 
       def html
-        Containers::FormGroup.new(@builder, @object_name, @attribute_name, classes: @form_group_classes).html do
+        Containers::FormGroup.new(@builder, @object_name, @attribute_name, **@form_group).html do
           Containers::Fieldset.new(@builder, @object_name, @attribute_name, **fieldset_options).html do
             safe_join([supplemental_content, hint_element, error_element, date])
           end
@@ -35,7 +35,7 @@ module GOVUKDesignSystemFormBuilder
       end
 
       def date
-        content_tag('div', class: %(#{brand}-date-input)) do
+        tag.div(class: %(#{brand}-date-input)) do
           safe_join([day, month, year])
         end
       end
@@ -61,8 +61,8 @@ module GOVUKDesignSystemFormBuilder
       def date_part(segment, width:, link_errors: false)
         value = @builder.object.try(@attribute_name).try(segment)
 
-        content_tag('div', class: %w(date-input__item).prefix(brand)) do
-          content_tag('div', class: %w(form-group).prefix(brand)) do
+        tag.div(class: %(#{brand}-date-input__item)) do
+          tag.div(class: %(#{brand}-form-group)) do
             safe_join([label(segment, link_errors), input(segment, link_errors, width, value)])
           end
         end

data/lib/govuk_design_system_formbuilder/elements/error_message.rb

@@ -12,14 +12,14 @@ module GOVUKDesignSystemFormBuilder
       def html
         return nil unless has_errors?
 
-        content_tag('span', class: %(#{brand}-error-message), id: error_id) do
-          safe_join([prefix, message])
+        tag.span(class: %(#{brand}-error-message), id: error_id) do
+          safe_join([hidden_prefix, message])
         end
       end
 
     private
 
-      def prefix
+      def hidden_prefix
         tag.span('Error: ', class: %(#{brand}-visually-hidden))
       end
 

data/lib/govuk_design_system_formbuilder/elements/error_summary.rb

@@ -12,7 +12,7 @@ module GOVUKDesignSystemFormBuilder
       def html
         return nil unless object_has_errors?
 
-        content_tag('div', class: summary_class, **summary_options) do
+        tag.div(class: summary_class, **summary_options) do
           safe_join([title, summary])
         end
       end
@@ -24,8 +24,8 @@ module GOVUKDesignSystemFormBuilder
       end
 
       def summary
-        content_tag('div', class: summary_class('body')) do
-          content_tag('ul', class: [%(#{brand}-list), summary_class('list')]) do
+        tag.div(class: summary_class('body')) do
+          tag.ul(class: [%(#{brand}-list), summary_class('list')]) do
             safe_join(list)
           end
         end

data/lib/govuk_design_system_formbuilder/elements/file.rb

@@ -8,18 +8,18 @@ module GOVUKDesignSystemFormBuilder
       include Traits::Label
       include Traits::Supplemental
 
-      def initialize(builder, object_name, attribute_name, hint_text:, label:, caption:, form_group_classes:, **kwargs, &block)
+      def initialize(builder, object_name, attribute_name, hint:, label:, caption:, form_group:, **kwargs, &block)
         super(builder, object_name, attribute_name, &block)
 
-        @label              = label
-        @caption            = caption
-        @hint_text          = hint_text
-        @extra_options      = kwargs
-        @form_group_classes = form_group_classes
+        @label           = label
+        @caption         = caption
+        @hint            = hint
+        @html_attributes = kwargs
+        @form_group      = form_group
       end
 
       def html
-        Containers::FormGroup.new(@builder, @object_name, @attribute_name, classes: @form_group_classes).html do
+        Containers::FormGroup.new(@builder, @object_name, @attribute_name, **@form_group).html do
           safe_join([label_element, supplemental_content, hint_element, error_element, file])
         end
       end
@@ -27,7 +27,7 @@ module GOVUKDesignSystemFormBuilder
     private
 
       def file
-        @builder.file_field(@attribute_name, **options, **@extra_options)
+        @builder.file_field(@attribute_name, **options, **@html_attributes)
       end
 
       def options

data/lib/govuk_design_system_formbuilder/elements/hint.rb

@@ -3,26 +3,53 @@ module GOVUKDesignSystemFormBuilder
     class Hint < Base
       using PrefixableArray
 
-      include Traits::Hint
       include Traits::Localisation
 
-      def initialize(builder, object_name, attribute_name, text, value = nil, radio: false, checkbox: false)
+      def initialize(builder, object_name, attribute_name, value: nil, text: nil, content: nil, radio: false, checkbox: false, **kwargs)
         super(builder, object_name, attribute_name)
 
-        @value     = value
-        @hint_text = retrieve_text(text)
-        @radio     = radio
-        @checkbox  = checkbox
+        @value           = value
+        @radio           = radio
+        @checkbox        = checkbox
+        @html_attributes = kwargs
+
+        if content
+          @raw = capture { content.call }
+        else
+          @text = retrieve_text(text)
+        end
+      end
+
+      def active?
+        [@text, @raw].any?(&:present?)
       end
 
       def html
-        return nil if @hint_text.blank?
+        return nil unless active?
+
+        content_tag(hint_tag, **hint_options, **@html_attributes) { hint_body }
+      end
+
+      def hint_id
+        return nil unless active?
 
-        tag.span(@hint_text, class: classes, id: hint_id)
+        build_id('hint')
       end
 
     private
 
+      def hint_options
+        { class: classes, id: hint_id }
+      end
+
+      def hint_tag
+        @raw.presence ? 'div' : 'span'
+      end
+
+      def hint_body
+        @raw || @text
+      end
+
       def retrieve_text(supplied)
         supplied.presence || localised_text(:hint)
       end