Fix usages of aria-labelledby

These now conform to the W3C spec. https://www.w3.org/TR/wai-aria/#region
alphagov · Mar 26, 2020 · f9d46f6 · f9d46f6
1 parent a097b81
commit f9d46f6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/app/views/content_items/_attachments.html.erb b/app/views/content_items/_attachments.html.erb
@@ -1,12 +1,12 @@
 <% attachment_title_id = "#{title.parameterize}-title" %>
 
 <% if legacy_pre_rendered_documents.present? || attachments.any? %>
-  <%= render 'govuk_publishing_components/components/heading',
-      id: attachment_title_id,
-      text: title,
-      mobile_top_margin: true %>
+  <div role="region" aria-labelledby="<%= attachment_title_id %>">
+    <%= render 'govuk_publishing_components/components/heading',
+        id: attachment_title_id,
+        text: title,
+        mobile_top_margin: true %>
 
-  <div aria-labelledby="<%= attachment_title_id %>">
     <% if legacy_pre_rendered_documents.present? %>
       <%= render 'govuk_publishing_components/components/govspeak',
                  content: legacy_pre_rendered_documents.html_safe,

diff --git a/app/views/content_items/publication.html.erb b/app/views/content_items/publication.html.erb
@@ -34,12 +34,13 @@
           legacy_pre_rendered_documents: @content_item.documents,
           attachments: @content_item.featured_attachments %>
 
-      <%= render 'govuk_publishing_components/components/heading',
-        text: t("publication.details"),
-        id: "details-title",
-        mobile_top_margin: true %>
 
-      <div aria-labelledby="details-title">
+      <div role="region" aria-labelledby="details-title">
+        <%= render 'govuk_publishing_components/components/heading',
+          text: t("publication.details"),
+          id: "details-title",
+          mobile_top_margin: true %>
+
         <%= render 'govuk_publishing_components/components/govspeak',
                    content: @content_item.details.html_safe,
                    direction: page_text_direction %>