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>
This commit is contained in:
Owen Leonard 2021-10-04 19:18:04 +00:00 committed by Jonathan Druart
parent 4ebf1788c8
commit a7f7fba378

View file

@ -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 ) %]
<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,6 +132,7 @@
<h1>[% IF ( singleBranchMode ) %]Library[% ELSE %]Libraries[% END %]</h1>
[% FOREACH library IN libraries %]
<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>
@ -129,9 +141,11 @@
[% END %]
</h2>
[% PROCESS library_info %]
<hr>
</div>
[% IF ( libraries.count == 1 ) %]
<div class="library_description">
[% PROCESS library_description %]
</div>
[% END %]
[% END %]
</div> <!-- /#library_info -->