From df7aa25572576b3088dc2348e70c629678d2610c Mon Sep 17 00:00:00 2001
From: Kat <katrina@kosiak.co.uk>
Date: Wed, 8 May 2024 15:59:58 +0100
Subject: [PATCH] Set label size on filter questions

---
 app/views/filters/_checkbox_filter.html.erb    | 2 +-
 app/views/filters/_radio_filter.html.erb       | 2 +-
 app/views/locations/_location_filters.html.erb | 1 +
 app/views/logs/_log_filters.html.erb           | 7 +++++++
 app/views/schemes/_scheme_filters.html.erb     | 2 ++
 app/views/users/_user_filters.html.erb         | 1 +
 6 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/app/views/filters/_checkbox_filter.html.erb b/app/views/filters/_checkbox_filter.html.erb
index d08c42a77..d0c88c5ac 100644
--- a/app/views/filters/_checkbox_filter.html.erb
+++ b/app/views/filters/_checkbox_filter.html.erb
@@ -1,4 +1,4 @@
-<%= f.govuk_check_boxes_fieldset category.to_sym, legend: { text: label, size: "s" }, small: true, form_group: { classes: "app-filter__group" } do %>
+<%= f.govuk_check_boxes_fieldset category.to_sym, legend: { text: label, size: }, small: true, form_group: { classes: "app-filter__group" } do %>
   <% options.map do |key, option| %>
     <%= f.govuk_check_box category, key.to_s,
     label: { text: option },
diff --git a/app/views/filters/_radio_filter.html.erb b/app/views/filters/_radio_filter.html.erb
index d1a567ff5..6eb902dba 100644
--- a/app/views/filters/_radio_filter.html.erb
+++ b/app/views/filters/_radio_filter.html.erb
@@ -1,4 +1,4 @@
-<%= f.govuk_radio_buttons_fieldset category.to_sym, legend: { text: label, size: "s" }, small: true, form_group: { classes: "app-filter__group" } do %>
+<%= f.govuk_radio_buttons_fieldset category.to_sym, legend: { text: label, size: }, small: true, form_group: { classes: "app-filter__group" } do %>
   <% options.map do |key, option| %>
     <%= f.govuk_radio_button category, key.to_s,
       label: { text: option[:label] },
diff --git a/app/views/locations/_location_filters.html.erb b/app/views/locations/_location_filters.html.erb
index f19d8a17b..aa5b2faa9 100644
--- a/app/views/locations/_location_filters.html.erb
+++ b/app/views/locations/_location_filters.html.erb
@@ -21,6 +21,7 @@
                     options: location_status_filters,
                     label: "Status",
                     category: "status",
+                    size: "s",
                   } %>
 
         <% if request.params["search"].present? %>
diff --git a/app/views/logs/_log_filters.html.erb b/app/views/logs/_log_filters.html.erb
index ea7496bdd..aaef70377 100644
--- a/app/views/logs/_log_filters.html.erb
+++ b/app/views/logs/_log_filters.html.erb
@@ -22,6 +22,7 @@
                        options: bulk_upload_options(@bulk_upload),
                        label: "Bulk upload",
                        category: "bulk_upload_id",
+                       size: "s",
                      } %>
         <% end %>
 
@@ -32,6 +33,7 @@
                        options: collection_year_options,
                        label: "Collection year",
                        category: "years",
+                       size: "s",
                      } %>
 
           <%= render partial: "filters/checkbox_filter",
@@ -40,6 +42,7 @@
                        options: status_filters,
                        label: "Status",
                        category: "status",
+                       size: "s",
                      } %>
 
           <% if logs_for_both_needstypes_present?(@organisation) && user_or_org_lettings_path? %>
@@ -49,6 +52,7 @@
                          options: needstype_filters,
                          label: "Needs type",
                          category: "needstypes",
+                         size: "s",
                        } %>
           <% end %>
         <% end %>
@@ -71,6 +75,7 @@
                      },
                      label: "Assigned to",
                      category: "assigned_to",
+                     size: "s",
                    } %>
 
          <% if current_user.support? || non_support_with_multiple_owning_orgs? %>
@@ -90,6 +95,7 @@
             },
             label: "Owned by",
             category: "owning_organisation_select",
+            size: "s",
           } %>
         <% end %>
 
@@ -110,6 +116,7 @@
             },
             label: user_or_org_lettings_path? ? "Managed by" : "Reported by",
             category: "managing_organisation_select",
+            size: "s",
           } %>
         <% end %>
 
diff --git a/app/views/schemes/_scheme_filters.html.erb b/app/views/schemes/_scheme_filters.html.erb
index af9ed093f..ca0538463 100644
--- a/app/views/schemes/_scheme_filters.html.erb
+++ b/app/views/schemes/_scheme_filters.html.erb
@@ -21,6 +21,7 @@
                     options: scheme_status_filters,
                     label: "Status",
                     category: "status",
+                    size: "s",
                   } %>
 
          <% if show_scheme_managing_org_filter?(current_user) %>
@@ -40,6 +41,7 @@
             },
             label: "Owned by",
             category: "owning_organisation_select",
+            size: "s",
           } %>
         <% end %>
 
diff --git a/app/views/users/_user_filters.html.erb b/app/views/users/_user_filters.html.erb
index d047beac7..aa9185c8d 100644
--- a/app/views/users/_user_filters.html.erb
+++ b/app/views/users/_user_filters.html.erb
@@ -21,6 +21,7 @@
                     options: user_status_filters,
                     label: "Status",
                     category: "status",
+                    size: "s",
                   } %>
 
         <% if request.params["search"].present? %>