Remove tabindex attribute from app-table-group sections for accessibility improvements

7 months ago · 95d2680196
5 changed files with 5 additions and 5 deletions
--- a/app/views/merge_requests/_merge_request_list.html.erb
+++ b/app/views/merge_requests/_merge_request_list.html.erb
@ -1,4 +1,4 @@
-<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
+<section class="app-table-group" aria-labelledby="<%= title.dasherize %>">
  <% if @merge_requests.empty? %>
    <p>No merge requests</p>
  <% else %>
--- a/app/views/organisation_relationships/_related_org_list.erb
+++ b/app/views/organisation_relationships/_related_org_list.erb
@ -1,4 +1,4 @@
-<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
+<section class="app-table-group" aria-labelledby="<%= title.dasherize %>">
  <%= govuk_table do |table| %>
    <%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %>
      <%= render(SearchResultCaptionComponent.new(searched:, count: pagy.count, item_label:, total_count:, item: search_item, filters_count: 0)) %>
--- a/app/views/organisations/_organisation_list.html.erb
+++ b/app/views/organisations/_organisation_list.html.erb
@ -1,4 +1,4 @@
-<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
+<section class="app-table-group" aria-labelledby="<%= title.dasherize %>">
  <%= govuk_table do |table| %>
    <%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %>
      <%= render(SearchResultCaptionComponent.new(searched:, count: pagy.count, item_label:, total_count:, item: "organisation", filters_count: applied_filters_count(@filter_type))) %>
--- a/app/views/schemes/_scheme_list.html.erb
+++ b/app/views/schemes/_scheme_list.html.erb
@ -1,4 +1,4 @@
-<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
+<section class="app-table-group" aria-labelledby="<%= title.dasherize %>">
  <%= govuk_table do |table| %>
    <%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %>
    <span class="app-search__caption">
--- a/app/views/users/_user_list.html.erb
+++ b/app/views/users/_user_list.html.erb
@ -1,4 +1,4 @@
-<section class="app-table-group" tabindex="0" aria-labelledby="<%= title.dasherize %>">
+<section class="app-table-group" aria-labelledby="<%= title.dasherize %>">
  <%= govuk_table do |table| %>
    <%= table.with_caption(classes: %w[govuk-!-font-size-19 govuk-!-font-weight-regular]) do |caption| %>
     <%= render(SearchResultCaptionComponent.new(searched:, count: pagy.count, item_label:, total_count:, item: "user", filters_count: applied_filters_count(@filter_type))) %>