Attachment 142560 Details for Bug 31911 – Bug 31911: Improve rotating collections item management

[patch] Bug 31911: Improve rotating collections item management

Bug-31911-Improve-rotating-collections-item-manage.patch (text/plain), 4.36 KB, created by Katrin Fischer on 2022-10-24 19:42:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2022-10-24 19:42:00 UTC

Size: 4.36 KB

patch

obsolete

>From 386c6fb325b09abdd16c117d1452a9b8320737b2 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Fri, 21 Oct 2022 15:14:56 +0100
>Subject: [PATCH] Bug 31911: Improve rotating collections item management
>
>This patch renames 'Add/remove items from collection' to just 'Manage
>items' in the button in the rotating collections list table, the page
>heading and the breadcrumbs.
>
>We also move the heading below the toolbar to be consistent with
>elsewhere, remove a superflous second level heading and wrap the content
>table in a page-section too.
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> .../prog/en/modules/rotating_collections/addItems.tt  | 11 ++++-------
> .../rotating_collections/rotatingCollections.tt       |  2 +-
> 2 files changed, 5 insertions(+), 8 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/addItems.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/addItems.tt
>index 653077e9d1..7d44604d1f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/addItems.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/addItems.tt
>@@ -28,7 +28,7 @@
>         </li>
>         <li>
>             <a href="#" aria-current="page">
>-                Add or remove items
>+                Manage items
>             </a>
>         </li>
>     </ol>
>@@ -39,8 +39,6 @@
>     <div class="row">
>         <div class="col-sm-10 col-sm-push-2">
>             <main>
>-                <h1>Add/remove items from collection</h1>
>-
>               [% IF ( ! colId ) %]
>                 <div class="dialog message">
>                     <p>Invalid collection id</p>
>@@ -48,7 +46,7 @@
>                 </div>
>               [% ELSE %]
>               [% INCLUDE 'rotating-collections-toolbar.inc' %]
>-              <h2>Collection <em>[% colTitle | html %]</em></h2>
>+              <h1>Manage items for <em>[% colTitle | html %]</em></h1>
> 
>               [% IF ( previousActionAdd ) %]
>                 [% IF ( addSuccess ) %]
>@@ -115,10 +113,9 @@
>                 </form>
>               </div>
> 
>-              <div>
>-                <h2>Items in <em>[% colTitle | html %]</em></h2>
>+              <div class="page-section">
>                 [% IF ( collectionItemsLoop ) %]
>-                  <h3>Total items: [% collectionItemsLoop.size | html%]</h3>
>+                  <h2>Total items: [% collectionItemsLoop.size | html%]</h2>
>                   <table>
>                     <tr>
>                       <th>Title</th>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/rotatingCollections.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/rotatingCollections.tt
>index 58d806318a..9bbf4d4a23 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/rotatingCollections.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/rotatingCollections.tt
>@@ -61,7 +61,7 @@
>                                             <a class="btn btn-default btn-xs dropdown-toggle" id="collectionsactions[% collectionsLoo.colId | html %]" role="button" data-toggle="dropdown" href="#">
>                                             Actions <b class="caret"></b></a>
>                                             <ul class="dropdown-menu pull-right" role="menu" aria-labelledby="collectionsactions[% collectionsLoo.colId | html %]">
>-                                                <li><a href="/cgi-bin/koha/rotating_collections/addItems.pl?colId=[% collectionsLoo.colId | uri %]"><i class="fa fa-plus"></i> Add or remove items</a></li>
>+                                                <li><a href="/cgi-bin/koha/rotating_collections/addItems.pl?colId=[% collectionsLoo.colId | uri %]"><i class="fa fa-plus"></i> Manage items</a></li>
>                                                 <li><a href="/cgi-bin/koha/rotating_collections/transferCollection.pl?colId=[% collectionsLoo.colId | uri %]"><i class="fa fa-exchange"></i> Transfer</a></li>
>                                                 <li><a href="/cgi-bin/koha/rotating_collections/editCollections.pl?action=edit&amp;colId=[% collectionsLoo.colId | uri %]"><i class="fa fa-pencil"></i> Edit</a></li>
>                                                 <li><a class="confirmdelete" href="/cgi-bin/koha/rotating_collections/editCollections.pl?action=delete&amp;colId=[% collectionsLoo.colId | html %]"><i class="fa fa-trash"></i> Delete</a></li>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 31911: 142398 | 142504 | 142505 | 142558 | 142559 | 142560 | 142561 | 142562