Improve accessibility and implement translatable strings (#13)

Co-authored-by: Bernard Hamlin <948122+blueo@users.noreply.github.com>

Author: chrispenny

templates/SilverStripe/Discoverer/Includes/Pagination.ss

@@ -1,35 +1,55 @@
-<ul class="pagination">
-    <li class="pagination-item pagination-item--prev">
-        <% if $NotFirstPage %>
-            <a title="View previous page of results" class="pagination-prev-link" href="$PrevLink">&laquo;</a>
-        <% else %>
-            <span title="View previous page of results" class="pagination-prev-link pagination-prev-link--disabled">&laquo;</span>
-        <% end_if %>
-    </li>
+<nav aria-label="<%t SilverStripe\Discoverer\Includes\Pagination.PaginationLabel 'Search pagination' %>">
+    <ul class="pagination">
+        <li class="pagination-item pagination-item--prev">
+            <% if $NotFirstPage %>
+                <a
+                    class="pagination-prev-link"
+                    href="$PrevLink"
+                    {$Attributes}
+                    aria-label="<%t SilverStripe\Discoverer\Includes\Pagination.PreviousPage 'Previous page' %>"
+                >&laquo;</a>
+            <% else %>
+                <a
+                    class="pagination-prev-link pagination-prev-link--disabled"
+                    href="#"
+                    aria-label="<%t SilverStripe\Discoverer\Includes\Pagination.PreviousPageDisabled 'Previous page (not available)' %>"
+                    aria-disabled="true"
+                >&laquo;</a>
+            <% end_if %>
+        </li>
 
-    <% loop $PaginationSummary(4) %>
-        <% if $CurrentBool %>
-            <li class="pagination-item pagination-item--active">
-                <a title="Viewing page $PageNum of results" class="pagination-link pagination-link--disabled">$PageNum</a>
+        <% loop $PaginationSummary(4) %>
+            <li class="pagination-item">
+                <% if $CurrentBool %>
+                    <a title="Viewing page $PageNum of results"
+                       class="pagination-link pagination-link--disabled"
+                    >$PageNum</a>
+                <% else %>
+                    <% if $Link %>
+                        <a title="View page $PageNum of results" class="pagination-link" href="$Link">$PageNum</a>
+                    <% else %>
+                        <span class="pagination-link pagination-link--disabled">&hellip;</span>
+                    <% end_if %>
+                <% end_if %>
             </li>
-        <% else %>
-            <% if $Link %>
-                <li class="pagination-item">
-                    <a title="View page $PageNum of results" class="pagination-link" href="$Link">$PageNum</a>
-                </li>
+        <% end_loop %>
+
+        <li class="pagination-item pagination-item--next">
+            <% if $NotLastPage %>
+                <a
+                    class="pagination-next-link"
+                    href="$NextLink"
+                    {$Attributes}
+                    aria-label="<%t SilverStripe\Discoverer\Includes\Pagination.NextPage 'Next page' %>"
+                >&raquo;</a>
             <% else %>
-                <li class="pagination-item">
-                    <a class="pagination-link pagination-link--disabled">&hellip;</a>
-                </li>
+                <a
+                    class="pagination-next-link pagination-next-link--disabled"
+                    href="#"
+                    aria-label="<%t SilverStripe\Discoverer\Includes\Pagination.NextPageDisabled 'Next page (not available)' %>"
+                    aria-disabled="true"
+                >&raquo;</a>
             <% end_if %>
-        <% end_if %>
-    <% end_loop %>
-
-    <li class="pagination-item pagination-item--next">
-        <% if $NotLastPage %>
-            <a title="View next page of results" class="pagination-next-link" href="$NextLink">&raquo;</a>
-        <% else %>
-            <span title="View next page of results" class="pagination-next-link pagination-next-link--disabled">&raquo;</span>
-        <% end_if %>
-    </li>
-</ul>
+        </li>
+    </ul>
+</nav>

templates/SilverStripe/Discoverer/Includes/Summary.ss

@@ -1 +1 @@
-<p class="_summary">Displaying <% if $LastItem > 0 %>$FirstItem<% else %>0<% end_if %> - $LastItem results of $TotalItems</p>
+<p class="_summary"><%t SilverStripe\Discoverer\Includes\Summary.Results 'Displaying {first} - {last} results of {total}' first=$FirstItem last=$LastItem total=$TotalItems %></p>

templates/SilverStripe/Discoverer/Service/Results/Results.ss

@@ -9,5 +9,5 @@
         <% end_if %>
     <% end_with %>
 <% else %>
-    <p class="error">No search results.</p>
+    <p class="error"><%t SilverStripe\Discoverer\Service\Results\Results.NoResults 'No search results.' %></p>
 <% end_if %>