From 472ad08090ea39a8483398087e4ab4a8ae6b7b1c Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Fri, 20 Jun 2014 14:27:17 +0200
Subject: [PATCH] Accessibility: improve pagination

---
 application/views/scripts/mixedPagination.phtml | 11 +++++++----
 public/css/icinga/layout-structure.less         |  6 ++++++
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/application/views/scripts/mixedPagination.phtml b/application/views/scripts/mixedPagination.phtml
index 61055f35d..bac2b202b 100644
--- a/application/views/scripts/mixedPagination.phtml
+++ b/application/views/scripts/mixedPagination.phtml
@@ -9,10 +9,11 @@ use Icinga\Web\Url;
 
 if ($this->pageCount <= 1) return;
 
-?><ul class="pagination"
+?><p id="paginationlabel" class="audible"><?= t('Pagination') ?></p>
+<ul class="pagination" aria-labelledby="paginationlabel" role="navigation"
 <?php
 
-$fromto = t('%d to %d of %d');
+$fromto = t('Show rows %d to %d of %d');
 $total = $this->totalItemCount;
 $limit = $this->itemCountPerPage;
 $title_prev = sprintf(
@@ -28,7 +29,9 @@ $title_next = sprintf(
     ($this->current + 1) * $limit,
     $total
 );
-$li = '  ><li%s><a href="%s" title="%s">%s</a></li
+$li = '  ><li%s><span class="audible">'
+    . t('Page')
+    . ' </span><a href="%s" title="%s">%s</a></li
 ';
 
 ?>
@@ -65,7 +68,7 @@ foreach ($this->pagesInRange as $page) {
           $class,
           Url::fromRequest()->overwriteParams(
               array('page' => $page)
-          )->getAbsoluteUrl(),
+          ),
           $title,
           $page
       );
diff --git a/public/css/icinga/layout-structure.less b/public/css/icinga/layout-structure.less
index 1e6382cde..e7d415073 100644
--- a/public/css/icinga/layout-structure.less
+++ b/public/css/icinga/layout-structure.less
@@ -309,6 +309,12 @@ html {
   display: block;
 }
 
+.audible {
+  left: -999em;
+  position: absolute;
+}
+
+/* TODO: replace this with .error */
 .fileNotReadable {
   padding: 0.5em;
   background-color: @colorCritical;