From 8cb7f04da3732e152f05cbbe7d9f5ac5c7842ca0 Mon Sep 17 00:00:00 2001
From: Cesar Velez <cesar.velez@equinoxinitiative.org>
Date: Thu, 8 Jun 2017 11:33:31 -0400
Subject: [PATCH] LP#1677000 - webstaff Adds a 'has-notes visual indicator' for
 patrons with notes.

Made the indicator be clickable and route to Other->Notes.
Other ways to address this not mentioned in LP would be
using a glyphicon instead, or moving the Notes submenu item out of Others tab.
Or even making the entire tab/menu be resposive, and collapsing into an Others submenu
only when needed.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>
---
 Open-ILS/src/templates/staff/circ/patron/index.tt2 | 2 ++
 Open-ILS/src/templates/staff/css/circ.css.tt2      | 1 +
 Open-ILS/web/js/ui/default/staff/services/user.js  | 3 ++-
 3 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/Open-ILS/src/templates/staff/circ/patron/index.tt2 b/Open-ILS/src/templates/staff/circ/patron/index.tt2
index b017f19c9f..ae49ca7d06 100644
--- a/Open-ILS/src/templates/staff/circ/patron/index.tt2
+++ b/Open-ILS/src/templates/staff/circ/patron/index.tt2
@@ -76,6 +76,8 @@ angular.module('egCoreMod').run(['egStrings', function(s) {
                 '{{patron().first_given_name()}}',
                 '{{patron().second_given_name()}}') %]
           </div>
+
+          <p ng-show="patron().notes().length > 0"><a class='patron-summary-has-notes' href="./circ/patron/{{patron().id()}}/notes"><span class="label label-warning">Notes &nbsp;{{patron().notes().length}}</span></a></p>
           <div ng-show="tab != 'search'">
             <a href ng-click="toggle_expand_summary()"
               title="[% l('Collapse Patron Summary Display') %]"
diff --git a/Open-ILS/src/templates/staff/css/circ.css.tt2 b/Open-ILS/src/templates/staff/css/circ.css.tt2
index 9bae6c0a7e..272ffa511c 100644
--- a/Open-ILS/src/templates/staff/css/circ.css.tt2
+++ b/Open-ILS/src/templates/staff/css/circ.css.tt2
@@ -15,6 +15,7 @@ but the ones I'm finding aren't quite cutting it..*/
 .patron-summary-alert-small {color: red}
 .patron-summary-divider { border-top: 1px solid #CCC}
 .patron-summary-act-link {font-size: .8em;}
+.patron-summary-has-notes:hover, .patron-summary-has-notes:visited{ text-decoration: none; }
 
 /* FIXME: use .barcode instead */
 #patron-checkout-barcode,
diff --git a/Open-ILS/web/js/ui/default/staff/services/user.js b/Open-ILS/web/js/ui/default/staff/services/user.js
index 9fd32a8b5e..ccd1d0ab71 100644
--- a/Open-ILS/web/js/ui/default/staff/services/user.js
+++ b/Open-ILS/web/js/ui/default/staff/services/user.js
@@ -17,7 +17,8 @@ function($q,  $timeout,  egNet,  egAuth,  egOrg) {
             'billing_address',                                                     
             'mailing_address',                                                     
             'stat_cat_entries',                                                    
-            'usr_activity' 
+            'usr_activity',
+            'notes'
         ]
     };
 
-- 
2.11.0