Bug 9621

Summary: Replace YUI menu on lists 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/virtualshelves/shelves.pl
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 9420 - Work on Bootstrap for Koha: Lists toolbar
Bug 9621 - Replace YUI menu on lists pages with Bootstrap
[SIGNED-OFF] Bug 9621 - Replace YUI menu on lists pages with Bootstrap
[PASSED QA] Bug 9621 - Replace YUI menu on lists pages with Bootstrap

Description Owen Leonard 2013-02-15 15:49:11 UTC
The toolbar include file used by staff client lists pages uses YUI to generate buttons and menus. It should use Bootstrap instead.
Comment 1 Owen Leonard 2013-02-15 16:07:01 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2013-02-15 16:14:08 UTC Comment hidden (obsolete)
Comment 3 Bernardo Gonzalez Kriegel 2013-02-15 16:29:56 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-02-17 09:35:15 UTC
Created attachment 15447 [details] [review]
[PASSED QA] Bug 9621 - Replace YUI menu on lists pages with Bootstrap

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

To test, view any page in virtualshelves/ which uses
virtualshelves-toolbar.inc (shelves.pl). Buttons and menus should look
correct and work correctly when viewing own lists, others' lists, etc.

Revised: corrected bug number and title
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Comment: Work as described. No error. Looks very good.
As well as Bug 9616 the result is 10px downward from the original.

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