enhancements based on feedback from CW/MARS
authorJason Etheridge <jason@esilibrary.com>
Thu, 21 Feb 2013 23:48:39 +0000 (18:48 -0500)
committerJason Stephenson <jason@sigio.com>
Tue, 14 Mar 2017 15:55:42 +0000 (11:55 -0400)
commitcafefb8bb4aa0f1eb75c1370d29af3a07695ae64
treeb9c55a993bc420c99df1ae19c7fb4d1e00332374
parent00d160ace457f838991cba48ab9fb41f3689281e
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>
Signed-off-by: Jason Stephenson <jason@sigio.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