Attachment 15457 Details for Bug 9637 – [PASSED QA] Bug 9637 - Replace YUI button on local use system preferences page with Bootstrap

[patch] [PASSED QA] Bug 9637 - Replace YUI button on local use system preferences page with Bootstrap

PASSED-QA-Bug-9637---Replace-YUI-button-on-local-u.patch (text/plain), 3.10 KB, created by Katrin Fischer on 2013-02-17 11:58:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2013-02-17 11:58:57 UTC

Size: 3.10 KB

patch

obsolete

>From 087e1f4e7ef562cf55e94e4c67d6ff3150fd57a7 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Sun, 3 Feb 2013 09:33:45 -0500
>Subject: [PATCH] [PASSED QA] Bug 9637 - Replace YUI button on local use
> system preferences page with Bootstrap
>
>This patch converts the toolbar on the local use system preferences page
>to Bootstrap, replacing YUI button code with Bootstrap markup.
>
>To test, view the local use system preferences page (the "local use" tab
>on the main System preferences page) and confirm that the "New
>preference" button looks correct and works correctly.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>
>Comment: Works and looks very well. No errors.
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Works nicely, no problems found.
>---
> .../prog/en/modules/admin/systempreferences.tt     |   41 +++++++++-----------
> 1 file changed, 19 insertions(+), 22 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/systempreferences.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/systempreferences.tt
>index 8e69259..874708a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/systempreferences.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/systempreferences.tt
>@@ -70,8 +70,22 @@
>                 return false;
>             } ).nextAll( 'textarea, input[type=submit]' ).hide();
>         } );
>+
>         //]]>
> </script>
>+[% IF ( else ) %]
>+    <script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.min.js"></script>
>+    <script type="text/javascript">
>+    //<![CDATA[
>+    // prepare DOM for YUI Toolbar
>+     $(document).ready(function() {
>+        [% IF ( loop ) %]$("#sysprefst").tablesorter({
>+            sortList: [[0,0]],
>+            headers: { 1: {sorter:false},2: { sorter: false },3: { sorter: false },4: { sorter: false }}
>+        });[% END %]
>+     });
>+    </script>
>+[% END %]
> </head>
> <body id="admin_systempreferences" class="admin">
> [% INCLUDE 'header.inc' %]
>@@ -240,28 +254,11 @@
>     [% END %]
>    
>     [% IF ( else ) %]
>-    
>-    <div id="toolbar">
>-    <script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.min.js"></script>
>-    <script type="text/javascript">
>-    //<![CDATA[
>-    // prepare DOM for YUI Toolbar
>-     $(document).ready(function() {
>-        yuiToolbar();
>-        [% IF ( loop ) %]$("#sysprefst").tablesorter({
>-            sortList: [[0,0]],
>-            headers: { 1: {sorter:false},2: { sorter: false },3: { sorter: false },4: { sorter: false }}
>-        });[% END %]
>-     });
>-    // YUI Toolbar Functions
>-    function yuiToolbar() {
>-        new YAHOO.widget.Button("newstopword");
>-    }   //]]>
>-    </script>
>-    <ul class="toolbar">
>-        <li><a id="newstopword" href="[% script_name %]?op=add_form">New preference</a></li>
>-    </ul></div>
>-    
>+
>+    <div id="toolbar" class="btn-toolbar">
>+        <a class="btn btn-small" id="newstopword" href="[% script_name %]?op=add_form"><i class="icon-plus"></i> New preference</a>
>+    </div>
>+
>     <h1>System preferences admin</h1>
>     <h2>[% tab %] preferences</h2>
>     [% searchfield %]
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 9637: 15400 | 15409 | 15457