From: Stephanie Leary <stephanie.leary@equinoxOLI.org>
Date: Fri, 21 Oct 2022 21:15:57 +0000 (-0500)
Subject: LP1993859 Keyboard tabbing for results pagination
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=refs%2Fheads%2Fuser%2Fsleary%2Flp1993859-results-pagination-tabindex;p=working%2FEvergreen.git

LP1993859 Keyboard tabbing for results pagination

In the staff catalog search results pagination component, adds
tabindex="0" to link tags without href attributes to enable keyboard
navigation.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxOLI.org>
---

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/pagination.component.html b/Open-ILS/src/eg2/src/app/staff/catalog/result/pagination.component.html
index a15e428b2e..1ee5d026a8 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/pagination.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/pagination.component.html
@@ -5,8 +5,8 @@ unnecessary given we have to track paging externally anyway.
 <ul class="pagination">
   <li class="page-item" 
     [ngClass]="{disabled : searchContext.pager.isFirstPage()}">
-    <a (click)="prevPage()"
-      class="page-link" 
+    <a (click)="prevPage()" (keydown.enter)="prevPage()"
+      class="page-link" tabindex="0" 
       i18n-aria-label
       aria-label="Previous">
       <span aria-hidden="true">&laquo;</span>
@@ -15,13 +15,13 @@ unnecessary given we have to track paging externally anyway.
   <li class="page-item" 
     *ngFor="let page of currentPageList()"
     [ngClass]="{active : searchContext.pager.currentPage() === page}">
-    <a class="page-link" (click)="setPage(page)">
+    <a class="page-link" tabindex="0" (click)="setPage(page)" (keydown.enter)="setPage(page)">
       {{page}} <span class="sr-only" i18n>(current)</span></a>
   </li>
   <li class="page-item" 
     [ngClass]="{disabled : searchContext.pager.isLastPage()}">
-    <a (click)="nextPage()"
-      class="page-link" aria-label="Next" i18n-aria-label>
+    <a (click)="nextPage()" (keydown.enter)="nextPage()"
+      class="page-link" tabindex="0" aria-label="Next" i18n-aria-label>
       <span aria-hidden="true">&raquo;</span>
     </a>
   </li>