From: Bill Erickson <berickxx@gmail.com>
Date: Fri, 31 May 2019 15:45:52 +0000 (-0400)
Subject: LP1803787 Grid actions menu tabindex
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=233e72b04d9a44faa266d0aae763a36610ef545d;p=evergreen%2Fpines.git

LP1803787 Grid actions menu tabindex

Allow the browser to focus the actions menu entries on tab so 'Enter'
action will not inadvertantly fire the row activate handler as well.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
---

diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar-actions-menu.component.html b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar-actions-menu.component.html
index cdb6dc4fec..224128d310 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar-actions-menu.component.html
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar-actions-menu.component.html
@@ -1,7 +1,7 @@
 <button class="dropdown-item scrollable-menu" 
   [disabled]="shouldDisable(action)"
-  (click)="performAction(action)"
-  *ngFor="let action of gridContext.toolbarActions">
+  (click)="performAction(action)" tabindex="0"
+  *ngFor="let action of gridContext.toolbarActions; let idx = index">
   <ng-container *ngIf="action.isGroup">
     <span class="font-weight-bold font-italic">{{action.label}}</span>
   </ng-container>