Attachment 125884 Details for Bug 29162 – Bug 29162: Change template structure on OPAC library page so that a single library can easily be hidden

[patch] Bug 29162: Change template structure on OPAC library page so that a single library can easily be hidden

Bug-29162-Change-template-structure-on-OPAC-librar.patch (text/plain), 6.94 KB, created by Marcel de Rooy on 2021-10-07 13:59:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2021-10-07 13:59:04 UTC

Size: 6.94 KB

patch

obsolete

>From 8b560bb361a2598cf072e73672932943f947eaba Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 4 Oct 2021 19:18:04 +0000
>Subject: [PATCH] Bug 29162: Change template structure on OPAC library page so
> that a single library can easily be hidden
>Content-Type: text/plain; charset=utf-8
>
>This patch adds markup to the OPAC library page so that CSS or JS can
>more easily target elements of the page:
>
>- Each library is wrapped in a div, e.g. <div id="section_CPL">
>- Classes are added to the paragraphs containing phone, fax, URL, and
>  library description.
>- An ID has been added to the menu of libraries in the sidebar so that
>  they can be targetted individually.
>
>To test, apply the patch and go to Administration -> System prefernces.
>
>- Add some testing CSS to the OPACUserCSS system preference, if
>  necessary replacing "CPL" with a branchcode in your system:
>
>  div#section_CPL,
>  li#menu_CPL {
>     font-size: 80%;
>  }
>
>- In the OPAC, view the "Libraries" page.
>- In the view of all libraries you should see your CSS reflected in the
>  section for that library.
>- In the individual library view you should see the menu item for that
>  library affected by your custom CSS.
>
>Signed-off-by: David Nind <david@davidnind.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> .../bootstrap/en/modules/opac-library.tt      | 50 ++++++++++++-------
> 1 file changed, 32 insertions(+), 18 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 2c3add4825..8fc5f3d5f7 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-library.tt
>@@ -16,7 +16,16 @@
>     [% END %] catalog
> </title>
> [% INCLUDE 'doc-head-close.inc' %]
>-[% BLOCK cssinclude %][% END %]
>+[% BLOCK cssinclude %]
>+    <style>
>+        .section_library {
>+            margin-top: 1rem;
>+            margin-bottom: 1rem;
>+            border: 0;
>+            border-bottom: 1px solid rgba(0,0,0,.1);
>+        }
>+    </style>
>+[% END %]
> </head>
> 
> [% INCLUDE 'bodytag.inc' bodyid='opac-library' bodyclass='scrollto' %]
>@@ -33,13 +42,13 @@
>         [% PROCESS 'display-library-address' %]
>     </div> 
>     [% IF ( library.branchphone ) %]
>-        <p>Phone: <a href="tel:[% library.branchphone | url %]" property="telephone">[% library.branchphone | html %]</a></p>
>+        <p class="branchphone">Phone: <a href="tel:[% library.branchphone | url %]" property="telephone">[% library.branchphone | html %]</a></p>
>     [% END %]
>     [% IF ( library.branchfax ) %]
>-        <p>Fax: <span property="faxNumber">[% library.branchfax | html %]</span></p>
>+        <p class="branchfax">Fax: <span property="faxNumber">[% library.branchfax | html %]</span></p>
>     [% END %]
>     [% IF ( library.branchurl ) %]
>-        <p><a href="[% library.branchurl | url %]" property="url">[% library.branchurl | html %]</a></p>
>+        <p class="branchurl"><a href="[% library.branchurl | url %]" property="url">[% library.branchurl | html %]</a></p>
>     [% END %]
> [% END %]
> 
>@@ -86,8 +95,10 @@
>                             <div class="col-lg-8">
>                                 [% PROCESS library_info %]
>                                 [% IF ( library.opac_info ) %]
>-                                    <hr />
>-                                    [% PROCESS library_description %]
>+                                    <div class="library_description">
>+                                        <hr />
>+                                        [% PROCESS library_description %]
>+                                    </div>
>                                 [% END %]
>                             </div>
>                             <div class="col-lg-4">
>@@ -96,13 +107,13 @@
>                                         <ul class="fa-ul">
>                                             [% FOREACH library IN libraries %]
>                                                 [% IF ( branchcode == library.branchcode ) %]
>-                                                    <li class="current">
>+                                                    <li id="menu_[% library.branchcode| html %]" class="current">
>                                                         <a href="/cgi-bin/koha/opac-library.pl?branchcode=[% library.branchcode | url %]">
>                                                             <i class="fa fa-li fa-map-pin" aria-hidden="true"></i> [% library.branchname | html %]
>                                                         </a>
>                                                     </li>
>                                                 [% ELSE %]
>-                                                    <li>
>+                                                    <li id="menu_[% library.branchcode| html %]">
>                                                         <a href="/cgi-bin/koha/opac-library.pl?branchcode=[% library.branchcode | url %]">
>                                                             <i class="fa fa-li" aria-hidden="true"></i> [% library.branchname | html %]
>                                                         </a>
>@@ -121,17 +132,20 @@
>                         <h1>[% IF ( singleBranchMode ) %]Library[% ELSE %]Libraries[% END %]</h1>
> 
>                         [% FOREACH library IN libraries %]
>-                            <h2 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 %]
>-                            </h2>
>-                            [% PROCESS library_info %]
>-                            <hr>
>+                            <div class="section_library" id="section_[% library.branchcode | html %]">
>+                                <h2 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 %]
>+                                </h2>
>+                                [% PROCESS library_info %]
>+                            </div>
>                             [% IF ( libraries.count == 1 ) %]
>-                                [% PROCESS library_description %]
>+                                <div class="library_description">
>+                                    [% PROCESS library_description %]
>+                                </div>
>                             [% END %]
>                         [% END %]
>                     </div> 
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 29162: 125713 | 125760 | 125884