Bug 34627: CMS pages do not correctly display the opaccredits footer

This patch corrects the page structure of CMS pages in the OPAC so that
the contents of the footer, including opaccredits, OpacKohaUrl, and the
language selection bar, are displayed correctly.

This patch contains indentation changes, so ignore whitespace when
viewing the diff.

To test you should have these settings:

- OpacKohaUrl enabled in system preferences
- Some content in the opaccredits HTML customization region
- Some content in the OpacNav HTML customization region
- At least one translation installed and enabled
- The opaclanguagesdisplay preference enabled
- The OpacLangSelectorMode set to "only footer" or "both top and footer"

- Apply the patch and go to Tools -> Pages and create a page with the
  display location "OPAC."
- Open the OPAC link for your newly-created page and confirm that the
  page looks correct:
  - The "main" region with the white background should contain the
    breadcrumb navigation, the sidebar OpacNav content, and the contents
    of your CMS page.
  - The opaccredits and OpacKohaUrl content should appear below that
    region.
  - The language selection footer should be at the bottom of the page.

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Owen Leonard 2023-08-28 10:59:29 +00:00 committed by Tomas Cohen Arazi
parent c26dff61e7
commit 7d5dc65867
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F

View file

@ -32,40 +32,36 @@
<div class="container-fluid">
<div class="row">
[% IF ( OpacNav || OpacNavBottom ) %]
<div class="col-12 col-lg-2 order-3 order-lg-1">
[% IF ( OpacNav || OpacNavBottom ) %]
<div class="col-lg-2">
<div id="navigation">
[% IF ( OpacNav || OpacNavBottom ) %]
[% INCLUDE 'navigation.inc' %]
[% END %]
[% INCLUDE 'navigation.inc' %]
</div>
</div>
<div class="col-12 col-lg-10 order-md-1 maincontent">
<div class="col-10 order-first order-md-first order-lg-2">
[% ELSE %]
<div class="col order-md-1 maincontent">
<div class="col order-first order-md-first order-lg-2">
[% END %]
<div id="page_[% page.idnew | html %]" class="maincontent">
[% IF page %]
<div class="maincontent">
<h1>[% page.title | html %]</h1>
<h1>[% page.title | html %]</h1>
<div class="page_content">
[% page.content | $raw %]
</div>
<div class="page_content">
[% page.content | $raw %]
</div>
[% ELSE %]
[% ELSE %]
<div class="alert alert-error">
This page does not exist.
</div>
<div class="alert alert-error">
This page does not exist.
</div>
[% END %]
</div> <!-- / .col 6/8 -->
</div> <!-- /.container-fluid -->
</div> <!-- /.row -->
[% END %]
</div> <!-- /.maincontent -->
</div> <!-- / .col -->
</div> <!-- /.row -->
</div> <!-- /.container-fluid -->
</div> <!-- /.main -->
[% INCLUDE 'opac-bottom.inc' %]