From 3c5b129f1d33c83b390cda2d6db43a073ada5f65 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 18 Aug 2023 08:39:50 +0000 Subject: [PATCH] Bug 34378: Fix inconsistencies in Libraries page titles, breadcrumbs, and header This patch makes changes to the libraries administration page so that the page title, breadcrumbs, and page headings are more consistent with each other. To test, apply the patch and go to Administration -> Libraries. - Click "New library" - The page title, breadcrumb navigation, and page heading should all read "New library" - Enter some details, save your new library, and then click the "Delete" button corresponding to your new library. - On the deletion conformation page, the page title and breadcrumb navigation should match, "Confirm deletion of library 'X'" - Edit a library. On the edit page, the page title, breadcrumb navigation, and page heading should match: "Modify library 'X'" Signed-off-by: paul Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../prog/en/modules/admin/branches.tt | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt index 89360f79e7..9f4b1e09e2 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt @@ -11,9 +11,9 @@ [% FILTER collapse %] [% IF op == 'add_form' %] [% IF library %] - [% t("Modify library") | html %] + [% tx("Modify library '{library}'", { library = library.branchcode }) | html %] [% ELSE %] - [% tx("New library {library}", { library = library.branchcode }) | html %] + [% t("New library") | html %] [% END %] › [% ELSIF op == 'delete_confirm' %] [% tx("Confirm deletion of library '{library}'", { library = library.branchcode }) | html %] › @@ -49,17 +49,17 @@ [% IF op == 'add_form' %] [% IF library %] [% WRAPPER breadcrumb_item bc_active= 1 %] - <span>Modify library</span> + [% tx("Modify library '{library}'", { library = library.branchcode }) | html %] [% END %] [% ELSE %] [% WRAPPER breadcrumb_item bc_active= 1 %] - <span>New library [% library.branchcode | html %]</span> + <span>New library</span> [% END %] [% END %] [% ELSIF op == 'delete_confirm' %] [% WRAPPER breadcrumb_item bc_active= 1 %] - <span>Confirm deletion of library '[% library.branchcode | html %]'</span> + [% tx("Confirm deletion of library '{library}'", { library = library.branchcode }) | html %] [% END %] [% ELSIF op == 'view' %] @@ -116,7 +116,13 @@ [% END %] [% IF op == 'add_form' %] - <h1>[% IF library %]Modify library[% ELSE %]New library[% END %]</h1> + <h1> + [% IF library %] + [% tx("Modify library '{library}'", { library = library.branchcode }) | html %] + [% ELSE %] + New library + [% END %] + </h1> <form action="/cgi-bin/koha/admin/branches.pl" id="Aform" name="Aform" class="validated" method="post"> <fieldset class="rows"> <input type="hidden" name="op" value="add_validate" /> -- 2.39.5