From a16d72bff2cfca4c7c87671386ac71758184a148 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 25 Aug 2023 16:35:18 +0000 Subject: [PATCH] Bug 34400: Inconsistencies in OAI sets page titles, breadcrumbs, and header This patch fixes a couple of inconsistencies in the style of the OIA sets administration screen, making sure the page title, breadcrumb navigation, and page headers are consistent with each other. The patch also corrects the structure of the form on the OAI set mappings page so that the submit button is outside the "page-section" area. To test, apply the patch and go to Administration -> OAI sets. Test each variation of the page: - Main page - New OAI set - Edit OAI set - OAI set mappings --- .../prog/en/modules/admin/oai_set_mappings.tt | 20 +++++++++---------- .../prog/en/modules/admin/oai_sets.tt | 18 +++++++++-------- 2 files changed, 20 insertions(+), 18 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_set_mappings.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_set_mappings.tt index fe64fd4b8c..ce6e837231 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_set_mappings.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_set_mappings.tt @@ -4,7 +4,7 @@ [% SET footerjs = 1 %] [% INCLUDE 'doc-head-open.inc' %] [% FILTER collapse %] - [% t("OAI set mappings") | html %] › + [% tx("Mappings for set '{oai_set}'", { oai_set = setSpec }) | html %] › [% t("OAI sets") | html %] › [% t("Administration") | html %] › [% t("Koha") | html %] @@ -26,7 +26,7 @@ <a href="/cgi-bin/koha/admin/oai_sets.pl">OAI sets configuration</a> [% END %] [% WRAPPER breadcrumb_item bc_active= 1 %] - <span>OAI set mappings</span> + [% tx("Mappings for set '{oai_set}'", { oai_set = setSpec }) | html %] [% END %] [% END #/ WRAPPER breadcrumbs %] [% END #/ WRAPPER sub-header.inc %] @@ -42,13 +42,13 @@ <p><a href="/cgi-bin/koha/admin/oai_sets.pl">Return to sets management</a></p> </div> [% END %] - <h1>Mappings for set '[% setName | html %]' ([% setSpec | html %])</h1> + <h1>[% tx("Mappings for set '{oai_set}'", { oai_set = setSpec }) | html %]</h1> [% UNLESS ( mappings ) %] <div class="dialog alert"><p><strong>Warning:</strong> No mappings have been defined for this set</p></div> [% END %] - <div class="page-section"> - <form action="/cgi-bin/koha/admin/oai_set_mappings.pl" method="post" id="mappingform"> + <form action="/cgi-bin/koha/admin/oai_set_mappings.pl" method="post" id="mappingform"> + <div class="page-section"> <table id="mappings"> <thead> <tr> @@ -132,11 +132,11 @@ </table> <input type="hidden" name="id" value="[% id | html %]" /> <input type="hidden" name="op" value="save" /> - <fieldset class="action"> - <input type="submit" class="btn btn-primary" value="Save" /> <a class="cancel" href="/cgi-bin/koha/admin/oai_sets.pl">Cancel</a> - </fieldset> - </form> - </div> <!-- /.page-section --> + </div> <!-- /.page-section --> + <fieldset class="action"> + <input type="submit" class="btn btn-primary" value="Save" /> <a class="cancel" href="/cgi-bin/koha/admin/oai_sets.pl">Cancel</a> + </fieldset> + </form> </main> </div> <!-- /.col-sm-10.col-sm-push-2 --> diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_sets.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_sets.tt index 928cff31ec..8a82e5f44d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_sets.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/oai_sets.tt @@ -5,9 +5,11 @@ [% INCLUDE 'doc-head-open.inc' %] <title>[% FILTER collapse %] [% IF ( op_new ) %] - [% t("Add a new OAI set") | html %] › + [% t("New OAI set") | html %] › + [% ELSIF ( op_mod ) %] + [% tx("Modify OAI set '{oai_set}'", { oai_set = spec }) | html %] › [% END %] - [% t("OAI sets configuration") | html %] › + [% t("OAI sets") | html %] › [% t("Administration") | html %] › [% t("Koha") | html %] [% END %] @@ -27,23 +29,23 @@ [% IF ( op_new || op_mod ) %] [% WRAPPER breadcrumb_item %] - OAI sets configuration + OAI sets [% END %] [% END %] [% IF ( op_new ) %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Add a new OAI set + New OAI set [% END %] [% ELSIF ( op_mod ) %] [% WRAPPER breadcrumb_item bc_active= 1 %] - Edit OAI set '[% spec | html %]' + [% tx("Modify OAI set '{oai_set}'", { oai_set = spec }) | html %] [% END %] [% ELSE %] [% WRAPPER breadcrumb_item bc_active= 1 %] - OAI sets configuration + OAI sets [% END %] [% END %] [% END #/ WRAPPER breadcrumbs %] @@ -57,10 +59,10 @@ [% IF ( op_new || op_mod ) %]
[% IF ( op_new ) %] -

Add a new OAI set

+

New OAI set

[% ELSIF ( op_mod ) %] -

Edit OAI set '[% spec | html %]'

+

[% tx("Modify OAI set '{oai_set}'", { oai_set = spec }) | html %]

[% END %] -- 2.30.2