Bug 27631: reports and serials folders

Changed each of the pages in the reports and serials folders to
have one <h1> tag showing that describes the page, rather than the <h1>
describing the logo.

The hierarchy of heading tags may be broken in many pages, but this
will be dealt with in an additional bug.

To test:
1) Go to the Staff Client
2) Apply patch
3) Go to each of the pages in the reports and serials folders and
   check that they have an obvious and descriptive heading
4) Ensure that the heading in the page is <h1>

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Wainui Witika-Park 2021-04-05 09:34:34 +00:00 committed by Fridolin Somers
parent 275875c3b9
commit 44a8e0bd6e
17 changed files with 27 additions and 19 deletions

View file

@ -40,9 +40,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Acquisitions statistics </h1>
[% IF ( do_it ) %]
[% FOREACH mainloo IN mainloop %]
<h1>Acquisitions statistics </h1>
[% IF ( mainloo.loopfilter.size ) %]
<p><strong>Filtered on:</strong></p>
[% FOREACH loopfilte IN mainloo.loopfilter %]

View file

@ -48,9 +48,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Patrons statistics</h1>
[% IF ( do_it ) %]
[% FOREACH mainloo IN mainloop %]
<h1>Patrons statistics</h1>
[% IF ( mainloo.loopfilter.size>0 ) %]
<p>Filtered on:</p>
[% FOREACH loopfilte IN mainloo.loopfilter %]

View file

@ -41,9 +41,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Catalog statistics</h1>
[% IF ( do_it ) %]
[% FOREACH mainloo IN mainloop %]
<h1>Catalog statistics</h1>
[% IF ( mainloo.loopfilter ) %]
<p>Filtered on:</p>
[% FOREACH loopfilte IN mainloo.loopfilter %]

View file

@ -3,6 +3,7 @@
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
</head>
<body id="rep_report_convert" class="rep">
<h1>Convert report</h1>
<div>
[% IF msg == 'no_report' %]
There is no valid report for this id.

View file

@ -172,7 +172,7 @@
[% INCLUDE "reports-toolbar.inc" %]
[% IF ( start ) %]
<h2>Guided reports</h2>
<h1>Guided reports</h1>
<p>Use the guided reports engine to create non standard reports. This feature aims to provide some middle ground between the built in canned reports and writing custom SQL reports.</p>
<h3>Build and run reports</h3>
@ -219,8 +219,8 @@
[% END %]
[% IF ( saved1 ) %]
[% IF ( savedreports ) %]
<h1>Saved reports</h1>
[% IF ( savedreports ) %]
[% IF ( filters.date || filters.author || filters.keyword ) %]
<p>Filtered by:
@ -1193,6 +1193,7 @@
[% END #/IF ( execute ) %]
[% IF ( create ) %]
<h1>Create from SQL</h1>
<form action="/cgi-bin/koha/reports/guided_reports.pl" method="post" id="sql_report_form">
<fieldset class="rows">
<legend>Create report from SQL</legend>
@ -1278,6 +1279,7 @@
[% END # /IF saved_results %]
[% IF ( showsql ) %]
<h1>Saved reports - SQL</h1>
<fieldset class="rows">
<legend>[% reportname | html %]</legend>
<ol>

View file

@ -41,9 +41,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Average checkout period</h1>
[% IF ( do_it ) %]
[% FOREACH mainloo IN mainloop %]
<h1>Average checkout period</h1>
[% IF ( mainloo.loopfilter ) %]
<p>Filtered on:</p>
[% FOREACH loopfilte IN mainloo.loopfilter %]

View file

@ -44,9 +44,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Circulation statistics</h1>
[% IF ( do_it ) %]
[% FOREACH mainloo IN mainloop %]
<h1>Circulation statistics</h1>
[% IF ( mainloo.loopfilter ) %]
<p>Filtered on:</p>
<ul>

View file

@ -74,7 +74,7 @@
</table>
[% END %]
[% ELSE %]
<h3>View a count of items held at your library grouped by item type</h3>
<h1>View a count of items held at your library grouped by item type</h1>
<form method="post" action="/cgi-bin/koha/reports/manager.pl?report_name=itemtypes">
<fieldset class="rows">
<ol>

View file

@ -24,7 +24,7 @@
<div class="main container-fluid">
<div class="row">
<div class="col-md-10 col-md-offset-1 col-lg-8 col-lg-offset-2">
<h1>Reports</h1>
<div class="row">
<div class="col-xs-6">

View file

@ -45,9 +45,9 @@
<main>
<h1>Holds statistics</h1>
[% IF ( do_it ) %]
[% IF ( mainloop ) %]
<h1>Holds statistics</h1>
[% IF ( loopfilter ) %]
<p>Filtered on:</p>
<ul>

View file

@ -32,7 +32,7 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h2>Preview routing list for <em>[% title | html %]</em></h2>
<h1>Preview routing list for <em>[% title | html %]</em></h1>
<form method="post" action="routing-preview.pl">
<input type="hidden" id="subscriptionid" name="subscriptionid" value="[% subscriptionid | html %]" />

View file

@ -45,7 +45,6 @@
<div class="row">
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'blocking_errors.inc' %]
[% IF ( op ) %]
<h1>Create routing list for <em>[% title | html %]</em></h1>
@ -53,6 +52,7 @@
<h1>Edit routing list for <em>[% title | html %]</em></h1>
[% END %]
[% INCLUDE 'blocking_errors.inc' %]
<form method="post" action="routing.pl">
<input type="hidden" name="op" value="save" />
<input type="hidden" name="subscriptionid" value="[% subscriptionid | html %]" />

View file

@ -102,11 +102,12 @@ $(document).ready(function() {
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'modals/serials_multi_receiving.inc' %]
<h1>Serial edition <em>[% bibliotitle | html %]</em>
[% IF location %] ([% AuthorisedValues.GetByCode('LOC', location) | html %])[% END %]
[% IF ( callnumber ) %] callnumber: [% callnumber | html %][% END %]</h1>
[% INCLUDE 'modals/serials_multi_receiving.inc' %]
[% IF internalnotes %]<p>Nonpublic note: [% internalnotes | html | html_line_break %]</p>[% END %]
<form method="post" name="f" class="noEnterSubmit" action="serials-edit.pl" id="serials_edit">

View file

@ -31,6 +31,8 @@
[% INCLUDE 'serials-toolbar.inc' %]
<h1>Serials</h1>
</main>
</div> <!-- /.col-sm-10.col-sm-push-2 -->

View file

@ -231,15 +231,15 @@
[% IF ( mana ) %]
[% IF ( done_searched ) %]
<h2>Mana subscriptions ([% total | html %] found)</h2>
<h1>Mana subscriptions ([% total | html %] found)</h1>
[% ELSE %]
<h2>Mana subscriptions search</h2>
<h1>Mana subscriptions search</h1>
[% END %]
[% ELSE %]
[% IF ( done_searched ) %]
<h2>Serials subscriptions ([% total | html %] found)</h2>
<h1>Serials subscriptions ([% total | html %] found)</h1>
[% ELSE %]
<h2>Serials subscriptions search</h2>
<h1>Serials subscriptions search</h1>
[% END %]
[% END %]

View file

@ -1,5 +1,5 @@
[% USE KohaDates %]
<h2>Prediction pattern</h1>
<h2>Prediction pattern</h2>
[% IF (not_consistent_end_date) %]
<p><em>End date is not consistent with subscription length.</em></p>
[% END %]

View file

@ -13,6 +13,8 @@
<body id="ser_subscription-renew" class="ser">
<div class="container-fluid">
<h1>Serials subscription renew</h1>
[% INCLUDE 'blocking_errors.inc' %]
[% IF op == 'renew' OR op =='multi_renew' %]