enhancements based on feedback from CW/MARS
authorJason Etheridge <jason@esilibrary.com>
Thu, 21 Feb 2013 23:48:39 +0000 (18:48 -0500)
committerJason Etheridge <jason@esilibrary.com>
Thu, 21 Jul 2016 17:21:46 +0000 (13:21 -0400)
commitb246ea342360aaac87546f5b96dd8d707b3ba148
treeb67a07865939c0bfd0396ece09a7b1bc4aac21e7
parent2b980cac75561c0ca8880874d45f9597e4457db4
enhancements based on feedback from CW/MARS

Thanks!  Major changes includes different patron summary panes based on context,
removal of the new-tab behavior when selecting different patron interfaces, and
having the patron summary pane for searches replace the search form instead of
the list results (I wonder if catalogers would want similar behavior in the
z39.50 interface...)

For the search interface, the summary pane is mostly used to help identify
patrons, and thus focuses on displaying identifying information like addresses
and phone numbers.

For the patron interface, the summary pane focuses mostly on statuses, alerts,
and notifications.

I've backed away a little bit from the "only show information if it exists"
philosophy.  Another school of thought is that information should always be
in the same place to take advantage of spatial memory.  I was hedging my bets
before by hoping column position would be enough for spatial memory.  It might
still be true, but here's to experimentation. :)

For switching between patron interfaces, the different components are still
disintegrated, and get completely reloaded on context switch, including the
summary pane.  I fear this is going to be very noticeable, but it'll take a
major overhaul to have persistently loaded sub-interfaces that stay in sync.

Still, the load times for retrieving a patron are much faster this way, plus
this wrapper for the sub-interfaces is likely easier to maintain, and likely
less of a source of memory leaks than before.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/PatronSummary.pm
Open-ILS/src/templates/opac/PatronSearchSummary.tt2 [new file with mode: 0644]
Open-ILS/src/templates/opac/PatronSummary.tt2
Open-ILS/src/templates/opac/css/PatronSummary.css.tt2
Open-ILS/xul/staff_client/chrome/content/main/constants.js
Open-ILS/xul/staff_client/server/patron/display.xul
Open-ILS/xul/staff_client/server/patron/patron.js
Open-ILS/xul/staff_client/server/patron/search.js
Open-ILS/xul/staff_client/server/patron/search_result.js