From d3dde3c9cdf8ba11ce67ccb6755559f7f59bf700 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 6 Sep 2023 10:38:41 +0000 Subject: [PATCH] Bug 34411: Inconsistencies in Additional fields page titles, breadcrumbs, and header This patch makes changes to the additional fields administration template in order to make it more consistent with other pages in Koha. To test, apply the patch and go to Administration -> Additional fields. View each variation of the page to confirm that breadcrumb navigation, page title, and page headings are consistent with each other. - Main list - Table view (e.g. aqbasket, aqorders, etc) - New field - Edit field Signed-off-by: Salah Ghedda Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../en/modules/admin/additional-fields.tt | 40 +++++++++++-------- 1 file changed, 24 insertions(+), 16 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt index 7ebfb3d7fa..a569c43fc8 100755 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt @@ -7,13 +7,15 @@ [% SWITCH op %] [% CASE 'add_form' %] [% IF field %] - [% tx("Modify additional field '{field_name}'", { field_name = field.name }) | html %] + [% tx("Modify field '{field_name}'", { field_name = field.name }) | html %] › [% ELSE %] - [% t("Add additional field") | html %] + [% t("New field") | html %] › [% END %] - [% CASE %] - [% t("Manage additional fields") | html %] + [% tx("Table '{table_name}'", { table_name = tablename } ) | html %] + [% CASE 'list' %] + [% tx("Table '{table_name}'", { table_name = tablename } ) | html %] [% END %] › + [% t("Additional fields") | html %] › [% t("Administration") | html %] › [% t("Koha") | html %] [% END %] @@ -38,29 +40,35 @@ [% SWITCH op %] [% CASE 'add_form' %] [% WRAPPER breadcrumb_item %] - Manage additional fields + Additional fields [% END %] [% IF field %] + [% WRAPPER breadcrumb_item %] + [% tx("Table '{table_name}'", { table_name = tablename } ) | html %] + [% END %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Modify field '[% field.name | html %]' + [% tx("Modify field '{field_name}'", { field_name = field.name }) | html %] [% END %] [% ELSE %] + [% WRAPPER breadcrumb_item %] + [% tx("Table '{table_name}'", { table_name = tablename } ) | html %] + [% END %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Add field + [% t("New field" ) | html %] [% END %] [% END %] [% CASE 'list' %] [% WRAPPER breadcrumb_item %] - Manage additional fields + Additional fields [% END %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Modify fields for '[% tablename | html %]' + [% tx("Table '{table_name}'", { table_name = tablename } ) | html %] [% END %] [% CASE %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Manage additional fields + Additional fields [% END %] [% END %] @@ -74,7 +82,7 @@ [% IF op == 'list' %] [% END %] @@ -82,9 +90,9 @@ [% FOR message IN messages %] [% IF message.code == 'insert' %] [% IF message.number > 0 %] -
The field has been inserted
+
The field has been added
[% ELSE %] -
The field could not be inserted. Perhaps the name already exists?
+
The field could not be added. Perhaps the name already exists?
[% END %] [% ELSIF message.code == 'update' %] [% IF message.number > 0 %] @@ -107,7 +115,7 @@

Additional fields

-

Select a table:

+

Select a table:

[% BLOCK table_option %]
  • [% content | $raw %] ([% value | html %])
  • [% END %] @@ -183,9 +191,9 @@ [% END %] [% ELSIF op == 'add_form' %] [% IF field %] -

    Modify field

    +

    [% tx("Modify field '{field_name}' for table '{table_name}'", { field_name = field.name, table_name = tablename }) | html %]

    [% ELSE %] -

    Add field

    +

    [% tx("New field for table '{table_name}'", { table_name = tablename } ) | html %]

    [% END %]
    -- 2.39.5