Right now the table of saved SQL reports has a DataTables toolbar at the bottom showing pagination options and the search field. I think it would be useful to have it at the top too, especially to make the search field more readily accessible.
Created attachment 53329 [details] [review] Bug 16906 - Add DataTables pagination and filter to top of saved SQL reports table This patch adds a DataTables toolbar to the top of the saved SQL reports table so that pagination and filtering controls are at the top as well as the bottom. To test, apply the patch and go to Reports -> Saved reports. There should be controls at the top of the table of saved reports for setting number of results, pagination, and searching.
Created attachment 53336 [details] [review] Bug 16906 - Add DataTables pagination and filter to top of saved SQL reports table This patch adds a DataTables toolbar to the top of the saved SQL reports table so that pagination and filtering controls are at the top as well as the bottom. To test, apply the patch and go to Reports -> Saved reports. There should be controls at the top of the table of saved reports for setting number of results, pagination, and searching. Signed-off-by: Marc Véron <veron@veron.ch>
Owen, any reasons not to remove the sDom definition to use the default one?
(In reply to Jonathan Druart from comment #3) > Owen, any reasons not to remove the sDom definition to use the default one? No I guess not.
Created attachment 53525 [details] [review] Bug 16906 - Add DataTables pagination and filter to top of saved SQL reports table This patch adds a DataTables toolbar to the top of the saved SQL reports table so that pagination and filtering controls are at the top as well as the bottom. To test, apply the patch and go to Reports -> Saved reports. There should be controls at the top of the table of saved reports for setting number of results, pagination, and searching. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 53526 [details] [review] Bug 16906: Simply remove the sDom definition to use the default one Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 16.11, thanks Owen!
Pushed in 16.05. Will be in 16.05.02.