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>
Tue, 17 Sep 2013 17:52:15 +0000 (13:52 -0400)
commit812ce513e14977825151358a39ec823876566dac
tree1a4d4e55724ecc5e70691182c2e4086e31e88e46
parenta825eea41a64ab497403d2760aba4257cdb4c2f0
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