Attachment 15439 Details for Bug 9613 – [PASSED QA] Bug 9631 - Replace YUI buttons on classification sources administration page with Bootstrap

[patch] [PASSED QA] Bug 9631 - Replace YUI buttons on classification sources administration page with Bootstrap

PASSED-QA-Bug-9631---Replace-YUI-buttons-on-classi.patch (text/plain), 2.14 KB, created by Katrin Fischer on 2013-02-17 09:02:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2013-02-17 09:02:59 UTC

Size: 2.14 KB

patch

obsolete

>From 6d2c8822206c7697fcb4983225ff58dc76ad886d Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Sat, 2 Feb 2013 18:49:52 -0500
>Subject: [PATCH] [PASSED QA] Bug 9631 - Replace YUI buttons on classification
> sources administration page with Bootstrap
>
>This patch converts the toolbar on the class sources administration page
>to Bootstrap, replacing YUI button code with Bootstrap markup.
>
>To test, view the class sources administration page. Buttons should look
>correct and work correctly.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>
>Comment: Works as described. Change "rules" to singular.
>No errors.
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Works nicely, no problems found.
>
>http://bugs.koha-community.org/show_bug.cgi?id=9613
>---
> .../prog/en/modules/admin/classsources.tt          |   15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt
>index 21eb0ea..140c033 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt
>@@ -64,12 +64,6 @@ function CheckRuleForm(f) {
>         alert(alertString2);
>     }
> }
>-
>-$(document).ready(function() {
>-    new YAHOO.widget.Button("newsource");
>-    new YAHOO.widget.Button("newrule");
>-});
>-
> //]]>
> </script>
> </head>
>@@ -232,11 +226,10 @@ $(document).ready(function() {
> 
> [% IF ( display_lists ) %]
> 
>-<div id="toolbar">
>-	<ul class="toolbar">
>-    <li><a id="newsource" href="[% script_name %]?op=add_source">New classification source</a></li>
>-    <li><a id="newrule" href="[% script_name %]?op=add_sort_rule">New filing rules</a></li>
>-</ul></div>
>+<div id="toolbar" class="btn-toolbar">
>+    <a class="btn btn-small" id="newsource" href="[% script_name %]?op=add_source"><i class="icon-plus"></i> New classification source</a>
>+    <a class="btn btn-small" id="newrule" href="[% script_name %]?op=add_sort_rule"><i class="icon-plus"></i> New filing rule</a>
>+</div>
> 
> <h2>Classification sources</h2>
> [% IF ( added_source ) %]
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 9613: 15315 | 15350 | 15439 | 16897 | 17391 | 17492