From: Cesar Velez <cesar.velez@equinoxinitiative.org>
Date: Thu, 8 Jun 2017 15:33:31 +0000 (-0400)
Subject: LP#1677000 - webstaff Adds a 'has-notes visual indicator' for patrons with notes.
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=8cb7f04da3732e152f05cbbe7d9f5ac5c7842ca0;p=working%2FEvergreen.git

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>
---

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'
         ]
     };