Bug 9674

Summary: Replace YUI buttons and menu on acquisitions pages with Bootstrap
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P5 - low CC: bgkriegel
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9674 - Replace YUI buttons and menu on acquisitions pages with Bootstrap
[SIGNED-OFF] Bug 9674 - Replace YUI buttons and menu on acquisitions pages with Bootstrap
[PASSED QA] Bug 9674 - Replace YUI buttons and menu on acquisitions pages with Bootstrap

Description Owen Leonard 2013-02-20 17:58:51 UTC
The toolbar include file (acquisitions-toolbar.inc) used on several acquisitions pages (supplier.pl, booksellers.pl, etc.) uses YUI for buttons and button menus. It should use Bootstrap instead.
Comment 1 Owen Leonard 2013-02-22 20:05:53 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2013-02-22 20:22:15 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-03-02 17:09:47 UTC
Created attachment 15827 [details] [review]
[PASSED QA] Bug 9674 - Replace YUI buttons and menu on acquisitions pages with Bootstrap

This patch converts the toolbar on acquisitions pages to Bootstrap,
replacing YUI button code with Bootstrap markup.

To test, view acquisitions pages and confirm that buttons look correct
and work correctly:

- acqui-home.pl (Acquisitions home page)
- booksellers.pl (Acquisitions -> Vendor search)
- supplier.pl (Acquisitions -> Vendor search -> Vendor)
- aqcontract.pl (Acquisitions -> Vendor search -> Vendor -> Contracts)
- uncertainprice.pl (Aquisitions -> Vendor search -> Vendor -> Uncertain
  prices)

View these pages for vendors with and without orders.

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

Comment: Buttons works. No errors.
Tested vendors with and without orders (delete button in last case)

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works nicely, changes are consistent.
Comment 4 Jared Camins-Esakov 2013-03-02 17:22:20 UTC
This patch has been pushed to master.
Comment 5 Jared Camins-Esakov 2013-03-02 17:23:26 UTC
I look forward to another follow-up that adds Bootstrap on the basket groups page.