From 64066f780afd5d1c11ac8a5d3e598f6c7896bc5d Mon Sep 17 00:00:00 2001
From: Bill Erickson <berick@esilibrary.com>
Date: Mon, 12 May 2014 15:29:43 -0400
Subject: [PATCH] grid row selectors (clicking on index)

It's a start, anyway.  we may want a more obvious means of selecting a
row.  Easy enough to change.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
---
 Open-ILS/src/templates/staff/share/t_autogrid.tt2 | 14 +++++++++-----
 Open-ILS/web/js/ui/default/staff/services/grid.js |  7 ++++++-
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/Open-ILS/src/templates/staff/share/t_autogrid.tt2 b/Open-ILS/src/templates/staff/share/t_autogrid.tt2
index 4e037ade91..ccfc46ee68 100644
--- a/Open-ILS/src/templates/staff/share/t_autogrid.tt2
+++ b/Open-ILS/src/templates/staff/share/t_autogrid.tt2
@@ -159,11 +159,12 @@
 
   <div class="eg-grid-row eg-grid-header-row">
     <div class="eg-grid-cell eg-grid-cell-stock">
-      <div>[% l('#') %]</div>
+      <div title="[% l('Row Number Column') %]">[% l('#') %]</div>
     </div>
     <div class="eg-grid-cell eg-grid-cell-stock">
       <div>
-        <input type='checkbox' ng-click="toggleSelectAllItems()"/>
+        <input title="[% l('Row Selector Column') %]"
+          type='checkbox' ng-click="toggleSelectAllItems()"/>
       </div>
     </div>
     <div class="eg-grid-cell"
@@ -234,14 +235,17 @@
         ng-show="items.length > 0"
         ng-class="{'eg-grid-row-selected' : selected[indexValue(item)]}">
       <div class="eg-grid-cell eg-grid-cell-stock"
-        ng-click="handleRowClick($event, item)">
-        {{$index + offset() + 1}}
+        ng-click="handleRowClick($event, item)" title="[% l('Row Index') %]">
+        <a href ng-show="activateItem" ng-click="activateItem(item)" style="font-weight:bold">
+          {{$index + offset() + 1}}
+        </a>
+        <div ng-hide="activateItem">{{$index + offset() + 1}}</div>
       </div>
       <div class="eg-grid-cell eg-grid-cell-stock">
         <!-- ng-click=handleRowClick here has unintended 
              consequences and is unnecessary, avoid it -->
         <div>
-          <input type='checkbox'  
+          <input type='checkbox' title="[% l('Select Row') %]"
             ng-model="selected[indexValue(item)]"/>
         </div>
       </div>
diff --git a/Open-ILS/web/js/ui/default/staff/services/grid.js b/Open-ILS/web/js/ui/default/staff/services/grid.js
index 360ac91efe..e1cfd2be7f 100644
--- a/Open-ILS/web/js/ui/default/staff/services/grid.js
+++ b/Open-ILS/web/js/ui/default/staff/services/grid.js
@@ -65,7 +65,12 @@ angular.module('egGridMod',
             // called on each item retrieved in collect() with the item
             // as the argument.  Useful for modiying objects before they
             // are absorbed by the grid.
-            onItemRetrieved : '='
+            onItemRetrieved : '=',
+
+            // function;  if set, row index values will be hyperlinked and
+            // the onclick for an item will call activateItem with the item
+            // as the argument.
+            activateItem : '='
         },
 
         // TODO: avoid hard-coded url
-- 
2.11.0