From f636a041a62331a09cb1a2bd39c86fd3d8a04c13 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 MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 Signed-off-by: Émily-Rose Francoeur Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../prog/en/modules/admin/oai_set_mappings.tt | 22 +++++++++---------- .../prog/en/modules/admin/oai_sets.tt | 18 ++++++++------- 2 files changed, 21 insertions(+), 19 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..cd7c46481a 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 %] @@ -23,10 +23,10 @@ <a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a> [% END %] [% WRAPPER breadcrumb_item %] - <a href="/cgi-bin/koha/admin/oai_sets.pl">OAI sets configuration</a> + <a href="/cgi-bin/koha/admin/oai_sets.pl">OAI sets</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.39.5