Merge pull request #1991 from projectblacklight/add-missing-paginatio…

…n-keys

Adds missing pagination a11y translations
Use of region role for navigation following https://www.w3.org/WAI/GL/wiki/Using_the_region_role_to_identify_a_region_of_the_page

app/views/catalog/_results_pagination.html.erb

@@ -1,9 +1,9 @@
 <% if show_pagination? and @response.total_pages > 1 %>
  <div class="row record-padding">
   <div class="col-md-12">
-    <div class="pagination">
+    <nav class="pagination" role="region" aria-label="<%= t('views.pagination.aria.container_label') %>">
       <%= paginate @response, :outer_window => 2, :theme => 'blacklight' %>
-    </div>
+    </nav>
   </div>
  </div>
 <% end %>

app/views/kaminari/blacklight/_first_page.html.erb

@@ -7,5 +7,5 @@
     remote:        data-remote
 -%>
 <li class="page-item <%= 'disabled' if current_page.first? %>">
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, :remote => remote, class: 'page-link' %>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, :remote => remote, class: 'page-link', aria: { label: t('views.pagination.aria.go_to_first_page') } %>
 </li>

app/views/kaminari/blacklight/_last_page.html.erb

@@ -7,5 +7,5 @@
     remote:        data-remote
 -%>
 <li class="page-item <%= 'disabled' if current_page.last? %>">
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, :remote => remote, class: 'page-link' %>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, :remote => remote, class: 'page-link', aria: { label: t('views.pagination.aria.go_to_last_page') } %>
 </li>

app/views/kaminari/blacklight/_next_page.html.erb

@@ -8,10 +8,10 @@
 -%>
 <% if current_page.last? %>
     <li class="page-item disabled">
-      <%= link_to raw(t 'views.pagination.next'), '#', :rel => 'next', :onclick=>'return false;', class: 'page-link' %>
+      <%= link_to raw(t 'views.pagination.next'), '#', :rel => 'next', :onclick=>'return false;', class: 'page-link', aria: { label: t('views.pagination.aria.go_to_next_page') } %>
     </li>
 <% else %>
     <li class="page-item">
-      <%= link_to raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote, class: 'page-link' %>
+      <%= link_to raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote, class: 'page-link', aria: { label: t('views.pagination.aria.go_to_next_page') } %>
     </li>
-<% end %>
+<% end %>

app/views/kaminari/blacklight/_page.html.erb

@@ -7,11 +7,13 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
+<% page_display = number_with_delimiter(page.to_s) %>
+
 <li class="page-item <%= 'active' if page.current? %>">
   <% if page.current? %>
-    <span class="page-link"><%= number_with_delimiter(page.to_s) %></span>
+    <span class="page-link" aria-label="<%= t('views.pagination.aria.current_page', page: page_display) %>" aria-current="true"><%= page_display %></span>
   <% else %>
-    <%= link_to number_with_delimiter(page.to_s), url, :remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil, class: 'page-link' %>
+    <%= link_to page_display, url, :remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil, class: 'page-link', aria: { label: t('views.pagination.aria.go_to_page', page: page_display) } %>
   <% end %>
 </li>
 

app/views/kaminari/blacklight/_prev_page.html.erb

@@ -8,11 +8,11 @@
 -%>
 <% if current_page.first? %>
     <li class="page-item disabled">
-      <%= link_to raw(t 'views.pagination.previous'), '#', :rel => 'prev', :onclick=>'return false;', class: 'page-link' %>
+      <%= link_to raw(t 'views.pagination.previous'), '#', :rel => 'prev', :onclick=>'return false;', class: 'page-link', aria: { label: t('views.pagination.aria.go_to_previous_page') } %>
     </li>
 <% else %>
     <li class="page-item">
-      <%= link_to raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote, class: 'page-link' %>
+      <%= link_to raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote, class: 'page-link', aria: { label: t('views.pagination.aria.go_to_previous_page') } %>
     </li>
 <% end %>
 

config/locales/blacklight.de.yml

@@ -6,6 +6,14 @@ de:
       previous: '« Voherige'
       next: 'Nächste »'
       truncate: '…'
+      aria:
+        container_label: Paginierungslinks
+        current_page: Aktuelle Seite, Seite %{page}
+        go_to_first_page: Gehe zur ersten Seite
+        go_to_last_page: Gehe zur letzten Seite
+        go_to_next_page: Gehe zur nächsten Seite
+        go_to_page: Gehe zur Seite %{page}
+        go_to_previous_page: Zurück zur letzten Seite
 
     pagination_compact:
       previous: '« Voherige'

config/locales/blacklight.en.yml

@@ -6,6 +6,14 @@ en:
       previous: '« Previous'
       next: 'Next »'
       truncate: '…'
+      aria:
+        container_label: 'pagination links'
+        current_page: 'Current Page, Page %{page}'
+        go_to_page: 'Go to page %{page}'
+        go_to_previous_page: 'Go to previous page'
+        go_to_next_page: 'Go to next page'
+        go_to_first_page: 'Go to first page'
+        go_to_last_page: 'Go to last page'
 
     pagination_compact:
       previous: '« Previous'

config/locales/blacklight.es.yml

@@ -6,6 +6,14 @@ es:
       previous: '« Anterior'
       next: 'Siguiente »'
       truncate: '…'
+      aria:
+        container_label: enlaces de paginación
+        current_page: Página actual, página %{page}
+        go_to_first_page: Ir a la primera pagina
+        go_to_last_page: Ir a la última página
+        go_to_next_page: Ir a la página siguiente
+        go_to_page: Ir a la página %{page}
+        go_to_previous_page: Regresar a la pagina anterior
 
     pagination_compact:
       previous: '« Anterior'

config/locales/blacklight.fr.yml

@@ -6,6 +6,14 @@ fr:
       previous: '« Précédente'
       next: 'Suivante »'
       truncate: '…'
+      aria:
+        container_label: liens de pagination
+        current_page: Page actuelle, Page %{page}
+        go_to_first_page: Aller à la première page
+        go_to_last_page: Aller à la dernière page
+        go_to_next_page: Aller à la page suivante
+        go_to_page: Aller à la page %{page}
+        go_to_previous_page: Aller à la page précédente
 
     pagination_compact:
       previous: '« Précédente'

config/locales/blacklight.it.yml

@@ -6,6 +6,14 @@ it:
       previous: '« Pagina precedente'
       next: 'Pagina successiva »'
       truncate: '…'
+      aria:
+        container_label: collegamenti di paginazione
+        current_page: Pagina corrente, Pagina %{page}
+        go_to_first_page: Vai alla prima pagina
+        go_to_last_page: Vai all'ultima pagina
+        go_to_next_page: Vai alla pagina successiva
+        go_to_page: Vai alla pagina %{page}
+        go_to_previous_page: Vai alla pagina precedente
 
     pagination_compact:
       previous: '« Pagina precedente'

config/locales/blacklight.pt-BR.yml

@@ -6,6 +6,15 @@ pt-BR:
       previous: '« anterior'
       next: 'próximo »'
       truncate: '…'
+      aria:
+        container_label: links de paginação
+        current_page: Página atual, Página %{page}
+        go_to_first_page: Ir para a primeira página
+        go_to_last_page: Ir para a última página
+        go_to_next_page: Ir para a próxima página
+        go_to_page: Ir para a página %{page}
+        go_to_previous_page: Ir para a página anterior
+
 
     pagination_compact:
       previous: '« ant'