hyrax 2.2.0 → 2.2.1

data/app/views/hyrax/dashboard/collections/_work_action_menu.html.erb

@@ -28,12 +28,12 @@
 
     <li role="menuitem" tabindex="-1">
       <%= display_trophy_link @user, document.id do |text| %>
-        <i class='glyphicon glyphicon-star'></i> <%= text %>
+        <i class="glyphicon glyphicon-star" aria-hidden="true"></i> <%= text %>
       <% end %>
     </li>
     <li role="menuitem" tabindex="-1">
       <%= link_to(hyrax.new_work_transfer_path(document.id), class: 'itemicon itemtransfer', title: 'Transfer Ownership of Work') do %>
-        <i aria-hidden="true" class='glyphicon glyphicon-transfer'></i>
+        <i aria-hidden="true" class="glyphicon glyphicon-transfer"></i>
         <span> Transfer Ownership of Work </span>
       <% end %>
     </li>

data/app/views/hyrax/dashboard/collections/edit.html.erb

@@ -1,7 +1,7 @@
 <% provide :page_title, construct_page_title( t('.header', type_title: @collection.collection_type.title, title: @form.title.first) ) %>
 
 <% provide :page_header do %>
-  <h1><span class="fa fa-edit"></span><%= t('.header', type_title: @collection.collection_type.title, title: @form.title.first) %></h1>
+  <h1><span class="fa fa-edit" aria-hidden="true"></span><%= t('.header', type_title: @collection.collection_type.title, title: @form.title.first) %></h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/dashboard/collections/new.html.erb

@@ -1,6 +1,6 @@
 <% provide :page_title, construct_page_title( t('.header', type_title: @collection.collection_type.title) ) %>
 <% provide :page_header do %>
-  <h1><span class="fa fa-edit"></span> <%= t('.header', type_title: @collection.collection_type.title) %> </h1>
+  <h1><span class="fa fa-edit" aria-hidden="true"></span> <%= t('.header', type_title: @collection.collection_type.title) %> </h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/dashboard/collections/show.html.erb

@@ -1,7 +1,7 @@
 <% provide :page_title, construct_page_title(@presenter.title) %>
 
 <% provide :page_header do %>
-  <h1><span class="fa fa-file"></span> <%= t('.header') %></h1>
+  <h1><span class="fa fa-file" aria-hidden="true"></span> <%= t('.header') %></h1>
 <% end %>
 
 <div class="collections-wrapper">

data/app/views/hyrax/dashboard/profiles/_edit_primary.html.erb

@@ -33,14 +33,14 @@
   </div><!-- .form-group -->
 
   <div class="form-group">
-    <%= f.label :twitter_handle, '<i class="fa fa-twitter"></i> Twitter Handle'.html_safe, class: 'col-xs-4 control-label' %>
+    <%= f.label :twitter_handle, '<i class="fa fa-twitter" aria-hidden="true"></i> Twitter Handle'.html_safe, class: 'col-xs-4 control-label' %>
     <div class="col-xs-8">
        <%= f.text_field :twitter_handle, class: "form-control" %>
     </div>
   </div><!-- .form-group -->
 
   <div class="form-group">
-    <%= f.label :facebook_handle, '<i class="fa fa-facebook"></i> Facebook Handle'.html_safe, class: 'col-xs-4 control-label' %>
+    <%= f.label :facebook_handle, '<i class="fa fa-facebook" aria-hidden="true"></i> Facebook Handle'.html_safe, class: 'col-xs-4 control-label' %>
     <div class="col-xs-8">
        <%= f.text_field :facebook_handle, class: "form-control" %>
     </div>

data/app/views/hyrax/dashboard/profiles/edit.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-id-card"></span> Edit Profile</h1>
+  <h1><span class="fa fa-id-card" aria-hidden="true"></span> Edit Profile</h1>
 <% end %>
 <div class="panel panel-default">
   <div class="panel-body">

data/app/views/hyrax/dashboard/profiles/show.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-id-card"></span> <%= t("hyrax.admin.sidebar.profile") %></h1>
+  <h1><span class="fa fa-id-card" aria-hidden="true"></span> <%= t("hyrax.admin.sidebar.profile") %></h1>
   <% if can? :edit, current_user %>
     <div class="pull-right">
       <%= link_to hyrax.edit_dashboard_profile_path(@user), class: "btn btn-primary" do %>

data/app/views/hyrax/dashboard/sidebar/_activity.html.erb

@@ -7,20 +7,20 @@
                                  open: menu.user_activity_section? do %>
       <%= menu.nav_link(hyrax.dashboard_profile_path(current_user),
                         also_active_for: hyrax.edit_dashboard_profile_path(current_user)) do %>
-        <span class="fa fa-id-card"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.profile') %></span>
+        <span class="fa fa-id-card" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.profile') %></span>
       <% end %>
 
       <%= menu.nav_link(hyrax.notifications_path) do %>
-        <span class="fa fa-bell"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.notifications') %></span>
+        <span class="fa fa-bell" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.notifications') %></span>
       <% end %>
 
       <%= menu.nav_link(hyrax.transfers_path) do %>
-        <span class="fa fa-arrows-h"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.transfers') %></span>
+        <span class="fa fa-arrows-h" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.transfers') %></span>
       <% end %>
 
       <% if Flipflop.proxy_deposit? %>
         <%= menu.nav_link(hyrax.depositors_path) do %>
-          <span class="fa fa-users"></span> <span class="sidebar-action-text"><%= t('hyrax.dashboard.manage_proxies') %></span>
+          <span class="fa fa-users" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.dashboard.manage_proxies') %></span>
         <% end %>
       <% end %>
     <% end %>
@@ -28,6 +28,6 @@
 
   <% if can? :read, :admin_dashboard %>
     <%= menu.nav_link(hyrax.admin_stats_path) do %>
-      <span class="fa fa-bar-chart"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.statistics') %></span>
+      <span class="fa fa-bar-chart" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.statistics') %></span>
     <% end %>
   <% end %>

data/app/views/hyrax/dashboard/sidebar/_configuration.html.erb

@@ -7,30 +7,30 @@
                                    open: menu.settings_section? do %>
         <% if can?(:update, :appearance) %>
           <%= menu.nav_link(hyrax.admin_appearance_path) do %>
-            <span class="fa fa-paint-brush"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.appearance') %></span>
+            <span class="fa fa-paint-brush" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.appearance') %></span>
           <% end %>
         <% end %>
         <% if can?(:manage, :collection_types) %>
           <%= menu.nav_link(hyrax.admin_collection_types_path) do %>
-            <span class="fa fa-folder-open"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collection_types') %></span>
+            <span class="fa fa-folder-open" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collection_types') %></span>
           <% end %>
         <% end %>
         <% if can?(:manage, Hyrax::Feature) %>
           <%= menu.nav_link(hyrax.edit_pages_path) do %>
-            <span class="fa fa-file-text-o"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.pages') %></span>
+            <span class="fa fa-file-text-o" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.pages') %></span>
           <% end %>
           <%= menu.nav_link(hyrax.edit_content_blocks_path) do %>
-            <span class="fa fa-square-o"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.content_blocks') %></span>
+            <span class="fa fa-square-o" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.content_blocks') %></span>
           <% end %>
           <%= menu.nav_link(hyrax.admin_features_path) do %>
-            <span class="fa fa-wrench"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.technical') %></span>
+            <span class="fa fa-wrench" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.technical') %></span>
           <% end %>
         <% end %>
       <% end %>
     </li>
     <% if can?(:manage, Sipity::WorkflowResponsibility) %>
       <%= menu.nav_link(hyrax.admin_workflow_roles_path) do %>
-        <span class="fa fa-users"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.workflow_roles') %></span>
+        <span class="fa fa-users" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.workflow_roles') %></span>
       <% end %>
     <% end # end of configuration block %>
   <% end %>

data/app/views/hyrax/dashboard/sidebar/_repository_content.html.erb

@@ -2,10 +2,10 @@
 
   <%= menu.nav_link(hyrax.my_collections_path,
                     also_active_for: hyrax.dashboard_collections_path) do %>
-    <span class="fa fa-folder-open"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collections') %></span>
+    <span class="fa fa-folder-open" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collections') %></span>
   <% end %>
 
   <%= menu.nav_link(hyrax.my_works_path,
                     also_active_for: hyrax.dashboard_works_path) do %>
-    <span class="fa fa-file"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.works') %></span>
+    <span class="fa fa-file" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.works') %></span>
   <% end %>

data/app/views/hyrax/dashboard/sidebar/_tasks.html.erb

@@ -1,22 +1,22 @@
   <% if can? :review, :submissions %>
     <li class="h5"><%= t('hyrax.admin.sidebar.tasks') %></li>
     <%= menu.nav_link(hyrax.admin_workflows_path) do %>
-      <span class="fa fa-flag"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.workflow_review') %></span>
+      <span class="fa fa-flag" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.workflow_review') %></span>
     <% end %>
   <% end %>
 
   <% if can? :manage, User %>
     <%= menu.nav_link(hyrax.admin_users_path) do %>
-      <span class="fa fa-user"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.users') %></span>
+      <span class="fa fa-user" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.users') %></span>
     <% end %>
   <% end %>
 
   <% if can? :read, :admin_dashboard %>
     <%= menu.nav_link(hyrax.embargoes_path) do %>
-      <span class="fa fa-flag"></span> <span class="sidebar-action-text"><%= t('hyrax.embargoes.index.manage_embargoes') %></span>
+      <span class="fa fa-flag" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.embargoes.index.manage_embargoes') %></span>
     <% end %>
 
     <%= menu.nav_link(hyrax.leases_path) do %>
-      <span class="fa fa-flag"></span> <span class="sidebar-action-text"><%= t('hyrax.leases.index.manage_leases') %></span>
+      <span class="fa fa-flag" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.leases.index.manage_leases') %></span>
     <% end %>
   <% end %>

data/app/views/hyrax/depositors/index.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-users"></span>  <%= t("hyrax.dashboard.manage_proxies") %></h1>
+  <h1><span class="fa fa-users" aria-hidden="true"></span>  <%= t("hyrax.dashboard.manage_proxies") %></h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/embargoes/index.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-sitemap"></span><%= t('.manage_embargoes') %></h1>
+  <h1><span class="fa fa-sitemap" aria-hidden="true"></span><%= t('.manage_embargoes') %></h1>
 <% end %>
 
 <div class="panel panel-default tabs">

data/app/views/hyrax/file_sets/edit.html.erb

@@ -1,6 +1,6 @@
 <% provide :page_title, curation_concern_page_title(curation_concern) %>
 <% provide :page_header do %>
-  <h1><span class="fa fa-edit"></span><%= t('.header', file_set: curation_concern) %></h1>
+  <h1><span class="fa fa-edit" aria-hidden="true"></span><%= t('.header', file_set: curation_concern) %></h1>
 <% end %>
 
 <div class="row">
@@ -11,13 +11,13 @@
     <div class="panel panel-default tabs">
       <ul class="nav nav-tabs" role="tablist">
         <li id="edit_descriptions_link" class="active">
-          <a href="#descriptions_display" data-toggle="tab"><i class="fa fa-tags"></i></span> <%= t('.descriptions') %></a>
+          <a href="#descriptions_display" data-toggle="tab"><i class="fa fa-tags" aria-hidden="true"></i></span> <%= t('.descriptions') %></a>
         </li>
         <li id="edit_versioning_link">
-          <a href="#versioning_display" data-toggle="tab"><i class="fa fa-sitemap"></i> <%= t('.versions') %></a>
+          <a href="#versioning_display" data-toggle="tab"><i class="fa fa-sitemap" aria-hidden="true"></i> <%= t('.versions') %></a>
         </li>
         <li id="edit_permissions_link">
-          <a href="#permissions_display" data-toggle="tab"><i class="fa fa-key"></i> <%= t('.permissions') %></a>
+          <a href="#permissions_display" data-toggle="tab"><i class="fa fa-key" aria-hidden="true"></i> <%= t('.permissions') %></a>
         </li>
       </ul>
       <div class="panel-body">

data/app/views/hyrax/homepage/index.html.erb

@@ -8,18 +8,18 @@
           <%= link_to '#',
             class: "btn btn-primary btn-lg",
             data: { behavior: 'select-work', target: '#worktypes-to-create', 'create-type' => 'single' } do %>
-            <i class="glyphicon glyphicon-upload"></i> <%= t('hyrax.share_button') %>
+            <i class="glyphicon glyphicon-upload" aria-hidden="true"></i> <%= t('hyrax.share_button') %>
           <% end %>
         <% else # simple link to the first work type %>
           <%= link_to new_polymorphic_path([main_app, @presenter.first_work_type]),
                 class: 'btn btn-primary' do %>
-            <i class="glyphicon glyphicon-upload"></i> <%= t('hyrax.share_button') %>
+            <i class="glyphicon glyphicon-upload" aria-hidden="true"></i> <%= t('hyrax.share_button') %>
           <% end %>
         <% end %>
       <% else %>
         <%= link_to hyrax.my_works_path,
           class: "btn btn-primary btn-lg" do %>
-          <i class="glyphicon glyphicon-upload"></i> <%= t('hyrax.share_button') %>
+          <i class="glyphicon glyphicon-upload" aria-hidden="true"></i> <%= t('hyrax.share_button') %>
         <% end %>
       <% end %>
       <p><%= link_to t(:'hyrax.pages.tabs.terms_page'), hyrax.terms_path %></p>

data/app/views/hyrax/my/_admin_set_action_menu.html.erb

@@ -1,10 +1,12 @@
 <% id = admin_set_presenter.id %>
+<% ul_id = 'admin-set-action-dropdown-ul-' + id %>
+
 <div class="btn-group">
-  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= id %>" aria-haspopup="true">
+  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= id %>" aria-haspopup="true" aria-expanded="false" aria-controls="<%= ul_id %>">
     <span class="sr-only"><%= t("hyrax.dashboard.my.sr.press_to") %> </span>
     <%= t("hyrax.dashboard.my.action.select") %> <span class="caret" aria-hidden="true"></span>
   </button>
-  <ul role="menu" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= id %>">
+  <ul role="menu" id="<%= ul_id %>" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= id %>">
     <li role="menuitem" tabindex="-1">
       <%= link_to hyrax.admin_admin_set_path(id),
                   class: 'itemicon itemedit',

data/app/views/hyrax/my/_collection_action_menu.html.erb

@@ -1,12 +1,13 @@
 <% id = collection_presenter.id %>
-<div class="btn-group">
+<% ul_id = 'collection-action-dropdown-ul-' + id %>
 
-  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= id %>" aria-haspopup="true">
+<div class="btn-group">
+  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= id %>" aria-haspopup="true" aria-expanded="false" aria-controls="<%= ul_id %>">
     <span class="sr-only"><%= t("hyrax.dashboard.my.sr.press_to") %> </span>
     <%= t("hyrax.dashboard.my.action.select") %> <span class="caret" aria-hidden="true"></span>
   </button>
 
-  <ul role="menu" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= id %>">
+  <ul role="menu" id="<%= ul_id %>" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= id %>">
     <li role="menuitem" tabindex="-1">
       <%= link_to hyrax.dashboard_collection_path(id),
                   class: 'itemicon itemedit',

data/app/views/hyrax/my/_facet_layout.html.erb

@@ -1,5 +1,5 @@
 <div class="btn-group">
-  <button class="btn btn-default dropdown-toggle" type="button" id="<%= facet_field.field.parameterize %>" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
+  <button class="btn btn-default dropdown-toggle" type="button" id="<%= facet_field.field.parameterize %>" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true" aria-controls="<%= facet_field.field.parameterize + '-dropdown-options' %>">
     <%= facet_field_label(facet_field.field) %>
     <span class="caret"></span>
   </button>

data/app/views/hyrax/my/_facet_limit.html.erb

@@ -1,7 +1,7 @@
   <%# This template is used to render the initial facet list (#index action)
       and a paginated facet list in a modal (#facet action). The `dropdown-menu`
       class in only necessary for the former and causes display issues in the latter. -%>
-  <ul class="<%= 'dropdown-menu ' if action_name == 'index' %>facet-values" aria-labelledby="<%= facet_field.field.parameterize %>">
+  <ul id="<%= facet_field.field.parameterize + '-dropdown-options' %>" class="<%= 'dropdown-menu ' if action_name == 'index' %>facet-values" aria-labelledby="<%= facet_field.field.parameterize %>">
     <% paginator = facet_paginator(facet_field, display_facet) %>
     <%= render_facet_limit_list paginator, facet_field.key %>
 

data/app/views/hyrax/my/_facet_pivot.html.erb

@@ -1,5 +1,5 @@
 <% if !subfacet ||= false %>
-  <ul class="dropdown-menu pivot-facet list-unstyled">
+  <ul class="dropdown-menu pivot-facet list-unstyled" id="<%= facet_field.field.parameterize + '-dropdown-options' %>">
 <% end %>
 
   <% display_facet.items.each do |item| -%>

data/app/views/hyrax/my/_work_action_menu.html.erb

@@ -1,12 +1,14 @@
+<% ul_id = 'admin-set-action-dropdown-ul-' + document.id %>
+
 <div class="btn-group">
 
-  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= document.id %>" aria-haspopup="true">
+  <button class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown" type="button" id="dropdownMenu_<%= document.id %>" aria-haspopup="true" aria-expanded="false" aria-controls="<%= ul_id %>">
     <span class="sr-only"><%= t("hyrax.dashboard.my.sr.press_to") %> </span>
     <%= t("hyrax.dashboard.my.action.select") %>
     <span class="caret" aria-hidden="true"></span>
   </button>
 
-  <ul role="menu" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= document.id %>">
+  <ul role="menu" id="<%= ul_id %>" class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdownMenu_<%= document.id %>">
 
     <% if can? :edit, document.id %>
       <li role="menuitem" tabindex="-1">
@@ -29,14 +31,14 @@
 
     <li role="menuitem" tabindex="-1">
       <%= display_trophy_link(current_user, document.id) do |text| %>
-        <i class='glyphicon glyphicon-star'></i> <%= text %>
+        <i class="glyphicon glyphicon-star" aria-hidden="true"></i> <%= text %>
       <% end %>
     </li>
 
     <% if can? :transfer, document.id %>
       <li role="menuitem" tabindex="-1">
         <%= link_to(hyrax.new_work_transfer_path(document.id), class: 'itemicon itemtransfer', title: t("hyrax.dashboard.my.action.transfer")) do %>
-          <i aria-hidden="true" class='glyphicon glyphicon-transfer'></i>
+          <i class="glyphicon glyphicon-transfer" aria-hidden="true"></i>
           <span> <%= t("hyrax.dashboard.my.action.transfer") %> </span>
         <% end %>
       </li>

data/app/views/hyrax/my/collections/index.html.erb

@@ -13,7 +13,7 @@
 </script>
 
 <% provide :page_header do %>
-  <h1><span class="fa fa-folder-open"></span> <%= t("hyrax.admin.sidebar.collections") %></h1>
+  <h1><span class="fa fa-folder-open" aria-hidden="true"></span> <%= t("hyrax.admin.sidebar.collections") %></h1>
 <% end %>
 
 <div class="collections-wrapper">

data/app/views/hyrax/my/index.html.erb

@@ -14,7 +14,7 @@
 </script>
 
 <% provide :page_header do %>
-  <h1><span class="fa fa-file"></span> <%= t("hyrax.admin.sidebar.works") %></h1>
+  <h1><span class="fa fa-file" aria-hidden="true"></span> <%= t("hyrax.admin.sidebar.works") %></h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/my/works/_list_works.html.erb

@@ -31,7 +31,7 @@
 
   <td class="date"><%= document.date_uploaded %></td>
   <td class='text-center'>
-    <span class="fa <%= current_user.trophies.where(work_id: document.id).exists? ? 'fa-star highlighted-work' : 'fa-star-o trophy-off' %>"></span></td>
+    <span class="fa <%= current_user.trophies.where(work_id: document.id).exists? ? 'fa-star highlighted-work' : 'fa-star-o trophy-off' %>" aria-hidden="true"></span></td>
   <td><%= render_visibility_link document %></td>
 
   <td>

data/app/views/hyrax/my/works/index.html.erb

@@ -14,7 +14,7 @@
 </script>
 
 <% provide :page_header do %>
-  <h1><span class="fa fa-file"></span> <%= t("hyrax.admin.sidebar.works") %></h1>
+  <h1><span class="fa fa-file" aria-hidden="true"></span> <%= t("hyrax.admin.sidebar.works") %></h1>
   <% if current_ability.can_create_any_work? %>
     <div class="pull-right">
       <% if @create_work_presenter.many? %>

data/app/views/hyrax/pages/edit.html.erb

@@ -1,5 +1,5 @@
 <% content_for :page_header do %>
-  <h1><span class="fa fa-file-text-o"></span> <%= t(:'hyrax.admin.sidebar.pages') %></h1>
+  <h1><span class="fa fa-file-text-o" aria-hidden="true"></span> <%= t(:'hyrax.admin.sidebar.pages') %></h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/transfers/index.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-arrows-h"></span>  <%= t("hyrax.dashboard.transfer_of_ownership") %></h1>
+  <h1><span class="fa fa-arrows-h" aria-hidden="true"></span>  <%= t("hyrax.dashboard.transfer_of_ownership") %></h1>
 <% end %>
 
 <div class="row">

data/app/views/hyrax/transfers/new.html.erb

@@ -1,5 +1,5 @@
 <% provide :page_header do %>
-  <h1><span class="fa fa-arrows-h"></span><%= t(:'.title', work_title: @proxy_deposit_request.to_s) %></h1>
+  <h1><span class="fa fa-arrows-h" aria-hidden="true"></span><%= t(:'.title', work_title: @proxy_deposit_request.to_s) %></h1>
 <% end %>
 
 <span class="sr-only"><%= t(:'.sr_only_description', work_title: @proxy_deposit_request.to_s) %></span>

data/app/views/hyrax/users/_left_sidebar.html.erb

@@ -1,4 +1,4 @@
 <%= render 'hyrax/users/user', user: @user %>
 
 <br />
-<a class="btn btn-primary" href="<%= hyrax.users_path %>"><i class="glyphicon glyphicon-globe"></i> View Users</a>
+<a class="btn btn-primary" href="<%= hyrax.users_path %>"><i class="glyphicon glyphicon-globe" aria-hidden="true"></i> View Users</a>

data/app/views/hyrax/users/_user_info.html.erb

@@ -11,35 +11,35 @@
 <% end %>
 
 <% if user.facebook_handle.present? %>
-  <dt><i class="fa fa-facebook"></i> Facebook Handle</dt>
+  <dt><i class="fa fa-facebook" aria-hidden="true"></i> Facebook Handle</dt>
   <dd><%= link_to user.facebook_handle, "http://facebook.com/#{user.facebook_handle}", {target:'_blank'} %></dd>
 <% end %>
 
 <% if user.twitter_handle.present? %>
-  <dt><i class="fa fa-twitter"></i> Twitter Handle</dt>
+  <dt><i class="fa fa-twitter" aria-hidden="true"></i> Twitter Handle</dt>
   <dd><%= link_to user.twitter_handle, "http://twitter.com/#{user.twitter_handle}", {target:'_blank'} %></dd>
 <% end %>
 
 <% if user.googleplus_handle.present? %>
-  <dt><i class="fa fa-google-plus"></i> Google+ Handle</dt>
+  <dt><i class="fa fa-google-plus" aria-hidden="true"></i> Google+ Handle</dt>
   <dd><%= link_to user.googleplus_handle, "http://google.com/+#{user.googleplus_handle}", {target:'_blank'} %></dd>
 <% end %>
 
 <% if user.linkedin_handle.present? %>
-  <dt><i class="fa fa-linkedin"></i> LinkedIn</dt>
+  <dt><i class="fa fa-linkedin" aria-hidden="true"></i> LinkedIn</dt>
   <dd><%= link_to "#{@linkedInUrl}", "#{@linkedInUrl}", { target: '_blank' } %></dd>
 <% end %>
 
-  <dt><i class="fa fa-envelope"></i> Email</dt>
+  <dt><i class="fa fa-envelope" aria-hidden="true"></i> Email</dt>
   <dd><%= mail_to user.email %></dd>
 
   <% if user.chat_id %>
-    <dt><i class="glyphicon glyphicon-bullhorn"></i> Chat ID</dt>
+    <dt><i class="glyphicon glyphicon-bullhorn" aria-hidden="true"></i> Chat ID</dt>
     <dd><%= user.chat_id %></dd>
   <% end %>
 
   <% if user.website %>
-    <dt><i class="glyphicon glyphicon-globe"></i> Website(s)</dt>
+    <dt><i class="glyphicon glyphicon-globe" aria-hidden="true"></i> Website(s)</dt>
     <dd><%= iconify_auto_link(user.website) %></dd>
   <% end %>
 
@@ -64,7 +64,7 @@
   <% end %>
 
   <% if user.address %>
-    <dt><i class="glyphicon glyphicon-map-marker"></i> Address</dt>
+    <dt><i class="glyphicon glyphicon-map-marker" aria-hidden="true"></i> Address</dt>
     <dd><%= user.address %></dd>
   <% end %>
 
@@ -74,7 +74,7 @@
   <% end %>
 
   <% if user.telephone %>
-    <dt><i class="glyphicon glyphicon-earphone"></i> Telephone</dt>
+    <dt><i class="glyphicon glyphicon-earphone" aria-hidden="true"></i> Telephone</dt>
     <dd><%= link_to_telephone(user) %></dd>
   <% end %>