I hear some libraries do use it, so why not bring the printers administration page up to date? I have a patch to add DataTables, Font awesome icons, and plugin-validated forms.
Created attachment 52500 [details] [review] Bug 16764 - Update printers administration page This patch updates the template for printers administration according to current interface guidelines. - Custom JavaScript form validation is removed in favor of the built-in form validation plugin. - Printer-specific header search form is removed, along with server-side pagination code. The functionality of both is now taken care of the adding DataTables sorting and pagination to the table. - Edit and delete links are styled as Boostrap buttons with Font Awesome icons. - Confirmation dialogs are updated to use Font Awesome icons. To test properly you will need to add multiple printers. - Apply the patch and open /cgi-bin/koha/admin/printers.pl (there is no menu link for the page). - When no printers exist you should see a message dialog saying so. - Add a new printer. Confirm that printer name and queue are required fields. - After adding multiple printers, confirm that the table of printers is correctly sorted and searched by the DataTables plugin. - Delete a printer and verify that the confirmation dialog is correctly styled.
In case it helps with testing: INSERT INTO `printers` (`printername`, `printqueue`, `printtype`) VALUES ('Printer 01', 'print_queue_01', 'type01'), ('Printer 02', 'print_queue_02', 'type02'), ('Printer 03', 'print_queue_03', 'type03'), ('Printer 04', 'print_queue_04', 'type04'), ('Printer 05', 'print_queue_05', 'type05'), ('Printer 06', 'print_queue_06', 'type06'), ('Printer 07', 'print_queue_07', 'type07'), ('Printer 08', 'print_queue_08', 'type08'), ('Printer 09', 'print_queue_09', 'type09'), ('Printer 10', 'print_queue_10', 'type10'), ('Printer 11', 'print_queue_11', 'type11'), ('Printer 12', 'print_queue_12', 'type12'), ('Printer 13', 'print_queue_13', 'type13'), ('Printer 14', 'print_queue_14', 'type14'), ('Printer 15', 'print_queue_15', 'type15'), ('Printer 16', 'print_queue_16', 'type16'), ('Printer 17', 'print_queue_17', 'type17'), ('Printer 18', 'print_queue_18', 'type18'), ('Printer 19', 'print_queue_19', 'type19'), ('Printer 20', 'print_queue_20', 'type20'), ('Printer 21', 'print_queue_21', 'type21'), ('Printer 22', 'print_queue_22', 'type22'), ('Printer 23', 'print_queue_23', 'type23'), ('Printer 24', 'print_queue_24', 'type24'), ('Printer 25', 'print_queue_25', 'type25');
Created attachment 52511 [details] [review] Bug 16764 - Update printers administration page This patch updates the template for printers administration according to current interface guidelines. - Custom JavaScript form validation is removed in favor of the built-in form validation plugin. - Printer-specific header search form is removed, along with server-side pagination code. The functionality of both is now taken care of the adding DataTables sorting and pagination to the table. - Edit and delete links are styled as Boostrap buttons with Font Awesome icons. - Confirmation dialogs are updated to use Font Awesome icons. To test properly you will need to add multiple printers. - Apply the patch and open /cgi-bin/koha/admin/printers.pl (there is no menu link for the page). - When no printers exist you should see a message dialog saying so. - Add a new printer. Confirm that printer name and queue are required fields. - After adding multiple printers, confirm that the table of printers is correctly sorted and searched by the DataTables plugin. - Delete a printer and verify that the confirmation dialog is correctly styled. Signed-off-by: Marc Véron <veron@veron.ch>
Bug 8352 revamps the printer administration as well.
(In reply to Kyle M Hall from comment #4) > Bug 8352 revamps the printer administration as well. Yes but it is in discussion and does no longer apply. What do you suggest?
(In reply to Jonathan Druart from comment #5) > (In reply to Kyle M Hall from comment #4) > > Bug 8352 revamps the printer administration as well. > > Yes but it is in discussion and does no longer apply. What do you suggest? Since this is a purely cosmetic change, I would appreciate if we held off on it for bug 8352. I haven't had time to get back to that one recently, but I'm still pursuing it. The blocker on it is getting the needed Perl libs packaged for Debian. I really need s For Dummies version of getting libraries packaged, but alas, there is none ; )
Created attachment 69141 [details] [review] Bug 16764 - Update printers administration page This patch updates the template for printers administration according to current interface guidelines. - Custom JavaScript form validation is removed in favor of the built-in form validation plugin. - Printer-specific header search form is removed, along with server-side pagination code. The functionality of both is now taken care of the adding DataTables sorting and pagination to the table. - Edit and delete links are styled as Boostrap buttons with Font Awesome icons. - Confirmation dialogs are updated to use Font Awesome icons. To test properly you will need to add multiple printers. - Apply the patch and open /cgi-bin/koha/admin/printers.pl (there is no menu link for the page). - When no printers exist you should see a message dialog saying so. - Add a new printer. Confirm that printer name and queue are required fields. - After adding multiple printers, confirm that the table of printers is correctly sorted and searched by the DataTables plugin. - Delete a printer and verify that the confirmation dialog is correctly styled. Signed-off-by: Marc Véron <veron@veron.ch> Update: rebased and updated Bootstrap 3 classes
I'd like to put this back in the queue since Bug 8352 has been idle for so long.
Created attachment 70090 [details] [review] Bug 16764 - Update printers administration page This patch updates the template for printers administration according to current interface guidelines. - Custom JavaScript form validation is removed in favor of the built-in form validation plugin. - Printer-specific header search form is removed, along with server-side pagination code. The functionality of both is now taken care of the adding DataTables sorting and pagination to the table. - Edit and delete links are styled as Boostrap buttons with Font Awesome icons. - Confirmation dialogs are updated to use Font Awesome icons. To test properly you will need to add multiple printers. - Apply the patch and open /cgi-bin/koha/admin/printers.pl (there is no menu link for the page). - When no printers exist you should see a message dialog saying so. - Add a new printer. Confirm that printer name and queue are required fields. - After adding multiple printers, confirm that the table of printers is correctly sorted and searched by the DataTables plugin. - Delete a printer and verify that the confirmation dialog is correctly styled. Signed-off-by: Marc Véron <veron@veron.ch> Update: rebased and updated Bootstrap 3 classes Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> https://bugs.koha-community.org/show_bug.cgi?id=16765
Pushed to master for 18.05, thanks to everybody involved!
Awesome work all, pushed to stable for 17.11.02
Enhancement not pushed to 17.05.x