Bug 27605

Summary: Add floating toolbar to patron search page
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: caroline.cyr-la-rose, fridolin.somers, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This patch modifies the patron module's search interface so that the toolbar of results-related controls sticks to the top of the screen as the user scrolls down. This gives access to the controls for adding patrons to a list and for merging patrons.
Version(s) released in:
21.05.00
Circulation function:
Bug Depends on: 27403    
Bug Blocks:    
Attachments: Bug 27605: Add floating toolbar to patron search page
Bug 27605: Add floating toolbar to patron search page
Bug 27605: Add floating toolbar to patron search page
Bug 27605: (follow-up) Make the feature work with Bug 27403

Description Owen Leonard 2021-02-03 15:28:43 UTC
I would be useful to have a floating toolbar on the patron search results page in the patrons module so that one has access to controls for adding patrons to lists and merging patrons.
Comment 1 Owen Leonard 2021-02-03 15:58:22 UTC Comment hidden (obsolete)
Comment 2 Lucas Gass (lukeg) 2021-02-03 16:59:01 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2021-02-06 22:18:45 UTC
Created attachment 116446 [details] [review]
Bug 27605: Add floating toolbar to patron search page

This patch modifies the patron search results page so that the toolbar
with results-related controls "floats" when the user scrolls down. This
gives access to the controls for adding patrons to a list and for
merging patrons.

Other templates are modified to incorporate a change to the CSS which
changes "#searchheader" to ".searchheader," enabling multiple instances
of a <div> styled with the .searchheader class.

To test, apply the patch and rebuild the staff interface CSS
(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client).

 - Go to Patrons in the staff interface.
 - Perform a patron search which will return many results.
 - Scroll down the page far enough to trigger the floating toolbar.
 - Confirm that the toolbar is positioned correctly when paging through
   results.
 - Confirm that the patron list and merge controls still work correctly.
 - Confirm that other pages which used the "#searchheader" id are still
   styled correctly:
   - Catalog search results
   - List contents
   - Patron list contents
   - Add orders from MARC file

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2021-02-06 22:19:50 UTC
I was a little surprised that the "second" toolbar was the floating one, not the one with "Patron lists" and "New patron". But the ones floating make more sense.
Comment 5 Jonathan Druart 2021-02-08 14:18:37 UTC
It does not work with bug 27403 applied.
Comment 6 Katrin Fischer 2021-02-08 21:17:07 UTC
I've marked this one dependent, can it be fixed?
Comment 7 Owen Leonard 2021-02-08 21:49:33 UTC
Created attachment 116528 [details] [review]
Bug 27605: (follow-up) Make the feature work with Bug 27403

This patch adds the "fh-fixedHeader" class to the floating toolbar so
that the floating DataTable header knows what element to append itself
to.
Comment 8 Jonathan Druart 2021-02-09 08:28:20 UTC
Not blocker, just noting: there is a very small shift (right/left):
https://snipboard.io/LuA9Td.jpg
Comment 9 Jonathan Druart 2021-02-09 08:50:14 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 10 Fridolin Somers 2021-02-12 14:47:34 UTC
Enhancement not pushed to 20.11.x