We need the page-section div when placing holds to ensure proper contrast for even/odd rows on the tables. To replicate: 1. Place a hold or an item. 2. On the details page for a record select the holds section/tab.
Are you referring to this page? /cgi-bin/koha/reserve/request.pl?biblionumber=X&borrowernumber=X I don't think I'm seeing the issue.
Created attachment 142242 [details] Screenshot - Record details > Holds On the Record details > Holds page: http://127.0.0.1:8081/cgi-bin/koha/reserve/request.pl?biblionumber=126
Created attachment 142309 [details] [review] Bug 31806: Add page-section to holds tables This patch adds the page-section class to each holds view on reserve/request.pl Test plan 1) Find an biblio to test on and add holds items of different types and homebranches. 2) Work through each setting of the HoldsSplitQueue preference and confirm the page-sections appear as expected
I wasn't entirely sure where to put the page-section on this one.. especially given the various options for HoldsSplitQueue. At the moment, I've sectioned off each 'Split' as governed by HoldsSplitQueue.. but the argument could be made that the whole 'Existing holds' form could be a single page-section container? Also.. do the buttons look a bit crowded together for anyone else.. probabably a different bug entirely, but I think the markup there might be wrong too.
Created attachment 142310 [details] [review] Bug 31806: Add page-section container to existing holds This patch adds a page-section container around the existing holds content section
OK.. after a little more testing I decided a container around the whole existing holds section feels like it makes more sense.. second patch attached does this so you can test one after the other if you'd like to see the distinction.
Created attachment 142468 [details] [review] Bug 31806: Add page-section to holds tables This patch adds the page-section class to each holds view on reserve/request.pl Test plan 1) Find an biblio to test on and add holds items of different types and homebranches. 2) Work through each setting of the HoldsSplitQueue preference and confirm the page-sections appear as expected Signed-off-by: David Nind <david@davidnind.com>
Created attachment 142469 [details] [review] Bug 31806: Add page-section container to existing holds This patch adds a page-section container around the existing holds content section Signed-off-by: David Nind <david@davidnind.com>
Created attachment 142470 [details] Screenshots - options and display of holds page I've attached a PDF with screenshots of the options for HoldsSplitQueue. Thanks Martin for logging separate bugs for the table heading row and button placement - I've added a comment about how the light grey background continues after the buttons. Testing notes: - For item types to display: . I added an item to the record I tested this on (books) and made it a Music item type . Placed record level and item level holds
Created attachment 142527 [details] [review] Bug 31806: Add page-section to holds tables This patch adds the page-section class to each holds view on reserve/request.pl Test plan 1) Find an biblio to test on and add holds items of different types and homebranches. 2) Work through each setting of the HoldsSplitQueue preference and confirm the page-sections appear as expected Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Created attachment 142528 [details] [review] Bug 31806: Add page-section container to existing holds This patch adds a page-section container around the existing holds content section Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Pushed to master for 22.11. Nice work everyone, thanks!