From 47f28a4a5535a6bd2c47ebb4eb0dcd174fea0ffd Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Thu, 13 Jan 2022 13:47:12 +0000
Subject: [PATCH] PR accessibility comments

---
 app/helpers/tasklist_helper.rb         |  2 +-
 app/views/case_logs/_tasklist.html.erb | 11 +++++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/app/helpers/tasklist_helper.rb b/app/helpers/tasklist_helper.rb
index fa4543d25..148f2f1e7 100644
--- a/app/helpers/tasklist_helper.rb
+++ b/app/helpers/tasklist_helper.rb
@@ -37,7 +37,7 @@ module TasklistHelper
   def subsection_link(subsection, case_log)
     if subsection.status(case_log) != :cannot_start_yet
       next_page_path = first_page_or_check_answers(subsection, case_log).to_s
-      govuk_link_to(subsection.label, next_page_path.dasherize, class: "task-name")
+      govuk_link_to(subsection.label, next_page_path.dasherize, aria: { describedby: subsection.id.dasherize })
     else
       subsection.label
     end
diff --git a/app/views/case_logs/_tasklist.html.erb b/app/views/case_logs/_tasklist.html.erb
index 6765d6f82..85a8d6349 100644
--- a/app/views/case_logs/_tasklist.html.erb
+++ b/app/views/case_logs/_tasklist.html.erb
@@ -8,10 +8,13 @@
       </h2>
       <ul class="app-task-list__items">
         <% section.subsections.map do |subsection| %>
-          <li class="app-task-list__item" id=<%= subsection.id %>>
-            <% subsection_status = subsection.status(@case_log) %>
-            <%= subsection_link(subsection, @case_log) %>
-            <strong class="govuk-tag app-task-list__tag <%= TasklistHelper::STYLES[subsection_status] %>">
+          <% subsection_status = subsection.status(@case_log) %>
+          <li class="app-task-list__item">
+            <span class="app-task-list__task-name">
+              <%= subsection_link(subsection, @case_log) %>
+            </span>
+            <strong class="govuk-tag app-task-list__tag <%= TasklistHelper::STYLES[subsection_status] %>"
+                    id=<%= subsection.id.dasherize %>>
               <%= TasklistHelper::STATUSES[subsection_status] %>
             </strong>
           </li>