Bug 37730 fixed problems with the batch patron modification DataTable when scrolling horizontally and vertically. Since the DataTables upgrade there are new problems, so I propose we remove the container around the table which has "overflow:auto" and the DataTables option "scrollX". This will make the page behave like others, where wide tables cause a full page horizontal scroll.
Created attachment 176437 [details] [review] Bug 38870: Remove overflow scroll from batch patron modification This patch removes the <div> surrounding batch patron modification results with the style "overflow:auto". Also removed is the DataTable option "scrollX". This eliminates the horizontal scrolling for the table itself, and causes the whole page to scroll horizontally instead. This is consistent with other pages in Koha. This fix lets us avoid bugs which are associated with the overflow scrolling and our DataTables implementation. (see Bug 37730). To test, apply the patch and go to Tools -> Batch patron modification - Submit a batch of patrons for modification. - On the screen showing the patrons which will be modified, the page will probably have a horizontal scrollbar already. If not, resize your browser to confirm that the whole page scrolls, not just the table. - Confirm that the table header "sticks" to the top of the screen when you scroll down, and that the header continues to be aligned correctly when you scroll horizontally. Sponsored-by: Athens County Public Libraries
scrollX's behaviour is better than this. But at least it's now consistent for all the other tables. If we want to fix that globally we should investigate a bit better how to fix this problem. For now I would vote for consistency and +1 Owen's patch.
Created attachment 176451 [details] [review] Bug 38870: Remove overflow scroll from batch patron modification This patch removes the <div> surrounding batch patron modification results with the style "overflow:auto". Also removed is the DataTable option "scrollX". This eliminates the horizontal scrolling for the table itself, and causes the whole page to scroll horizontally instead. This is consistent with other pages in Koha. This fix lets us avoid bugs which are associated with the overflow scrolling and our DataTables implementation. (see Bug 37730). To test, apply the patch and go to Tools -> Batch patron modification - Submit a batch of patrons for modification. - On the screen showing the patrons which will be modified, the page will probably have a horizontal scrollbar already. If not, resize your browser to confirm that the whole page scrolls, not just the table. - Confirm that the table header "sticks" to the top of the screen when you scroll down, and that the header continues to be aligned correctly when you scroll horizontally. Sponsored-by: Athens County Public Libraries Signed-off-by: William Lavoie <william.lavoie@inLibro.com>
Created attachment 176947 [details] [review] Bug 38870: Remove overflow scroll from batch patron modification This patch removes the <div> surrounding batch patron modification results with the style "overflow:auto". Also removed is the DataTable option "scrollX". This eliminates the horizontal scrolling for the table itself, and causes the whole page to scroll horizontally instead. This is consistent with other pages in Koha. This fix lets us avoid bugs which are associated with the overflow scrolling and our DataTables implementation. (see Bug 37730). To test, apply the patch and go to Tools -> Batch patron modification - Submit a batch of patrons for modification. - On the screen showing the patrons which will be modified, the page will probably have a horizontal scrollbar already. If not, resize your browser to confirm that the whole page scrolls, not just the table. - Confirm that the table header "sticks" to the top of the screen when you scroll down, and that the header continues to be aligned correctly when you scroll horizontally. Sponsored-by: Athens County Public Libraries Signed-off-by: William Lavoie <william.lavoie@inLibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
We didn't get feedback from the people involved on bug 37730 so I am passing QA on this.
Pushed for 25.05! Well done everyone, thank you!
Nice work everyone! Pushed to 24.11.x for 24.11.04