The styling on the paging options for the course reserves tables in OPAC is broken.
Caused by bug 25287.
(In reply to Jonathan Druart from comment #1) > Caused by bug 25287. How is that?
I don't know, git bisect. Certainly the changes to datatables.js
Ok, I've checked and we don't do pagination in the OPAC, neither on course reserves or items/holds. So this is purely styling issue (i.e. we added the elements but we shouldn't have).
Created attachment 109189 [details] [review] Bug 26262: Remove wrong dom setting in course reserves (OPAC) The current setting required pagination to be displayed. This was hidden probably as a side effect of styling settings. But the docs are clear on the fact that p and l are intended for displaying pagination controls and table rows lenght information [1] As the course reserves datatable is not paginated, this needs to be fixed properly by removing those controls in the configuration. This patch does so. To test: 1. Have some course reserves 2. See them in the OPAC => FAIL: Some funky, misaligned pagination artifacts are displayed 3. Apply this patch and reload => SUCCESS: Things are back to normal! 4. Sign off :-D [1] https://datatables.net/examples/basic_init/dom.html Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 109362 [details] [review] Bug 26262: Remove wrong dom setting in course reserves (OPAC) The current setting required pagination to be displayed. This was hidden probably as a side effect of styling settings. But the docs are clear on the fact that p and l are intended for displaying pagination controls and table rows lenght information [1] As the course reserves datatable is not paginated, this needs to be fixed properly by removing those controls in the configuration. This patch does so. To test: 1. Have some course reserves 2. See them in the OPAC => FAIL: Some funky, misaligned pagination artifacts are displayed 3. Apply this patch and reload => SUCCESS: Things are back to normal! 4. Sign off :-D [1] https://datatables.net/examples/basic_init/dom.html Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 109363 [details] [review] Bug 26262: (follow-up) Fix course details page, default configuration Bug 25287 changed the default DataTable configuration in the OPAC so that tables are paginated by default. This is inconsistent with the way we currently use DataTables in the OPAC, and the change was probably unintentional. This patch returns the default pagination configuration to what it was before: Off. The patch also updates opac-course-details.tt with the same DOM configuration as opac-course-reserves.tt. To test, apply the patch and view the list of course reserves and the list of titles attached to a course. In both cases there should be no pagination controls, and the tables should show all results.
Created attachment 109673 [details] [review] Bug 26262: (follow-up) Fix course details page, default configuration Bug 25287 changed the default DataTable configuration in the OPAC so that tables are paginated by default. This is inconsistent with the way we currently use DataTables in the OPAC, and the change was probably unintentional. This patch returns the default pagination configuration to what it was before: Off. The patch also updates opac-course-details.tt with the same DOM configuration as opac-course-reserves.tt. To test, apply the patch and view the list of course reserves and the list of titles attached to a course. In both cases there should be no pagination controls, and the tables should show all results. Signed-off-by: Alexis Ripetti <alexis.ripetti@inLibro.com>
Created attachment 109831 [details] [review] Bug 26262: Remove wrong dom setting in course reserves (OPAC) The current setting required pagination to be displayed. This was hidden probably as a side effect of styling settings. But the docs are clear on the fact that p and l are intended for displaying pagination controls and table rows lenght information [1] As the course reserves datatable is not paginated, this needs to be fixed properly by removing those controls in the configuration. This patch does so. To test: 1. Have some course reserves 2. See them in the OPAC => FAIL: Some funky, misaligned pagination artifacts are displayed 3. Apply this patch and reload => SUCCESS: Things are back to normal! 4. Sign off :-D [1] https://datatables.net/examples/basic_init/dom.html Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 109832 [details] [review] Bug 26262: (follow-up) Fix course details page, default configuration Bug 25287 changed the default DataTable configuration in the OPAC so that tables are paginated by default. This is inconsistent with the way we currently use DataTables in the OPAC, and the change was probably unintentional. This patch returns the default pagination configuration to what it was before: Off. The patch also updates opac-course-details.tt with the same DOM configuration as opac-course-reserves.tt. To test, apply the patch and view the list of course reserves and the list of titles attached to a course. In both cases there should be no pagination controls, and the tables should show all results. Signed-off-by: Alexis Ripetti <alexis.ripetti@inLibro.com>
Created attachment 109989 [details] [review] Bug 26262: Remove wrong dom setting in course reserves (OPAC) The current setting required pagination to be displayed. This was hidden probably as a side effect of styling settings. But the docs are clear on the fact that p and l are intended for displaying pagination controls and table rows lenght information [1] As the course reserves datatable is not paginated, this needs to be fixed properly by removing those controls in the configuration. This patch does so. To test: 1. Have some course reserves 2. See them in the OPAC => FAIL: Some funky, misaligned pagination artifacts are displayed 3. Apply this patch and reload => SUCCESS: Things are back to normal! 4. Sign off :-D [1] https://datatables.net/examples/basic_init/dom.html Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 109990 [details] [review] Bug 26262: (follow-up) Fix course details page, default configuration Bug 25287 changed the default DataTable configuration in the OPAC so that tables are paginated by default. This is inconsistent with the way we currently use DataTables in the OPAC, and the change was probably unintentional. This patch returns the default pagination configuration to what it was before: Off. The patch also updates opac-course-details.tt with the same DOM configuration as opac-course-reserves.tt. To test, apply the patch and view the list of course reserves and the list of titles attached to a course. In both cases there should be no pagination controls, and the tables should show all results. Signed-off-by: Alexis Ripetti <alexis.ripetti@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 20.11, thanks to everybody involved!
missing dependencies for 20.05.x, no backport