Bug 9633 - Replace YUI button on the itemtypes administration page with Bootstrap
Summary: Replace YUI button on the itemtypes administration page with Bootstrap
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Owen Leonard
QA Contact: Katrin Fischer
URL: /cgi-bin/koha/admin/itemtypes.pl
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-15 17:46 UTC by Owen Leonard
Modified: 2014-05-26 21:05 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 9633 - Replace YUI button on the itemtypes administration page with Bootstrap (2.19 KB, patch)
2013-02-15 17:49 UTC, Owen Leonard 		Details | Diff | Splinter Review
[SIGNED-OFF] Bug 9633 - Replace YUI button on the itemtypes administration page with Bootstrap (2.27 KB, patch)
2013-02-15 19:47 UTC, Bernardo Gonzalez Kriegel 		Details | Diff | Splinter Review
[PASSED QA] Bug 9633 - Replace YUI button on the itemtypes administration page with Bootstrap (2.35 KB, patch)
2013-02-17 09:16 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2013-02-15 17:46:53 UTC 
The "New item type" button on the item types administration page is generated using YUI. It should use Bootstrap instead.
Comment 1 Owen Leonard 2013-02-15 17:49:54 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2013-02-15 19:47:09 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-02-17 09:16:40 UTC 
Created attachment 15443 [details] [review]
[PASSED QA] Bug 9633 - Replace YUI button on the itemtypes administration page with Bootstrap

This patch converts the toolbar on the itemtypes administration page to
Bootstrap, replacing YUI button code with Bootstrap markup.

Small unrelated edit: Button label text case correction.

To test, view the itemtypes administration page and confirm that the
'New item type' button looks correct and works correctly.

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

Comment: Work and look correctly. No errors.

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