Attachment 58728 Details for Bug 17859 – Bug 17859 - Move JavaScript to the footer on about and auth pages

[patch] Bug 17859 - Move JavaScript to the footer on about and auth pages

Bug-17859---Move-JavaScript-to-the-footer-on-about.patch (text/plain), 2.90 KB, created by Jonathan Druart on 2017-01-10 09:59:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2017-01-10 09:59:48 UTC

Size: 2.90 KB

patch

obsolete

>From c49a0bf40c7742db805de605078b48be9fd2344c Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 6 Jan 2017 16:12:25 +0000
>Subject: [PATCH] Bug 17859 - Move JavaScript to the footer on about and auth
> pages
>
>This patch modifies the about page and the login page templates so that
>JavaScript is included in the footer instead of the header.
>
>To test, apply the patch and test each page to confirm that
>JavaScript-based interactions are unaffected:
>
>- On the About page tabs and header menu dropdowns should work correctly
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/about.tt | 18 ++++++++++--------
> koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt  | 19 +++++++++++--------
> 2 files changed, 21 insertions(+), 16 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
>index ddba5fd..492aee4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt
>@@ -1,14 +1,7 @@
>+[% SET footerjs = 1 %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; About Koha</title>
> [% INCLUDE 'doc-head-close.inc' %]
>-<script type="text/javascript">
>-//<![CDATA[
>-
>-     $(document).ready(function() {
>-        $('#abouttabs').tabs();
>-     });
>-//]]>
>-</script>
> </head>
> <body id="about_about" class="about">
> [% INCLUDE 'header.inc' %]
>@@ -896,4 +889,13 @@
> 
>     </div>
> </div></div></div>
>+
>+[% MACRO jsinclude BLOCK %]
>+    <script type="text/javascript">
>+         $(document).ready(function() {
>+            $('#abouttabs').tabs();
>+         });
>+    </script>
>+[% END %]
>+
> [% INCLUDE 'intranet-bottom.inc' %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt
>index 8d0e58f..82743dd 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt
>@@ -1,4 +1,5 @@
> [% USE Branches %]
>+[% SET footerjs = 1 %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; 
>     [% IF ( nopermission ) %]Access denied[% END %]
>@@ -103,12 +104,14 @@
> </div>
> </div>
> 
>-<script>
>-$(document).ready( function() {
>-    if ( document.location.hash ) {
>-        $( '#loginform' ).append( '<input name="auth_forwarded_hash" type="hidden" value="' + document.location.hash + '"/>' );
>-    }
>-} );
>-</script>
>-
>+[% MACRO jsinclude BLOCK %]
>+    <script type="text/javascript">
>+        $(document).ready( function() {
>+            if ( document.location.hash ) {
>+                $( '#loginform' ).append( '<input name="auth_forwarded_hash" type="hidden" value="' + document.location.hash + '"/>' );
>+            }
>+        });
>+    </script>
>+[% END %]
>+
> [% INCLUDE 'intranet-bottom.inc' %]
>-- 
>2.9.3

Actions: View | Diff | Splinter Review

Attachments on bug 17859: 58649 | 58650 | 58728