Bug 27631: labels and members folders

Changed each of the pages in the labels and members 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 labels and members 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-19 05:51:05 +00:00 committed by Fridolin Somers
parent 2ed6902783
commit 26e38d4990
24 changed files with 66 additions and 22 deletions

View file

@ -30,6 +30,7 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'labels-toolbar.inc' %]
<h1>Label creator</h1>
</main>
</div> <!-- /.col-sm-10.col-sm-push-2 -->

View file

@ -47,6 +47,7 @@
<main>
[% INCLUDE 'labels-toolbar.inc' %]
<h1>Manage labels</h1>
[% IF ( error ) %]
<div class="dialog alert">

View file

@ -8,6 +8,7 @@
<body id="labels_label-print" class="tools labels">
<div class="main container-fluid">
<h1>Label printing/exporting</h1>
[% IF ( batches ) %]
<form>
<h3>Click on the following links to download the exported batch(es).</h3>

View file

@ -30,7 +30,7 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h2>Quick spine label creator</h2>
<h1>Quick spine label creator</h1>
<form action="/cgi-bin/koha/labels/spinelabel-print.pl" method="post" id="spinelabelprint">
<fieldset class="brief">
<ol>

View file

@ -60,6 +60,14 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>
[% IF type == 'credit' %]
Details of payment
[% ELSIF type == 'debit' %]
Details of fee
[% END %]
</h1>
[% INCLUDE 'members-toolbar.inc' borrowernumber=patron.borrowernumber %]
[% IF accountline %]

View file

@ -44,6 +44,7 @@
<main>
[% INCLUDE 'members-toolbar.inc' %]
<h1>Account for [% INCLUDE 'patron-title.inc' %]</h1>
<form action="/cgi-bin/koha/members/boraccount.pl" method="get"><input type="hidden" name="borrowernumber" id="borrowernumber" value="[% patron.borrowernumber | html %]" /></form>
<!-- The manual invoice and credit buttons -->

View file

@ -35,7 +35,7 @@
[% INCLUDE 'members-toolbar.inc' %]
[% IF ItemsOnIssues || debits || is_guarantor %]
<div class="dialog alert">
<h3>Cannot delete patron</h3>
<h1>Cannot delete patron</h1>
<ul>
[% IF ( ItemsOnIssues ) %]
<li>Patron has [% ItemsOnIssues | html %] item(s) checked out.</li>
@ -64,7 +64,8 @@
[% END %]
</ul>
[% END %]
<h3>Are you sure you want to delete the patron [% patron.firstname | html %] [% patron.surname | html %]? This cannot be undone.</h3>
<h1>Are you sure you want to delete the patron [% patron.firstname | html %] [% patron.surname | html %]?</h1>
<p>This cannot be undone.</p>
<form action="/cgi-bin/koha/members/deletemem.pl">
<input type="hidden" name="csrf_token" value="[% csrf_token | html %]" />
<input type="hidden" name="member" value="[% patron.borrowernumber | html %]"/>

View file

@ -32,7 +32,7 @@
<div class="col-md-8 col-md-offset-2">
[% IF pending_discharges %]
<h2>Pending discharge requests</h2>
<h1>Pending discharge requests</h1>
<div id="pending_updates">
<table>
<thead>
@ -57,7 +57,7 @@
</div>
[% ELSE %]
<div class="dialog message">
<p>There are no pending discharge requests.</p>
<h1>There are no pending discharge requests.</h1>
</div>
[% END %]

View file

@ -34,10 +34,10 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
<h1>Files</h1>
[% INCLUDE 'members-toolbar.inc' %]
[% IF errors %]
<div class="dialog alert">
[% IF errors.empty_upload %]The file you are attempting to upload has no contents.[% END %]

View file

@ -79,10 +79,10 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
<!-- Title -->
<h1>Housebound details for [% INCLUDE 'patron-title.inc' %]</h1>
<!-- Title -->
<h1>Housebound details for [% INCLUDE 'patron-title.inc' %]</h1>
[% INCLUDE 'members-toolbar.inc' %]
[% FOR m IN messages %]
<div class="dialog [% m.type | html %]">

View file

@ -37,8 +37,8 @@
<div class="row">
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
<h1>ILL requests history</h1>
[% INCLUDE 'members-toolbar.inc' %]
[% INCLUDE 'ill-list-table.inc' %]
</main>
</div> <!-- .col-sm-10.col-sm-push-2 -->

View file

@ -36,6 +36,8 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Manual credit</h1>
[% INCLUDE 'members-toolbar.inc' %]
<!-- The manual invoice and credit buttons -->

View file

@ -37,6 +37,8 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Manual invoice</h1>
[% INCLUDE 'members-toolbar.inc' %]
<!-- The manual invoice and credit buttons -->

View file

@ -82,7 +82,7 @@
[% END %]
<fieldset class="brief"><legend>Change username and/or password for [% patron.firstname | html %] [% patron.surname | html %]</legend>
<fieldset class="brief"><legend><h1>Change username and/or password for [% patron.firstname | html %] [% patron.surname | html %]</h1></legend>
<ol>
<li><label for="newuserid">New username:</label>
<input type="hidden" name="member" value="[% patron.borrowernumber | html %]" /><input type="text" id="newuserid" name="newuserid" size="20" value="[% patron.userid | html %]" /></li>

View file

@ -108,8 +108,8 @@
<div class="row">
<div class="col-md-8 col-md-offset-2">
<h1>Update patron records</h1>
[% IF PendingModifications %]
<h2>Update patron records</h2>
<form method="post" action="members-update-do.pl">
<div id="pending_updates">
[% FOREACH pm IN PendingModifications %]

View file

@ -96,7 +96,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">
<h3>Merge patron records</h3>
<h1>Merge patron records</h1>
[% IF action == 'show' %]
[% IF patrons.count > 1 %]

View file

@ -12,7 +12,7 @@
<div class="container-fluid">
<div class="row">
<div class="col-xs-12">
<h3>[% INCLUDE 'patron-title.inc' %]</h3>
<h1>[% INCLUDE 'patron-title.inc' %]</h1>
</div>
</div>
<div class="row">

View file

@ -102,6 +102,12 @@
[% INCLUDE 'patron_messages.inc' %]
</div>
<h1>
[% UNLESS ( I ) %]
[% patron.title | html %] [% patron.firstname | html %]
[% END %]
[% patron.surname | html %] ([% patron.cardnumber | html %])
</h1>
<div class="col-sm-6">
<div id="patron-information" class="patroninfo-section">

View file

@ -38,6 +38,8 @@
<div class="col-sm-10 col-sm-push-2">
<main>
<h1>Make a payment</h1>
[% INCLUDE 'members-toolbar.inc' borrowernumber=patron.borrowernumber %]
<!-- The manual invoice and credit buttons -->

View file

@ -73,8 +73,25 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' borrowernumber=patron.borrowernumber %]
<h1>
[% IF ( pay_individual ) %]
Pay an individual fine
[% ELSIF ( writeoff_individual ) %]
Write off an individual fine
[% ELSE %]
[% IF ( selected_accts ) %]
[% IF type == 'WRITEOFF' %]
Write off an amount toward selected fines
[% ELSE %]
Pay an amount toward selected fines
[% END %]
[% ELSE %]
Pay an amount toward all fines
[% END %]
[% END %]
</h1>
[% INCLUDE 'members-toolbar.inc' borrowernumber=patron.borrowernumber %]
<!-- The manual invoice and credit buttons -->
<div class="statictabs">

View file

@ -35,8 +35,8 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
<h1>Purchase suggestions</h1>
[% INCLUDE 'members-toolbar.inc' %]
<div id="toolbar" class="btn-toolbar">
<a class="btn btn-default" id="newsuggestion" href="/cgi-bin/koha/suggestion/suggestion.pl?op=add&amp;suggestedby=[% patron.borrowernumber | html %]&amp;redirect=purchase_suggestions&amp;borrowernumber=[% patron.borrowernumber | html %]"><i class="fa fa-plus"></i> New purchase suggestion</a>

View file

@ -36,9 +36,10 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
<h1>Circulation history</h1>
[% INCLUDE 'members-toolbar.inc' %]
[% UNLESS Koha.Preference('intranetreadinghistory') %]
<div class="dialog alert">Staff members are not allowed to access patron's checkout history</div>
[% ELSIF is_anonymous %]

View file

@ -34,8 +34,9 @@
<div class="col-sm-10 col-sm-push-2">
<main>
[% INCLUDE 'members-toolbar.inc' %]
[% INCLUDE 'members-toolbar.inc' %]
<h1>Subscription routing lists for [% INCLUDE 'patron-title.inc' %]</h1>
<!-- Search Bar -->
<p class="tip">Search subscriptions:</p>
<form action="/cgi-bin/koha/serials/serials-search.pl" method="get">[% IF ( routing ) %]<input type="hidden" name="routing" value="[% routing | html %]" />[% END %]<input type="hidden" name="searched" value="1" /> <label for="ISSN_filter">ISSN:</label> <input type="text" size="10" maxlength="11" name="ISSN_filter" id="ISSN_filter" value="[% ISSN_filter | html %]" /> <label for="title_filter">Title:</label> <input type="text" size="20" maxlength="40" name="title_filter" id="title_filter" value="[% title_filter | html %]" /><input type="submit" value="Search" class="submit" />
@ -43,7 +44,7 @@
<!-- Search Bar End -->
[% SET routinglists = patron.get_routing_lists %]
<h1>
<h2>
[% UNLESS ( routinglists ) %]
0 subscription routing lists
[% ELSIF ( routinglists.count == 1 ) %]
@ -51,7 +52,7 @@
[% ELSE %]
[% routinglists.count | html %] subscription routing lists
[% END %]
</h1>
</h2>
<div id="subscriptions">

View file

@ -11,7 +11,7 @@
[% IF ( MULTI ) %]
<h3> Choose adult category </h3>
<h1> Choose adult category </h1>
[% IF patron_categories %]
<form method="post" action="update-child.pl">