Bug 9694

Summary: Remove YUI styling from buttons on the patron card creator manage pages
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/patroncards/manage.pl?card_element=
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 9694 - Remove YUI styling from buttons on the patron card creator manage pages
Bug 9694 - Remove YUI styling from buttons on the patron card creator manage pages
[SIGNED-OFF] Bug 9694 - Remove YUI styling from buttons on the patron card creator manage pages
[PASSED QA] Bug 9694 - Remove YUI styling from buttons on the patron card creator manage pages

Description Owen Leonard 2013-02-22 14:24:20 UTC
Several buttons on the patron card creator's manage page (used to manage layouts, profiles, etc) are styled using YUI. Typically we don't apply special button styling to submit buttons, so to make this page consistent with others in Koha I propose to remove YUI styling.
Comment 1 Owen Leonard 2013-02-22 14:30:59 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2013-02-22 15:40:16 UTC Comment hidden (obsolete)
Comment 3 Bernardo Gonzalez Kriegel 2013-02-22 17:56:32 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-03-05 07:27:38 UTC
Created attachment 15875 [details] [review]
[PASSED QA] Bug 9694 - Remove YUI styling from buttons on the patron card creator manage pages

Buttons on the patron card creator's manage pages (manage templates,
manage profiles, etc) are styled using YUI. They can be classified as
submit buttons rather than toolbar buttons, so they should have default
submit button styles.

This patch removes YUI styling and related JavaScript. Other minor
changes: Escaping strings in JavaScript for translation.

To test, view the patron card creator's manage page for layouts,
templates, profiles, and batches. "Edit," "Delete," and "Export" buttons
should look correct and work correctly.

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

Comment: Work as described. On errors.
Tested all manage pages, edit, delete, and export buttons.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works nicely, improves consistency.
Comment 5 Jared Camins-Esakov 2013-03-07 14:57:54 UTC
This patch has been pushed to master.