Bug 27631: acqui folder

Changed each of the pages in the acqui folder 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 acqui folder 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 21:50:20 +00:00 committed by Fridolin Somers
parent b034e0e41e
commit aba4ce142b
19 changed files with 22 additions and 18 deletions

View file

@ -185,7 +185,7 @@
[% END %] [% END %]
[% IF ( NO_BOOKSELLER ) %] [% IF ( NO_BOOKSELLER ) %]
<h2>Vendor not found</h2> <h1>Vendor not found</h1>
[% ELSE %] [% ELSE %]
[% IF ( delete_confirmed ) %] [% IF ( delete_confirmed ) %]
<div class="dialog message"> <div class="dialog message">

View file

@ -39,6 +39,7 @@
<main> <main>
[% INCLUDE 'acquisitions-toolbar.inc' %] [% INCLUDE 'acquisitions-toolbar.inc' %]
<h1>Search for vendor</h1>
[% UNLESS ( count == 1 ) %] [% UNLESS ( count == 1 ) %]
[% UNLESS count %] [% UNLESS count %]
<h1>Your search returned no results.</h1> <h1>Your search returned no results.</h1>

View file

@ -32,7 +32,7 @@
[% UNLESS ( confirmcancel ) %] [% UNLESS ( confirmcancel ) %]
<form method="post" action=""> <form method="post" action="">
<div class="dialog alert"> <div class="dialog alert">
<h3>Are you sure you want to cancel this order ([% ordernumber | html %])</h3> <h1>Are you sure you want to cancel this order ([% ordernumber | html %])</h1>
<p> <p>
[% IF (del_biblio) %] [% IF (del_biblio) %]
Bibliographic record will be deleted too. Bibliographic record will be deleted too.

View file

@ -50,10 +50,9 @@ Acquisitions &rsaquo; Koha
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>Duplicate existing orders</h1>
[% INCLUDE 'blocking_errors.inc' %] [% INCLUDE 'blocking_errors.inc' %]
<h2>Duplicate existing orders</h2>
[% IF op == 'search' || op == 'select' %] [% IF op == 'search' || op == 'select' %]
<form action="/cgi-bin/koha/acqui/duplicate_orders.pl" method="post"> <form action="/cgi-bin/koha/acqui/duplicate_orders.pl" method="post">
<fieldset class="rows"> <fieldset class="rows">

View file

@ -26,7 +26,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h2>Select the library account submitting the EDI order</h2> <h1>Select the library account submitting the EDI order</h1>
<br /> <br />
<form action="/cgi-bin/koha/acqui/basket.pl" method="get"> <form action="/cgi-bin/koha/acqui/basket.pl" method="get">
<p>Select ordering library account: </p> <p>Select ordering library account: </p>

View file

@ -25,6 +25,7 @@
<div class="row"> <div class="row">
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>EDIFACT message display</h1>
<div id="edimsg"> <div id="edimsg">
[% IF no_message %] [% IF no_message %]
<div class"dialog message">The requested message cannot be displayed</div> <div class"dialog message">The requested message cannot be displayed</div>

View file

@ -45,6 +45,9 @@
<div class="row"> <div class="row">
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>Order search</h1>
[% IF !order_loop && search_done %] [% IF !order_loop && search_done %]
<div class="dialog message"> <div class="dialog message">
Your search returned no results. Your search returned no results.

View file

@ -37,7 +37,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h2>Files for invoice: [% invoicenumber | html %]</h2> <h1>Files for invoice: [% invoicenumber | html %]</h1>
<p><strong>Vendor: </strong><a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% suppliername | html %]</a></p> <p><strong>Vendor: </strong><a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% suppliername | html %]</a></p>
<br /> <br />
[% IF errors %] [% IF errors %]

View file

@ -40,6 +40,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>Invoice: [% invoicenumber | html %]</h1>
[% INCLUDE 'blocking_errors.inc' %] [% INCLUDE 'blocking_errors.inc' %]
[% IF ( modified ) %] [% IF ( modified ) %]
@ -47,7 +48,6 @@
<p>Invoice has been modified</p> <p>Invoice has been modified</p>
</div> </div>
[% END %] [% END %]
<h1>Invoice: [% invoicenumber | html %]</h1>
<p>Vendor: <a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% suppliername | html %]</a></p> <p>Vendor: <a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% suppliername | html %]</a></p>
<form action="/cgi-bin/koha/acqui/invoice.pl" method="post" class="validated"> <form action="/cgi-bin/koha/acqui/invoice.pl" method="post" class="validated">

View file

@ -53,13 +53,13 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h2> <h1>
[% IF ( ordernumber ) %] [% IF ( ordernumber ) %]
Modify order line Modify order line
[% ELSE %] [% ELSE %]
New order New order
[% END %] [% END %]
</h2> </h1>
<div class="dialog alert order_error" style="display:none"></div> <div class="dialog alert order_error" style="display:none"></div>

View file

@ -33,7 +33,7 @@
<main> <main>
<div class="dialog alert"> <div class="dialog alert">
<h4>Duplicate warning</h4> <h1>Duplicate warning</h1>
<p>You selected a record from an external source that matches an existing record in your catalog: <a target="_blank" title="Open in new window" href="[% PROCESS biblio_a_href biblionumber => biblionumber %]"><i class="fa fa-window-restore"></i> [% duplicatetitle | html %]</a></p> <p>You selected a record from an external source that matches an existing record in your catalog: <a target="_blank" title="Open in new window" href="[% PROCESS biblio_a_href biblionumber => biblionumber %]"><i class="fa fa-window-restore"></i> [% duplicatetitle | html %]</a></p>
</div> </div>

View file

@ -41,7 +41,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h2>Serials subscriptions</h2> <h1>Serials subscriptions</h1>
[% IF (done_searched) %] [% IF (done_searched) %]
<label for="show_only_renewed"> <label for="show_only_renewed">
<input type="checkbox" style="vertical-align: middle;" id="show_only_renewed" /> <input type="checkbox" style="vertical-align: middle;" id="show_only_renewed" />

View file

@ -36,8 +36,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>Fund: [% fund_code | html %]</h1> <h1>Ordered - Fund: [% fund_code | html %]</h1>
<h2>Ordered</h2>
<table id="spent"> <table id="spent">
<thead> <thead>

View file

@ -38,13 +38,13 @@
<div class="col-md-10 col-md-offset-1 col-lg-8 col-lg-offset-2"> <div class="col-md-10 col-md-offset-1 col-lg-8 col-lg-offset-2">
[% END %] [% END %]
<main> <main>
<h1>Receive shipment from vendor <a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% name | html %]</a></h1>
[% IF ( error_failed_to_create_invoice ) %] [% IF ( error_failed_to_create_invoice ) %]
<div id="error" class="dialog alert"> <div id="error" class="dialog alert">
<p>An error has occurred. Invoice cannot be created.</p> <p>An error has occurred. Invoice cannot be created.</p>
</div> </div>
[% END %] [% END %]
<h1>Receive shipment from vendor <a href="/cgi-bin/koha/acqui/supplier.pl?booksellerid=[% booksellerid | uri %]">[% name | html %]</a></h1>
[% IF duplicate_invoices %] [% IF duplicate_invoices %]
<div id="parcels_duplicate_invoice" class="dialog alert"> <div id="parcels_duplicate_invoice" class="dialog alert">

View file

@ -4,6 +4,7 @@
</head> </head>
<body id="acq_showorder" class="acq"> <body id="acq_showorder" class="acq">
<div id="main"> <div id="main">
<h1>Order details</h1>
[% IF order %] [% IF order %]
<fieldset class="rows"> <fieldset class="rows">
<ol> <ol>

View file

@ -36,8 +36,7 @@
<div class="col-sm-10 col-sm-push-2"> <div class="col-sm-10 col-sm-push-2">
<main> <main>
<h1>Fund: [% fund_code | html %]</h1> <h1>Spent - Fund: [% fund_code | html %]</h1>
<h2>Spent</h2>
<table id="spent"> <table id="spent">
<thead> <thead>

View file

@ -8,6 +8,7 @@
<body id="acq_transferorder" class="acq"> <body id="acq_transferorder" class="acq">
<div class="container-fluid"> <div class="container-fluid">
<h1>Transfer order</h1>
[% IF ( bookselleridfrom ) %] [% IF ( bookselleridfrom ) %]
<h3>Search for a vendor to transfer to</h3> <h3>Search for a vendor to transfer to</h3>
<div class="transferinfo"> <div class="transferinfo">

View file

@ -79,7 +79,7 @@
[% END %] [% END %]
[% END %] [% END %]
<h2>Orders with uncertain prices</h2> <h1>Orders with uncertain prices</h1>
<form action="[% scriptname | html %]" method="post"> <form action="[% scriptname | html %]" method="post">
<input type="hidden" name="booksellerid" value="[% booksellerid | html %]" /> <input type="hidden" name="booksellerid" value="[% booksellerid | html %]" />
<label for="owner">Orders from:</label> <label for="owner">Orders from:</label>

View file

@ -55,7 +55,7 @@ tr.selected { background-color : #FFFFCC; } tr.selected td { background-color :
<form method="post" action="z3950_search.pl" name="f" class="checkboxed"> <form method="post" action="z3950_search.pl" name="f" class="checkboxed">
<div class="row"> <div class="row">
<div class="col-sm-6"> <div class="col-sm-6">
<h2>Order from external source</h2> <h1>Order from external source</h1>
<input type="hidden" name="op" id="op" value="do_search" /> <input type="hidden" name="op" id="op" value="do_search" />
<fieldset class="rows"> <fieldset class="rows">
<ol> <ol>