Attachment 106127 Details for Bug 22807 – Bug 22807: (follow-up) Add .maincontent class to libraries page

[patch] Bug 22807: (follow-up) Add .maincontent class to libraries page

Bug-22807-follow-up-Add-maincontent-class-to-libra.patch (text/plain), 3.14 KB, created by Katrin Fischer on 2020-06-21 14:46:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2020-06-21 14:46:40 UTC

Size: 3.14 KB

patch

obsolete

>From bff5fce51a7c9d5bf813fa1201e9a1d517e30887 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 10 Jun 2020 18:42:06 +0000
>Subject: [PATCH] Bug 22807: (follow-up) Add .maincontent class to libraries
> page
>
>This patch adds a missing "maincontent" class to the "all libraries"
>view. Previously it was only present on the single library view.
>
>To test, apply the patch and go to the "Libraries" page in the OPAC.
>
> - Hit the "tab" key to highlight the "Skip to main content" link.
> - Hit "Enter."
> - The page should scroll to the top of the list of libraries.
> - Open one of the library detail pages and confirm that the "Skip to
>   main content" link works on that page too.
>
>Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../opac-tmpl/bootstrap/en/modules/opac-library.tt | 29 +++++++++++-----------
> 1 file changed, 15 insertions(+), 14 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt
>index dec3db9b48..dfba205306 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt
>@@ -129,23 +129,24 @@
>                     </div> 
> 
>                 [% ELSE %]
>-                    <h2>[% IF ( singleBranchMode ) %]Library[% ELSE %]Libraries[% END %]</h2>
>+                    <div id="library_info" class="maincontent">
>+                        <h2>[% IF ( singleBranchMode ) %]Library[% ELSE %]Libraries[% END %]</h2>
> 
>-                    [% FOREACH library IN libraries %]
>-                        <h3 property="name">
>-                            [% IF ( libraries.count > 1 ) %]
>-                                <a href="/cgi-bin/koha/opac-library.pl?branchcode=[% library.branchcode | url %]">[% library.branchname | html %]</a>
>-                            [% ELSE %]
>-                                [% library.branchname | html %]
>+                        [% FOREACH library IN libraries %]
>+                            <h3 property="name">
>+                                [% IF ( libraries.count > 1 ) %]
>+                                    <a href="/cgi-bin/koha/opac-library.pl?branchcode=[% library.branchcode | url %]">[% library.branchname | html %]</a>
>+                                [% ELSE %]
>+                                    [% library.branchname | html %]
>+                                [% END %]
>+                            </h3>
>+                            [% PROCESS library_info %]
>+                            <hr>
>+                            [% IF ( libraries.count == 1 ) %]
>+                                [% PROCESS library_description %]
>                             [% END %]
>-                        </h3>
>-                        [% PROCESS library_info %]
>-                        <hr>
>-                        [% IF ( libraries.count == 1 ) %]
>-                            [% PROCESS library_description %]
>                         [% END %]
>-                    [% END %]
>-
>+                    </div>
>                 [% END %]
> 
>         </div> 
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 22807: 105663 | 105664 | 105678 | 105698 | 105699 | 105700 | 105708 | 105709 | 105710 | 105711 | 105712 | 105713 | 105714 | 106124 | 106125 | 106126 | 106127 | 106128