WIP: flatten text property

Need to consolidate and/or remove: - heading_size (in and outside of label property) - bold - required
alphagov · Feb 18, 2025 · 9cd1601 · 9cd1601
1 parent 2140ccb
commit 9cd1601
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/app/views/admin/detailed_guides/_form.html.erb b/app/views/admin/detailed_guides/_form.html.erb
@@ -22,7 +22,7 @@
           include_blank: true,
           label: {
             text: "Related guides",
-            heading_size: "m",
+            heading_size: "m", # TODO medium
           },
           select: {
             options: taggable_detailed_guides_container,

diff --git a/app/views/admin/historical_accounts/_form.html.erb b/app/views/admin/historical_accounts/_form.html.erb
@@ -22,7 +22,7 @@
         error_items: errors_for(historical_account.errors, :political_parties),
         label: {
           text: "Political parties (required)",
-          heading_size: "l",
+          heading_size: "l", # TODO large
         },
         select: {
           options: PoliticalParty.all.map do |party|

diff --git a/app/views/admin/needs/edit.html.erb b/app/views/admin/needs/edit.html.erb
@@ -11,10 +11,10 @@
           include_blank: true,
           label: {
             text: "Select associated user needs",
-            bold: true,
-            required: true,
+            bold: true, # TODO bold
+            required: true, # TODO required
           },
-          heading_size: "l",
+          heading_size: "l", # TODO separate heading size??
           select: {
             options: taggable_needs_container,
             multiple: true,