From fdee6785d49d371a241abebd87f73c8de54afab4 Mon Sep 17 00:00:00 2001
From: rkuehn-uofl <randy.kuehn@louisville.edu>
Date: Wed, 30 Oct 2024 09:12:43 -0400
Subject: [PATCH] Add presentation role to list items

Accessibility fix for issues #6797 and #6809
---
 app/views/hyrax/dashboard/collections/_form.html.erb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/views/hyrax/dashboard/collections/_form.html.erb b/app/views/hyrax/dashboard/collections/_form.html.erb
index b59eb4b73f..5e8c87441d 100644
--- a/app/views/hyrax/dashboard/collections/_form.html.erb
+++ b/app/views/hyrax/dashboard/collections/_form.html.erb
@@ -1,28 +1,28 @@
 <%= render "shared/nav_safety_modal" %>
 <div class="tabs mt-4" id="collection-edit-controls">
   <ul class="nav nav-tabs" id="dashboard-collection-tab" role="tablist">
-    <li class="nav-item">
+    <li class="nav-item" role="presentation">
       <a href="#description" role="tab" data-toggle="tab" class="nav-link active nav-safety-confirm">
         <%= t('.tabs.description') %>
       </a>
     </li>
     <% if @form.persisted? %>
       <% if collection_brandable?(collection: @collection) %>
-      <li class="nav-item">
+      <li class="nav-item" role="presentation">
         <a href="#branding" role="tab" data-toggle="tab" class="nav-link nav-safety-confirm">
           <%= t('.tabs.branding') %>
         </a>
       </li>
       <% end %>
       <% if collection_discoverable?(collection: @collection) %>
-      <li class="nav-item">
+      <li class="nav-item" role="presentation">
         <a href="#discovery" role="tab" data-toggle="tab" class="nav-link nav-safety-confirm">
           <%= t('.tabs.discovery') %>
         </a>
       </li>
       <% end %>
       <% if collection_sharable?(collection: @collection) %>
-      <li class="nav-item">
+      <li class="nav-item" role="presentation">
         <a href="#sharing" role="tab" data-toggle="tab" class="nav-link nav-safety-confirm">
           <%= t('.tabs.sharing') %>
         </a>