govuk-components 3.0.4 → 3.1.0

data/app/components/govuk_component/tab_component.rb

@@ -1,29 +1,31 @@
 class GovukComponent::TabComponent < GovukComponent::Base
+  using HTMLAttributesUtils
+
   renders_many :tabs, "Tab"
 
   attr_reader :title, :id
 
   def initialize(title:, id: nil, classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @title = title
     @id    = id
+
+    super(classes: classes, html_attributes: html_attributes)
   end
 
 private
 
-  def default_classes
-    %w(govuk-tabs)
+  def default_attributes
+    { id: id, class: %w(govuk-tabs), data: { module: 'govuk-tabs' } }
   end
 
   class Tab < GovukComponent::Base
     attr_reader :label, :text
 
     def initialize(label:, text: nil, classes: [], html_attributes: {})
-      super(classes: classes, html_attributes: html_attributes)
-
       @label = label
       @text  = text
+
+      super(classes: classes, html_attributes: html_attributes)
     end
 
     def id(prefix: nil)
@@ -31,7 +33,9 @@ private
     end
 
     def hidden_class(i = nil)
-      %(govuk-tabs__panel--hidden) unless i&.zero?
+      return [] if i&.zero?
+
+      %w(govuk-tabs__panel--hidden)
     end
 
     def li_classes(i = nil)
@@ -42,8 +46,12 @@ private
       link_to(label, id(prefix: '#'), class: "govuk-tabs__tab")
     end
 
-    def default_classes
-      %w(govuk-tabs__panel)
+    def default_attributes
+      { id: id, class: %w(govuk-tabs__panel) }
+    end
+
+    def combined_attributes(i)
+      html_attributes.deep_merge_html_attributes({ class: hidden_class(i) }).deep_tidy_html_attributes
     end
 
     def call

data/app/components/govuk_component/table_component/body_component.html.erb

@@ -1,4 +1,4 @@
-<%= tag.tbody(class: classes, **html_attributes) do %>
+<%= tag.tbody(**html_attributes) do %>
   <% rows.each do |row| %>
     <%= row %>
   <% end %>

data/app/components/govuk_component/table_component/body_component.rb

@@ -15,7 +15,7 @@ private
     data.each { |d| row(cell_data: d, first_cell_is_header: first_cell_is_header) }
   end
 
-  def default_classes
-    %w(govuk-table__body)
+  def default_attributes
+    { class: %w(govuk-table__body) }
   end
 end

data/app/components/govuk_component/table_component/caption_component.rb

@@ -4,15 +4,15 @@ class GovukComponent::TableComponent::CaptionComponent < GovukComponent::Base
   SIZES = %w(s m l xl).freeze
 
   def initialize(text: nil, id: nil, size: 'm', classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @id   = id
     @text = text
     @size = size
+
+    super(classes: classes, html_attributes: html_attributes)
   end
 
   def call
-    tag.caption(caption_content, class: classes, **html_attributes)
+    tag.caption(caption_content, **html_attributes)
   end
 
   def render?
@@ -25,8 +25,8 @@ private
     @caption_content ||= (content || text)
   end
 
-  def default_classes
-    class_names("govuk-table__caption", caption_size_class => size).split
+  def default_attributes
+    { class: class_names("govuk-table__caption", caption_size_class => size).split }
   end
 
   def caption_size_class

data/app/components/govuk_component/table_component/cell_component.rb

@@ -13,12 +13,16 @@ class GovukComponent::TableComponent::CellComponent < GovukComponent::Base
   }.freeze
 
   def initialize(header: false, text: nil, numeric: false, width: nil, classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @header  = header
     @text    = text
     @numeric = numeric
     @width   = width
+
+    super(classes: classes, html_attributes: html_attributes)
+  end
+
+  def call
+    content_tag(cell_element, cell_content, **html_attributes)
   end
 
 private
@@ -35,6 +39,10 @@ private
     header ? :th : :td
   end
 
+  def default_attributes
+    { class: default_classes }
+  end
+
   def default_classes
     if header
       class_names("govuk-table__header", "govuk-table__header--numeric" => numeric?, width_class => width?).split

data/app/components/govuk_component/table_component/head_component.html.erb

@@ -1,4 +1,4 @@
-<%= tag.thead(class: classes, **html_attributes) do %>
+<%= tag.thead(**html_attributes) do %>
   <% rows.each do |row| %>
     <%= row %>
   <% end %>

data/app/components/govuk_component/table_component/head_component.rb

@@ -17,7 +17,7 @@ private
     data.each { |d| row(cell_data: d, header: true) }
   end
 
-  def default_classes
-    %w(govuk-table__head)
+  def default_attributes
+    { class: %w(govuk-table__head) }
   end
 end

data/app/components/govuk_component/table_component/row_component.html.erb

@@ -1,4 +1,4 @@
-<%= tag.tr(class: classes, **html_attributes) do %>
+<%= tag.tr(**html_attributes) do %>
   <% cells.each do |cell| %>
     <%= cell %>
   <% end %>

data/app/components/govuk_component/table_component/row_component.rb

@@ -4,11 +4,11 @@ class GovukComponent::TableComponent::RowComponent < GovukComponent::Base
   attr_reader :header, :first_cell_is_header
 
   def initialize(cell_data: nil, first_cell_is_header: false, header: false, classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @header = header
     @first_cell_is_header = first_cell_is_header
 
+    super(classes: classes, html_attributes: html_attributes)
+
     build_cells_from_cell_data(cell_data)
   end
 
@@ -24,7 +24,7 @@ private
     header || (first_cell_is_header && count.zero?)
   end
 
-  def default_classes
-    %w(govuk-table__row)
+  def default_attributes
+    { class: %w(govuk-table__row) }
   end
 end

data/app/components/govuk_component/table_component.html.erb

@@ -1,4 +1,4 @@
-<%= tag.table(id: id, class: classes, **html_attributes) do %>
+<%= tag.table(**html_attributes) do %>
   <%= caption %>
   <%= head %>
   <% bodies.each do |body| %>

data/app/components/govuk_component/table_component.rb

@@ -7,12 +7,12 @@ module GovukComponent
     attr_accessor :id, :first_cell_is_header, :caption_text
 
     def initialize(id: nil, rows: nil, head: nil, caption: nil, first_cell_is_header: false, classes: [], html_attributes: {})
-      super(classes: classes, html_attributes: html_attributes)
-
       @id                   = id
       @first_cell_is_header = first_cell_is_header
       @caption_text         = caption
 
+      super(classes: classes, html_attributes: html_attributes)
+
       # when no rows are passed in it's likely we're taking the slot approach
       return unless rows.presence
 
@@ -28,8 +28,8 @@ module GovukComponent
       body(rows: body_data, first_cell_is_header: first_cell_is_header)
     end
 
-    def default_classes
-      %w(govuk-table)
+    def default_attributes
+      { id: id, class: %w(govuk-table) }
     end
   end
 end

data/app/components/govuk_component/tag_component.rb

@@ -4,14 +4,14 @@ class GovukComponent::TagComponent < GovukComponent::Base
   COLOURS = %w(grey green turquoise blue red purple pink orange yellow).freeze
 
   def initialize(text: nil, colour: nil, classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @text   = text
     @colour = colour
+
+    super(classes: classes, html_attributes: html_attributes)
   end
 
   def call
-    tag.strong(tag_content, class: classes.append(colour_class), **html_attributes)
+    tag.strong(tag_content, **html_attributes)
   end
 
 private
@@ -20,8 +20,10 @@ private
     @text || content || fail(ArgumentError, "no text or content")
   end
 
-  def default_classes
-    %w(govuk-tag)
+  def default_attributes
+    {
+      class: ["govuk-tag", colour_class]
+    }
   end
 
   def colour_class

data/app/components/govuk_component/warning_text_component.rb

@@ -4,14 +4,14 @@ class GovukComponent::WarningTextComponent < GovukComponent::Base
   ICON = '!'.freeze
 
   def initialize(text: nil, icon_fallback_text: 'Warning', classes: [], html_attributes: {})
-    super(classes: classes, html_attributes: html_attributes)
-
     @text = text
     @icon_fallback_text = icon_fallback_text
+
+    super(classes: classes, html_attributes: html_attributes)
   end
 
   def call
-    tag.div(class: classes, **html_attributes) do
+    tag.div(**html_attributes) do
       safe_join([icon, warning_text])
     end
   end
@@ -32,7 +32,7 @@ private
     tag.span(icon_fallback_text, class: 'govuk-warning-text__assistive')
   end
 
-  def default_classes
-    %w(govuk-warning-text)
+  def default_attributes
+    { class: %w(govuk-warning-text) }
   end
 end

data/app/helpers/govuk_components_helper.rb

@@ -9,8 +9,10 @@ module GovukComponentsHelper
     govuk_header: 'GovukComponent::HeaderComponent',
     govuk_inset_text: 'GovukComponent::InsetTextComponent',
     govuk_notification_banner: 'GovukComponent::NotificationBannerComponent',
+    govuk_pagination: 'GovukComponent::PaginationComponent',
     govuk_panel: 'GovukComponent::PanelComponent',
     govuk_phase_banner: 'GovukComponent::PhaseBannerComponent',
+    govuk_section_break: 'GovukComponent::SectionBreakComponent',
     govuk_start_button: 'GovukComponent::StartButtonComponent',
     govuk_summary_list: 'GovukComponent::SummaryListComponent',
     govuk_table: 'GovukComponent::TableComponent',

data/app/helpers/govuk_link_helper.rb

@@ -68,7 +68,8 @@ module GovukLinkHelper
 
   def govuk_button_link_to(name = nil, options = nil, extra_options = {}, &block)
     extra_options = options if block_given?
-    html_options = build_html_options(extra_options, style: :button)
+    html_options = GovukComponent::StartButtonComponent::BUTTON_ATTRIBUTES
+      .merge build_html_options(extra_options, style: :button)
 
     if block_given?
       link_to(name, html_options, &block)

data/lib/govuk/components/version.rb

@@ -1,5 +1,5 @@
 module Govuk
   module Components
-    VERSION = '3.0.4'.freeze
+    VERSION = '3.1.0'.freeze
   end
 end

data/lib/govuk/components.rb

@@ -2,3 +2,4 @@ require "rails/engine"
 require "view_component"
 require "govuk/components/helpers/css_utilities"
 require "govuk/components/engine"
+require "html_attributes_utils"

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: govuk-components
 version: !ruby/object:Gem::Version
-  version: 3.0.4
+  version: 3.1.0
 platform: ruby
 authors:
 - DfE developers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-05-19 00:00:00.000000000 Z
+date: 2022-06-27 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: actionpack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
@@ -38,20 +52,54 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '6.1'
+- !ruby/object:Gem::Dependency
+  name: html-attributes-utils
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.9.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.9.2
 - !ruby/object:Gem::Dependency
   name: view_component
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.49.1
+        version: 2.56.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.49.1
+        version: 2.56.2
+- !ruby/object:Gem::Dependency
+  name: deep_merge
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
@@ -178,6 +226,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '4.11'
+- !ruby/object:Gem::Dependency
+  name: pagy
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 5.10.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 5.10.1
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
@@ -198,14 +260,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.28.0
+        version: 3.29.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.28.0
+        version: 3.29.0
 - !ruby/object:Gem::Dependency
   name: rubypants
   requirement: !ruby/object:Gem::Requirement
@@ -320,9 +382,15 @@ files:
 - app/components/govuk_component/inset_text_component.rb
 - app/components/govuk_component/notification_banner_component.html.erb
 - app/components/govuk_component/notification_banner_component.rb
+- app/components/govuk_component/pagination_component.rb
+- app/components/govuk_component/pagination_component/adjacent_page.rb
+- app/components/govuk_component/pagination_component/item.rb
+- app/components/govuk_component/pagination_component/next_page.rb
+- app/components/govuk_component/pagination_component/previous_page.rb
 - app/components/govuk_component/panel_component.rb
 - app/components/govuk_component/phase_banner_component.html.erb
 - app/components/govuk_component/phase_banner_component.rb
+- app/components/govuk_component/section_break_component.rb
 - app/components/govuk_component/start_button_component.rb
 - app/components/govuk_component/summary_list_component.html.erb
 - app/components/govuk_component/summary_list_component.rb
@@ -337,7 +405,6 @@ files:
 - app/components/govuk_component/table_component/body_component.html.erb
 - app/components/govuk_component/table_component/body_component.rb
 - app/components/govuk_component/table_component/caption_component.rb
-- app/components/govuk_component/table_component/cell_component.html.erb
 - app/components/govuk_component/table_component/cell_component.rb
 - app/components/govuk_component/table_component/head_component.html.erb
 - app/components/govuk_component/table_component/head_component.rb
@@ -345,7 +412,6 @@ files:
 - app/components/govuk_component/table_component/row_component.rb
 - app/components/govuk_component/tag_component.rb
 - app/components/govuk_component/traits.rb
-- app/components/govuk_component/traits/custom_classes.rb
 - app/components/govuk_component/traits/custom_html_attributes.rb
 - app/components/govuk_component/warning_text_component.rb
 - app/helpers/govuk_back_to_top_link_helper.rb

data/app/components/govuk_component/table_component/cell_component.html.erb

@@ -1 +0,0 @@
-<%= content_tag(cell_element, cell_content, class: classes, **html_attributes) %>

data/app/components/govuk_component/traits/custom_classes.rb

@@ -1,28 +0,0 @@
-module GovukComponent
-  module Traits
-    module CustomClasses
-      def classes
-        default_classes.concat(Array.wrap(@classes))
-      end
-
-      def default_classes
-        Rails.logger.warn(%(#default_classes hasn't been defined in #{self.class}))
-
-        []
-      end
-
-    private
-
-      def parse_classes(classes)
-        return [] if classes.blank?
-
-        case classes
-        when Array
-          classes
-        when String
-          classes.split
-        end
-      end
-    end
-  end
-end