From a7f7fba378050b42a09cba527283f01d6770d726 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

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>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../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> <!-- /div property=address -->
     [% 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> <!-- /#library_info -->
-- 
2.39.5