[Bug 15793] New: UX of circulation patron search with long lists of returned borrowers
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Bug ID: 15793 Summary: UX of circulation patron search with long lists of returned borrowers Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: liz@catalyst.net.nz QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com If you do a search from the circulation search that returns a long list of borrowers, it is highly likely that to do anything with your one selected borrower you will have to scroll longer than you'd like to get to the "select" button. We can improve this interface for librarians by either putting a "select" button at the top (or float it along the list), or changing the names (or some other part) into a link. I'm open to other implementation suggestions here as well. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |aleishaamohia@hotmail.com |ity.org | CC| |aleishaamohia@hotmail.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #1 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 47938 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47938&action=edit Bug 15793: Add floating toolbar to circulation.tt To test: 1) Put in a patron search that will bring back a long list of patrons (ie the letter 'a') 2) Confirm 'Select patron' button is now at the top 3) Scroll down page, confirm button toolbar follows you down the page 4) Select a patron and click button - confirm you are taken to the correct patron page -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Liz Rea <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #47938|0 |1 is obsolete| | --- Comment #2 from Liz Rea <liz@catalyst.net.nz> --- Created attachment 47940 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47940&action=edit Bug 15793: Add floating toolbar to circulation.tt To test: 1) Put in a patron search that will bring back a long list of patrons (ie the letter 'a') 2) Confirm 'Select patron' button is now at the top 3) Scroll down page, confirm button toolbar follows you down the page 4) Select a patron and click button - confirm you are taken to the correct patron page Signed-off-by: Liz Rea <liz@catalyst.net.nz> This is much nicer, thank you. :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch --- Comment #3 from Marc Véron <veron@veron.ch> --- Status is still 'Needs Signoff' - should it be switched to SO? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 47992 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47992&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers Using form controls to select a patron from the list of patron search results during checkout means that we have to have a submit button somewhere on a potentially very long form. This patch proposes that we use links to the patron records instead. This patch also makes a minor correction to the template to allow for the correct layout grid to be used, and enables the display of full library names in search results instead of branchcode. To test, apply the patch and search for a patron name from the "Check out" form in the header. Clicking the name of any patron in the search results should take you to the correct checkout page. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #5 from Owen Leonard <oleonard@myacpl.org> --- Sorry, I don't mean to be a busybody and I certainly appreciate Aleisha's work to come up with a good solution for this. I just want to push it a little further and see if we can simplify even more. Is there a reason not to use a link here? The method of using form submission includes a lot of hidden fields, but are they used? There are other places in Koha where a checkout is initiated using a plain link, so my assumption is that those hidden fields aren't necessary. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #6 from Aleisha Amohia <aleishaamohia@hotmail.com> --- We did think about having every name as a link and just thought a floating toolbar could be cool! Up to whoever signs off and QAs to see which they like best since either works :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #7 from Liz Rea <liz@catalyst.net.nz> --- I'm not fussy either way either, I'm happy as long as the problem of "I have to scroll for pages and pages to get to the submit button" is solved. :) Liz -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #47992|0 |1 is obsolete| | --- Comment #8 from Mirko Tietgen <mirko@abunchofthings.net> --- Created attachment 49457 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49457&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers Using form controls to select a patron from the list of patron search results during checkout means that we have to have a submit button somewhere on a potentially very long form. This patch proposes that we use links to the patron records instead. This patch also makes a minor correction to the template to allow for the correct layout grid to be used, and enables the display of full library names in search results instead of branchcode. To test, apply the patch and search for a patron name from the "Check out" form in the header. Clicking the name of any patron in the search results should take you to the correct checkout page. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |mirko@abunchofthings.net --- Comment #9 from Mirko Tietgen <mirko@abunchofthings.net> --- I prefer the simpler solution Owen provided and signed off on it. What do others think? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #10 from Marc Véron <veron@veron.ch> --- (In reply to Mirko Tietgen from comment #9)
I prefer the simpler solution Owen provided and signed off on it. What do others think?
+1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #11 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Could we have something between the two? :) It could be easier to have the tr clickable instead of clicking on a link. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49457|0 |1 is obsolete| | --- Comment #12 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 49472 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49472&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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. 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 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49457|1 |0 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #13 from Marc Véron <veron@veron.ch> --- (In reply to Aleisha Amohia from comment #12)
Created attachment 49472 [details] [review] Bug 15793 - UX of circulation patron search with long lists of returned borrowers
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.
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
Looks fine. Clickable row is the most ergonomic solution (patch 47940 + patch 449472 applied). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #47940|0 |1 is obsolete| | --- Comment #14 from Marc Véron <veron@veron.ch> --- Created attachment 49520 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49520&action=edit Bug 15793: Add floating toolbar to circulation.tt To test: 1) Put in a patron search that will bring back a long list of patrons (ie the letter 'a') 2) Confirm 'Select patron' button is now at the top 3) Scroll down page, confirm button toolbar follows you down the page 4) Select a patron and click button - confirm you are taken to the correct patron page Signed-off-by: Liz Rea <liz@catalyst.net.nz> This is much nicer, thank you. :) Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49457|0 |1 is obsolete| | Attachment #49472|0 |1 is obsolete| | --- Comment #15 from Marc Véron <veron@veron.ch> --- Created attachment 49521 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49521&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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. 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 <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #16 from Marc Véron <veron@veron.ch> --- Signed off on the two patches. Strange, while attaching the sign-offs, the alternate patch from Owen Leonard, was obsoleted as well (2016-03-23 18:10 UTC, signed-off by Mirko Tietgen, 4th patch if you show obsoleted ones). Maybe because of the same title? Leaving it obsoleted at the moment. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion --- Comment #17 from Owen Leonard <oleonard@myacpl.org> --- (In reply to Aleisha Amohia from comment #12)
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.
I strongly disagree with this solution. There is no indication at all, now, that the user can perform any action. Nothing looks like a link and the mouse cursor doesn't change to indicate anything is clickable. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #18 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Owen Leonard from comment #17)
(In reply to Aleisha Amohia from comment #12)
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.
I strongly disagree with this solution. There is no indication at all, now, that the user can perform any action. Nothing looks like a link and the mouse cursor doesn't change to indicate anything is clickable.
Does the solution work for you if we change the cursor? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49521|0 |1 is obsolete| | --- Comment #19 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 49611 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49611&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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. 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 <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49520|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #20 from Marc Véron <veron@veron.ch> --- (In reply to Aleisha Amohia from comment #19) (...)
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.
(...) Looks OK for me. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 --- Comment #21 from Liz Rea <liz@catalyst.net.nz> --- Hi, Overall I like this - one small thing, if you click the "select" button at the bottom it will say "borrower not found." I feel we should omit it. Owen? What do you think? Liz -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49611|0 |1 is obsolete| | --- Comment #22 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 49655 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49655&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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 <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Liz Rea <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49655|0 |1 is obsolete| | --- Comment #23 from Liz Rea <liz@catalyst.net.nz> --- Created attachment 49656 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49656&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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 <veron@veron.ch> Signed-off-by: Liz Rea <liz@catalyst.net.nz> Submit button removed, patron selection on this screen much better for long lists (and short ones as well.) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Magnus Enger <magnus@libriotech.no> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |magnus@libriotech.no Status|Needs Signoff |Signed Off --- Comment #24 from Magnus Enger <magnus@libriotech.no> --- As far as I can tell, this bug has one patch signed off by both Marc and Liz. Updating status accordingly. Please set back to NSO if I missed something. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49656|0 |1 is obsolete| | --- Comment #25 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 49734 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49734&action=edit Bug 15793 - UX of circulation patron search with long lists of returned borrowers 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 <veron@veron.ch> Signed-off-by: Liz Rea <liz@catalyst.net.nz> Submit button removed, patron selection on this screen much better for long lists (and short ones as well.) Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |brendan@bywatersolutions.co | |m --- Comment #26 from Brendan Gallagher <brendan@bywatersolutions.com> --- Pushed to Master - Should be in the May 2016 release. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com Severity|normal |enhancement -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=16334 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15793 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |16336 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16336 [Bug 16336] UX of holds patron search with long lists of results -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org