From 2fb195eedfecbc881eb04261625ffd267ae0c3d5 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 18 Aug 2023 11:43:15 +0000 Subject: [PATCH] Bug 34386: Fix inconsistencies in Cities and tows page titles, breadcrumbs, and header This patch updates the cities administration page so that page title, breadcrumb navigation, and headings are consistent with each other and with other parts of Koha. To test, apply the patch and go to Administration -> Cities and towns. Check each variation of the page to confirm that page title, breadcrumb navigation, and heading are correct. - Main page (listing cities) - New city - Edit city - Delete city Signed-off-by: Lucas Gass Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 8460158f6bf9596e933c496d86ce0591e1cfe2bc) Signed-off-by: Fridolin Somers --- .../prog/en/modules/admin/cities.tt | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt index 90e4e528a0..1d2ebf202b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt @@ -7,13 +7,13 @@ [% FILTER collapse %] [% IF op =='add_form' %] [% IF city.cityid %] - [% t("Modify city") | html %] + [% tx("Modify city '{city}'", { city = city.city_name }) | html %] [% ELSE %] [% t("New city") | html %] [% END %] › [% ELSE %] [% IF op == 'delete_confirm' %] - [% t("Confirm deletion of city") | html %] › + [% tx("Confirm deletion of city '{city}'", { city = city.city_name }) | html %] › [% END %] [% END %] [% t("Cities") | html %] › [% t("Administration") | html %] › [% t("Koha") | html %] @@ -38,7 +38,7 @@ [% END %] [% WRAPPER breadcrumb_item bc_active = 1 %] [% IF city.cityid %] - <span>Modify city</span> + [% tx("Modify city '{city}'", { city = city.city_name }) | html %] [% ELSE %] <span>New city</span> [% END %] @@ -49,7 +49,7 @@ <a href="/cgi-bin/koha/admin/cities.pl">Cities</a> [% END %] [% WRAPPER breadcrumb_item bc_active = 1 %] - <span>Confirm deletion of city</span> + [% tx("Confirm deletion of city '{city}'", { city = city.city_name }) | html %] [% END %] [% ELSE %] @@ -91,7 +91,9 @@ [% IF op == 'add_form' %] [% IF city %] - <h1>Modify a city</h1> + <h1> + [% tx("Modify city '{city}'", { city = city.city_name }) | html %] + </h1> [% ELSE %] <h1>New city</h1> [% END %] @@ -133,7 +135,9 @@ [% IF op == 'delete_confirm' %] <div class="dialog alert"> - <h1>Delete city "[% city.city_name | html %]?"</h1> + <h1> + [% tx("Confirm deletion of city '{city}'", { city = city.city_name }) | html %] + </h1> <table> <tr><th>City id</th> <td>[% city.cityid | html %]</td> -- 2.39.5