Attachment 77980 Details for Bug 20741 – Bug 20741: Update two-column templates with Bootstrap grid: Administration part 4

[patch] Bug 20741: Update two-column templates with Bootstrap grid: Administration part 4

Bug-20741-Update-two-column-templates-with-Bootstr.patch (text/plain), 10.53 KB, created by Owen Leonard on 2018-08-17 14:01:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2018-08-17 14:01:08 UTC

Size: 10.53 KB

patch

obsolete

>From febe4c9edb3a58bfdac06a0acced7ba54c0b6d29 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 2 May 2018 12:11:44 +0000
>Subject: [PATCH] Bug 20741: Update two-column templates with Bootstrap grid:
> Administration part 4
>
>This patch modifies several administration templates to use the
>Bootstrap grid instead of YUI.
>
>This patch also removes obsolete "text/javascript" attributes from
><script> tags in the modified templates.
>
>To test, apply the patch and view the following pages, confirming that
>they look correct at various browser widths:
>
> - Administration -> Keywords to MARC mapping
> - Administration -> Item circulation alerts
> - Administration -> Item types
>   - View and edit item types
>   - Edit and item type and translate a description
> - Administration -> Koha to MARC mapping
> - Administration -> MARC frameworks -> MARC structure
>   - View and edit tags
>---
> .../prog/en/modules/admin/fieldmapping.tt          | 26 +++++++++++--------
> .../en/modules/admin/item_circulation_alerts.tt    | 28 ++++++++++-----------
> .../prog/en/modules/admin/itemtypes.tt             | 29 +++++++++++-----------
> .../prog/en/modules/admin/koha2marclinks.tt        | 26 +++++++++----------
> .../prog/en/modules/admin/localization.tt          |  2 +-
> .../prog/en/modules/admin/marctagstructure.tt      | 24 ++++++++++--------
> 6 files changed, 71 insertions(+), 64 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
>index 1f6b36d..f036985 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
>@@ -9,9 +9,12 @@
> [% INCLUDE 'header.inc' %]
> [% INCLUDE 'prefs-admin-search.inc' %]
> <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a> &rsaquo; Keyword to MARC mapping</div>
>-<div id="doc3" class="yui-t2">
>-	<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>
>+
>             <h2>Keyword to MARC mapping</h2>
>             [% UNLESS ( fields.count ) %]
>                 <div class="dialog message"><p>There are no mappings for the [% IF framework.frameworktext %]<em>[% framework.frameworktext |html %]</em>[% ELSE %]default[% END %] framework. </p></div>
>@@ -66,17 +69,18 @@
> 									[% END %]
> 								</table>[% END %]
> 
>+            </main>
>+        </div> 
> 
>-		</div>
>-	</div>
>-
>-	<div class="yui-b">
>-		[% INCLUDE 'admin-menu.inc' %]
>-	</div>
>-
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'admin-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> [% MACRO jsinclude BLOCK %]
>     [% Asset.js("js/admin-menu.js") %]
>-    <script type="text/javascript">
>+    <script>
>         $(document).ready(function() {
>             $('#selectframework').find("input:submit").hide();
>             $('#framework').change(function() {
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
>index f47d5f9..51c613a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
>@@ -4,7 +4,7 @@
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; Administration &rsaquo; Item circulation alerts</title>
> [% INCLUDE 'doc-head-close.inc' %]
>-<style type="text/css">
>+<style>
> table.grid thead th {
>   vertical-align: bottom;
> }
>@@ -46,12 +46,10 @@ table.grid td.default {
> [% INCLUDE 'prefs-admin-search.inc' %]
> <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a> &rsaquo; Item circulation alerts </div>
> 
>-<div id="doc3" class="yui-t1">
>-
>-<div id="bd">
>-
>-<div id="yui-main" class="circulation-alert">
>-<div class="yui-b">
>+<div class="main container-fluid">
>+    <div class="row">
>+        <div class="col-sm-10 col-sm-push-2">
>+            <main>
> 
> <div class="yui-g">
> <div class="yui-u first">
>@@ -150,17 +148,19 @@ table.grid td.default {
> </div>
> </div>
> 
>-</div>
>-</div>
>+            </main>
>+        </div> 
> 
>-<div class="yui-b">
>-[% INCLUDE 'admin-menu.inc' %]
>-</div>
>-</div>
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'admin-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% Asset.js("js/admin-menu.js") %]
>-    <script type="text/javascript">
>+    <script>
>         var $branch = "[% branch %]";
>         $(function(){
>             $('#alerttabs').tabs();
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
>index 526ea9c..6bb4c5f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
>@@ -24,7 +24,7 @@ Data deleted
> </title>
> [% INCLUDE 'doc-head-close.inc' %]
> [% Asset.css("css/datatables.css") %]
>-<style type="text/css">
>+<style>
> 	fieldset.rows div.toptabs li { clear:none;margin-right:.5em;padding-bottom:0;width:auto; }
>   fieldset.rows div.toptabs .ui-tabs-nav li.ui-tabs-active {background-color : #F4F8F9; }
> 	fieldset.rows .ui-tabs-panel { margin-right : 10px; margin-left : 10px;margin-bottom:10px;}
>@@ -57,12 +57,10 @@ Data deleted
> Item types administration
> [% END %]</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>
> 
> [% IF op == 'list' %]<div id="toolbar" class="btn-toolbar">
>     <a class="btn btn-default btn-sm" id="newitemtype" href="/cgi-bin/koha/admin/itemtypes.pl?op=add_form"><i class="fa fa-plus"></i> New item type</a>
>@@ -388,18 +386,21 @@ Item types administration
>     [% END %]
> [% END %]
> 
>-</div>
>-</div>
>-<div class="yui-b">
>-[% INCLUDE 'admin-menu.inc' %]
>-</div>
>-</div>
>+            </main>
>+        </div> 
>+
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'admin-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% Asset.js("js/admin-menu.js") %]
>     [% INCLUDE 'greybox.inc' %]
>     [% INCLUDE 'datatables.inc' %]
>-    <script type="text/javascript">
>+    <script>
>          $(document).ready(function() {
>             $('#icons').tabs();
>             $("#table_item_type").dataTable($.extend(true, {}, dataTablesDefaults, {
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
>index a692ad2..2633dd5 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
>@@ -12,11 +12,10 @@
> 
> <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a> &rsaquo; <a href="/cgi-bin/koha/admin/koha2marclinks.pl">Koha to MARC Mapping</a></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>
> 
> <h1>Koha to MARC mapping</h1>
> <br/>
>@@ -58,19 +57,20 @@
> 
> </form>
> 
>-</div>
>-</div>
>-
>-<div class="yui-b">
>-[% INCLUDE 'admin-menu.inc' %]
>-</div>
>+            </main>
>+        </div> 
> 
>-</div>
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'admin-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'datatables.inc' %]
>     [% Asset.js("js/admin-menu.js") %]
>-    <script type="text/javascript">
>+    <script>
>         function AddFld(kohafield) {
>             var fieldstr = prompt( _("Adding a mapping for: %s.").format(kohafield) + "\n" + _("Please enter field tag and subfield code, separated by a comma. (For control fields: add '@' as subfield code.)\nThe change will be applied immediately.") );
>             var temp = fieldstr.split(',');
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt
>index fc62473..2bb6090 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt
>@@ -54,7 +54,7 @@
> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'datatables.inc' %]
>-    <script type="text/javascript">
>+    <script>
> 
>         function show_message( params ) {
>             var type = params.type;
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>index 0d0431f..a50ca24 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>@@ -33,10 +33,10 @@
> [% END %]
> </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>
> 
> [% IF ( else ) %]
> <div id="toolbar" class="btn-toolbar">
>@@ -245,19 +245,21 @@
> 
> [% END %]
> 
>+            </main>
>+        </div> 
> 
>-    </div>
>-  </div>
>-<div class="yui-b">
>-[% INCLUDE 'admin-menu.inc' %]
>-</div>
>-</div>
>+        <div class="col-sm-2 col-sm-pull-10">
>+            <aside>
>+                [% INCLUDE 'admin-menu.inc' %]
>+            </aside>
>+        </div> 
>+     </div> 
> 
> [% MACRO jsinclude BLOCK %]
>     [% INCLUDE 'datatables.inc' %]
>     [% Asset.js("js/admin-menu.js") %]
>     [% Asset.js("lib/jquery/plugins/jquery.cookie.min.js") %]
>-    <script type="text/javascript">
>+    <script>
>         $(document).ready(function() {
>             $("#table_marctagstructure").dataTable($.extend(true, {}, dataTablesDefaults, {
>                 "aoColumnDefs": [
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 20741: 77980 | 78265 | 78308 | 78316