Bug 9635

Summary: Replace YUI buttons on label batches pages with Bootstrap
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: enhancement    
Priority: P5 - low CC: bgkriegel
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/labels/label-edit-batch.pl?op=edit&amp;element_id=X
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 9635 - Replace YUI buttons on label batches pages with Bootstrap
[SIGNED-OFF] Bug 9635 - Replace YUI buttons on label batches pages with Bootstrap
[PASSED QA] Bug 9635 - Replace YUI buttons on label batches pages with Bootstrap

Description Owen Leonard 2013-02-15 18:10:12 UTC
The toolbar buttons on the label batch pages are generated using YUI. They should use Bootstrap instead.
Comment 1 Owen Leonard 2013-02-15 19:22:33 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2013-02-15 19:57:25 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-02-17 12:53:25 UTC
Created attachment 15464 [details] [review]
[PASSED QA] Bug 9635 - Replace YUI buttons on label batches pages with Bootstrap

This patch converts the toolbar used by label batches pages to
Bootstrap, replacing YUI button and menu code with Bootstrap markup.

This patch also consolidates the include file and main template. The
include file was used in only one template.

To test, create a label batch and test all functions related to that
batch: adding and removing items, de-duplicating the batch, exporting
individual items, exporting the batch, and deleting the batch. Buttons
should look correct and work correctly.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Comment: Work as described, no errors.
Tested all functions (add, del, de-duplicate, export item, export batch)

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works nicely, no problems found.
Comment 4 Jared Camins-Esakov 2013-02-18 13:56:08 UTC
This patch has been pushed to master.