From bfaddbb616f348687b72b557a9beb9dd353e3d01 Mon Sep 17 00:00:00 2001 From: Aleisha Date: Wed, 23 Mar 2016 21:53:44 +0000 Subject: [PATCH] Bug 15793 - UX of circulation patron search with long lists of returned borrowers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Okay how about this: This patch has no floating toolbar or names styled as links. I've removed the radio buttons. The row is clickable and will redirect you to the user's page. EDIT: This patch makes the cursor a pointer when hovering over the patrons. The row will also be highlighted on hover to make it even clearer. EDIT2: Removes unnecessary 'Select' button To test: 1) Put in a patron search that will bring back a long list of patrons (ie the letter 'a') 2) Select a patron by clicking anywhere on the row - confirm you are taken to the correct patron page Sponsored-by: Catalyst IT Signed-off-by: Marc VĂ©ron Signed-off-by: Liz Rea Submit button removed, patron selection on this screen much better for long lists (and short ones as well.) Signed-off-by: Jonathan Druart --- koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css | 8 ++++++++ koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 11 ++++++----- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css index 58cb902..a638129 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css @@ -282,6 +282,14 @@ tr.odd.onissue td { background-color: #FFFFE1; } +tr.clickable { + cursor: pointer; +} + +.table_borrowers tr:hover td { + background-color: #ffff99; +} + tfoot td { background-color : #f3f3f3; font-weight : bold; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt index 0efe49b..aa7bbdb 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -89,6 +89,10 @@ $(document).ready(function() { } }); + $(".clickable").click(function() { + window.document.location = $(this).data('url'); + }); + [% IF !( CircAutoPrintQuickSlip == 'clear' ) %] // listen submit to trigger qslip on empty checkout $('#mainform').bind('submit',function() { @@ -568,10 +572,9 @@ No patron matched [% message %] - +
- @@ -581,8 +584,7 @@ No patron matched [% message %] [% FOREACH borrower IN borrowers %] - - + @@ -592,7 +594,6 @@ No patron matched [% message %] [% END %]
Name Cardnumber Category
[% borrower.surname %], [% borrower.firstname %] [% borrower.cardnumber %] [% borrower.categorycode %]
-
[% ELSE %] -- 2.7.0