LP#1677000 - webstaff Adds a 'has-notes visual indicator' for patrons with notes.
authorCesar Velez <cesar.velez@equinoxinitiative.org>
Thu, 8 Jun 2017 15:33:31 +0000 (11:33 -0400)
committerJason Etheridge <jason@equinoxinitiative.org>
Fri, 30 Jun 2017 19:56:48 +0000 (15:56 -0400)
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>

Signed-off-by: Jason Etheridge <jason@equinoxinitiative.org>

Open-ILS/src/templates/staff/circ/patron/index.tt2
Open-ILS/src/templates/staff/css/circ.css.tt2
Open-ILS/web/js/ui/default/staff/services/user.js

index b9d19c8..9f2d4cc 100644 (file)
@@ -86,6 +86,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') %]"
index 7b165e8..8f3475b 100644 (file)
@@ -15,6 +15,7 @@ but the ones I'm finding aren't quite cutting it..*/
 .patron-summary-alert-small {color: red; background-color: white;}
 .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,
index 9fd32a8..ccd1d0a 100644 (file)
@@ -17,7 +17,8 @@ function($q,  $timeout,  egNet,  egAuth,  egOrg) {
             'billing_address',                                                     
             'mailing_address',                                                     
             'stat_cat_entries',                                                    
-            'usr_activity' 
+            'usr_activity',
+            'notes'
         ]
     };