Attachment 83606 Details for Bug 21438 – Bug 21438: Update two-column templates with Bootstrap grid: Patron card creator

[patch] Bug 21438: Update two-column templates with Bootstrap grid: Patron card creator

Bug-21438-Update-two-column-templates-with-Bootstr.patch (text/plain), 17.87 KB, created by Josef Moravec on 2019-01-02 21:02:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2019-01-02 21:02:56 UTC

Size: 17.87 KB

patch

obsolete

>From 4fc67374a85a007a166d47fa4f0ff9674ab3a66f Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 2 May 2018 18:33:04 +0000
>Subject: [PATCH] Bug 21438: Update two-column templates with Bootstrap grid:
> Patron card creator
>
>This patch modifies the patron card creator templates to use the
>Bootstrap grid instead of YUI.
>
>To test, apply the patch and view the following pages, confirming that
>they look correct at various browser widths:
>
> - Tools -> patron card creator
>   - Manage -> Card batches
>     - Edit card batch
>   - Manage -> Layouts
>     - Edit layout
>   - Manage -> Images
>   - Manage -> Card templates
>     - Edit template
>   - Manage -> Printer profiles
>     - Edit profile
>
>Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>---
> .../prog/en/modules/patroncards/edit-batch.tt      | 30 ++++++------
> .../prog/en/modules/patroncards/edit-layout.tt     | 37 +++++++-------
> .../prog/en/modules/patroncards/edit-profile.tt    | 34 +++++++------
> .../prog/en/modules/patroncards/edit-template.tt   | 56 +++++++++++-----------
> .../prog/en/modules/patroncards/home.tt            | 36 +++++++-------
> .../prog/en/modules/patroncards/image-manage.tt    | 37 +++++++-------
> .../prog/en/modules/patroncards/manage.tt          | 30 +++++++-----
> 7 files changed, 134 insertions(+), 126 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
>index ff26bef..c3310f4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
>@@ -23,16 +23,13 @@
>             New
>         [% END %]
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-            <div id="yui-main">
>-                <div class="yui-b">
>-                    <div class="yui-g">
>-                    [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
> 
>-                        </div>
>-                    </div>
>+    <div class="main container-fluid">
>+        <div class="row">
>+            <div class="col-sm-10 col-sm-push-2">
>+                <main>
>+
>+                    [% INCLUDE 'patroncards-toolbar.inc' %]
> 
>                 [% IF ( duplicate_message ) %]
>                     <div class="dialog message">
>@@ -124,13 +121,16 @@
>                                 </fieldset>
>                             [% END %]
>                         </div>
>-                    </div>
>-                </div>
>-                <div class="yui-b">
>-                  [% INCLUDE 'tools-menu.inc' %]
>-                </div>
>-            </div>
> 
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'greybox.inc' %]
>     [% INCLUDE 'datatables.inc' %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-layout.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-layout.tt
>index b93385b..8a5c487 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-layout.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-layout.tt
>@@ -14,21 +14,18 @@
>         <a href="/cgi-bin/koha/patroncards/manage.pl?card_element=layout">Layouts</a> &rsaquo;
>         [% IF ( layout_id ) %]Edit ([% layout_id | html %])[% ELSE %]New[% END %]
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-            <div id="yui-main">
>-                <div class="yui-b">
>-                    <div class="yui-g">
>-                    [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
> 
>-                        </div>
>-                    </div>
>+    <div class="main container-fluid">
>+        <div class="row">
>+            <div class="col-sm-10 col-sm-push-2">
>+                <main>
>+
>+                    [% INCLUDE 'patroncards-toolbar.inc' %]
> 
>                 
>                 <form name="input" action="/cgi-bin/koha/patroncards/edit-layout.pl" method="get">
>-                <div class="yui-g">
>-                <div class="yui-g first">
>+                <div class="row">
>+                <div class="col-sm-6">
>                         <fieldset class="rows">
>                             <legend>[% IF ( layout_id ) %]Edit[% ELSE %]Create[% END %] patron card text layout</legend>
>                             <ol>
>@@ -259,7 +256,7 @@
>                             </ol>
>                         </fieldset>
>                 </div>
>-                <div class="yui-g">
>+                <div class="col-sm-6">
>                     <fieldset class="rows">
>                     <legend>[% IF ( layout_id ) %]Edit[% ELSE %]Create[% END %] patron card graphic layout</legend>
>                     <ol>
>@@ -460,12 +457,16 @@
>                         <input type="hidden" name="layout_id" value="[% layout_id | html %]" />
>                     </fieldset>
>                 </form>
>-                </div>
>-            </div>
>-            <div class="yui-b">
>-              [% INCLUDE 'tools-menu.inc' %]
>-            </div>
>-        </div>
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     <script>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-profile.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-profile.tt
>index 0b5e8c9..d6ee17c 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-profile.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-profile.tt
>@@ -14,19 +14,15 @@
>         <a href="/cgi-bin/koha/patroncards/manage.pl?card_element=profile">Profiles</a> &rsaquo;
>         [% IF ( profile_id ) %]Edit ([% profile_id | html %])[% ELSE %]New[% END%]
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-        <div id="yui-main">
>-            <div class="yui-b">
>-                    <div class="yui-g">
>+
>+    <div class="main container-fluid">
>+        <div class="row">
>+            <div class="col-sm-10 col-sm-push-2">
>+                <main>
>+
>                     [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
> 
>-                        </div>
>-                    </div>
>-                        <div class="yui-g">
>                             <h3>[% IF (profile_id) %]Edit[% ELSE %]Create[% END%] printer profile[% IF (profile_id) %] ([% profile_id | html %])[% END %]</h3>
>-                                <div class="yui-g first">
>                                     <form name="input" action="/cgi-bin/koha/patroncards/edit-profile.pl" method="get">
>                                     <fieldset class="rows"><legend>Profile settings</legend>
>                                         <ol>
>@@ -98,14 +94,16 @@
>                                         <input type="hidden" name="profile_id" value="[% profile_id | html %]" />
>                                     </fieldset>
>                     </form>
>-                                </div>
>-                            </div>
>-                        </div>
>-                </div>
>-                <div class="yui-b">
>-                  [% INCLUDE 'tools-menu.inc' %]
>-                </div>
>-            </div>
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     <script>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-template.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-template.tt
>index b561bbd..7d73d8b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-template.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-template.tt
>@@ -16,20 +16,17 @@
>         <a href="/cgi-bin/koha/patroncards/manage.pl?card_element=template">Templates</a> &rsaquo;
>         [% IF (template_id) %]Edit ([% template_id | html %])[% ELSE %]New[% END %]
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-            <div id="yui-main">
>-                <div class="yui-b">
>-                    <div class="yui-g">
>-                    [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
> 
>-                        </div>
>-                    </div>
>+    <div class="main container-fluid">
>+        <div class="row">
>+            <div class="col-sm-10 col-sm-push-2">
>+                <main>
>+
>+                    [% INCLUDE 'patroncards-toolbar.inc' %]
>                     <form name="input" action="/cgi-bin/koha/patroncards/edit-template.pl" method="get">
>-                    <div class="yui-g">
>                         <h3>[% IF (template_id) %]Edit[% ELSE %]Create[% END %] patron card template[% IF (template_id) %] ([% template_id | html %])[% END %]</h3>
>-                        <div class="yui-u first">
>+                        <div class="row">
>+                        <div class="col-sm-6">
>                             <fieldset class="rows">
>                                 <ol>
>                                     <li>
>@@ -92,7 +89,7 @@
>                                 </ol>
>                             </fieldset>
>                         </div>
>-                        <div class="yui-u">
>+                        <div class="col-sm-6">
>                             <fieldset class="rows">
>                                 <ol><li>
>                                     <label for="top_margin">Top page margin:</label>
>@@ -141,23 +138,26 @@
>                             </fieldset>
>                         </div>
>                     </div>
>-                    <div class="yui-g">
>-                        <fieldset class="action">
>-                            <input type="submit" class="submit" value="Save" />
>-                            <a class="cancel" href="/cgi-bin/koha/patroncards/manage.pl?card_element=template">Cancel</a>
>-                            <input type="hidden" name="op" value="save" />
>-                            [% IF ( template_id ) %]
>-                            <input type="hidden" name="template_id" value="[% template_id | html %]" />
>-                            [% END %]
>-                        </fieldset>
>-                    </div>
>+
>+                    <fieldset class="action">
>+                        <input type="submit" class="submit" value="Save" />
>+                        <a class="cancel" href="/cgi-bin/koha/patroncards/manage.pl?card_element=template">Cancel</a>
>+                        <input type="hidden" name="op" value="save" />
>+                        [% IF ( template_id ) %]
>+                        <input type="hidden" name="template_id" value="[% template_id | html %]" />
>+                        [% END %]
>+                    </fieldset>
>                 </form>
>-            </div>
>-        </div>
>-        <div class="yui-b">
>-          [% INCLUDE 'tools-menu.inc' %]
>-        </div>
>-    </div>
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% Asset.js("js/tools-menu.js") | $raw %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/home.tt
>index a251810..a87f597 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/home.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/home.tt
>@@ -10,19 +10,23 @@
>         <a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a> &rsaquo;
>         Patron card creator
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-            <div id="yui-main">
>-                <div class="yui-b">
>-                    <div class="yui-g">
>-                    [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
>-                        </div>
>-                    </div>
>-                </div>
>-            </div>
>-            <div class="yui-b">
>-              [% INCLUDE 'tools-menu.inc' %]
>-            </div>
>-        </div>
>-        [% INCLUDE 'intranet-bottom.inc' %]
>+
>+<div class="main container-fluid">
>+    <div class="row">
>+        <div class="col-sm-10 col-sm-push-2">
>+            <main>
>+
>+                [% INCLUDE 'patroncards-toolbar.inc' %]
>+
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
>+
>+[% INCLUDE 'intranet-bottom.inc' %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/image-manage.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/image-manage.tt
>index fc9bed9..22fcf76 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/image-manage.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/image-manage.tt
>@@ -13,19 +13,16 @@
>     <a href="/cgi-bin/koha/patroncards/home.pl">Patron card creator</a> &rsaquo;
>     Images
> </div>
>-<div id="doc3" class="yui-t2">
>-    <div id="bd">
>-        <div id="yui-main">
>-            <div class="yui-b">
>-                    <div class="yui-g">
>-                    [% INCLUDE 'patroncards-toolbar.inc' %]
>-                        <div class="yui-u first">
> 
>-                        </div>
>-                    </div>
>+<div class="main container-fluid">
>+    <div class="row">
>+        <div class="col-sm-10 col-sm-push-2">
>+            <main>
>+
>+                [% INCLUDE 'patroncards-toolbar.inc' %]
>                 [% INCLUDE 'patroncards-errors.inc' %]
>-                <div class="yui-g">
>-                    <div class="yui-u first">
>+                <div class="row">
>+                    <div class="col-sm-6">
>                         <h1>Upload additional images for patron cards</h1>
>                         <p>Manage additional images to use as logo, decoration or background on a patron card layout.</p>
>                         <form name="upload_images" method="post" action="/cgi-bin/koha/patroncards/image-manage.pl" enctype="multipart/form-data">
>@@ -64,7 +61,7 @@
> 
>                         </form>
>                     </div>
>-                    <div class="yui-u">
>+                    <div class="col-sm-6">
>                     <h1>Delete Images</h1>
>                         [% IF ( TABLE ) %]
>                         <form name="delete_images" method="post" action="/cgi-bin/koha/patroncards/image-manage.pl" enctype="multipart/form-data">
>@@ -134,12 +131,16 @@
>                         [% END %]
>                     </div>
>                 </div>
>-            </div>
>-        </div>
>-        <div class="yui-b">
>-          [% INCLUDE 'tools-menu.inc' %]
>-        </div>
>-    </div>
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     <script>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt
>index 6bd66b3..2df180a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt
>@@ -36,14 +36,15 @@
>         <a href="/cgi-bin/koha/patroncards/home.pl">Patron card creator</a> &rsaquo;
>         [% PROCESS translate_card_element element=card_element_title %]
>     </div>
>-    <div id="doc3" class="yui-t2">
>-        <div id="bd">
>-            <div id="yui-main">
>-                <div class="yui-b">
>+
>+    <div class="main container-fluid">
>+        <div class="row">
>+            <div class="col-sm-10 col-sm-push-2">
>+                <main>
>+
>                     [% INCLUDE 'patroncards-toolbar.inc' %]
>                     [% INCLUDE 'patroncards-errors.inc' %]
>-                    <div class="yui-gc">
>-                        <div class="yui-u first" id="manage-patroncards-layouts">
>+                        <div id="manage-patroncards-layouts">
>                             <div class="hint">Current library: [% LoginBranchname | html %]</div>
>                             [% IF ( table_loop ) %]
>                             <form name="layouts" action="/cgi-bin/koha/manage.pl?card_element=[% card_element | html %]">
>@@ -121,13 +122,16 @@
>                             <p>Use the toolbar above to create a new [% PROCESS translate_card_element element=card_element %].</p></div>
>                             [% END %]
>                         </div>
>-                    </div>
>-                </div>
>-            </div>
>-            <div class="yui-b">
>-              [% INCLUDE 'tools-menu.inc' %]
>-            </div>
>-        </div>
>+
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'tools-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'greybox.inc' %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 21438: 79527 | 82678 | 83282 | 83606