From 74ab71b892621cc45df6ae6c54cb9c3ee7a11221 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 21 Aug 2023 15:11:38 +0000 Subject: [PATCH] Bug 34388: Fix inconsistencies in Patron restriction types page headers This patch fixes a couple of inconsistencies in the style of the patron restriction types administration screen, making sure the restriction type name is consistently in single quotes in headings and formatting the deletion confirmation page like other administration interfaces. Note: This patch contains indentation spaces, so ignore whitespace when viewing the diff. To test, apply the patch and go to Administration -> Patron restriction types. - If necessary, add a custom restriction type. - After saving it, edit it. - The page title, breadcrumb, and page heading should match: "Modify restriction type 'X' - From the list of restriction types, click "Delete" on one of your custom restriction types. - On the deletion confirmation page you should see a standard alert-style dialog box and two button options: "Yes, delete" and "No, do not delete." Both should work correctly. --- .../prog/en/modules/admin/restrictions.tt | 30 ++++++++----------- 1 file changed, 13 insertions(+), 17 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt index 59576114c1..dd530d964f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt @@ -96,7 +96,7 @@ [% IF restriction %] -

Modify restriction type [% PROCESS restriction_type_description restriction_type=restriction %]

+

Modify restriction type '[% PROCESS restriction_type_description restriction_type=restriction %]'

[% ELSE %]

New restriction type

@@ -136,22 +136,18 @@ [% END %] [% IF op == 'delete_confirm' %] -
-
- - Confirm restriction type deletion - - -

Are you sure you want to delete '[% PROCESS restriction_type_description restriction_type=restriction %]'?

- -
- - - - Cancel -
-
-
+
+

Confirm restriction type deletion

+

Are you sure you want to delete '[% PROCESS restriction_type_description restriction_type=restriction %]'?

+
+ + + +
+
+ +
+
[% END %] [% IF op == 'list' %] -- 2.30.2