Bug 34389: Fix inconsistencies in debit types titles, breadcrumbs, and header

This patch fixes a couple of inconsistencies in the debit types
administration page, making sure the page title, breadcrumb
navigation, and page headers are consistent with each other.

To test, apply the patch and go to Administration -> Debit types. Test
each variation of the page:

- Main page
- New debit type
- Modify debit type

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Owen Leonard 2023-10-02 15:43:29 +00:00 committed by Tomas Cohen Arazi
parent 0b2cc01ca5
commit 86628b3523
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F

View file

@ -9,7 +9,7 @@
<title>[% FILTER collapse %]
[% IF op =='add_form' %]
[% IF debit_type.code %]
[% t("Modify debit type") | html %]
[% tx("Modify debit type '{debit_type}'", { debit_type = debit_type.code }) | html %]
[% ELSE %]
[% t("New debit type") | html %]
[% END %] &rsaquo;
@ -38,7 +38,7 @@
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
[% IF debit_type.code %]
<span>Modify debit type</span>
[% tx("Modify debit type '{debit_type}'", { debit_type = debit_type.code }) | html %]
[% ELSE %]
<span>New debit type</span>
[% END %]
@ -76,7 +76,7 @@
[% IF op == 'add_form' %]
[% IF debit_type %]
<h1>Modify a debit type</h1>
<h1>[% tx("Modify debit type '{debit_type}'", { debit_type = debit_type.code }) | html %]</h1>
[% ELSE %]
<h1>New debit type</h1>
[% END %]
@ -159,7 +159,7 @@
<a class="btn btn-default" id="newdebittype" href="/cgi-bin/koha/admin/debit_types.pl?op=add_form"><i class="fa fa-plus"></i> New debit type</a>
</div>
<h1>Account debit types</h1>
<h1>Debit types</h1>
[% IF debit_types.count %]
<div class="page-section">
<table id="table_debit_types">