From e2d97188760dedb2c68e6dcda62db4c1193c7de6 Mon Sep 17 00:00:00 2001 From: Aleisha Date: Thu, 7 Jul 2016 22:20:55 +0000 Subject: [PATCH] Bug 16874: Making serials collections actions buttons To test: 1) Go to serials -> Search a subscription (or make a new one) 2) Ensure that subscription has an end-date that has passed or will pass soon 3) Go to Serials collection (left sidebar menu) 4) Ensure that Subscription summary table has two buttons, Create routing list and Renew. These buttons should be bolded and the table cell highlighted 5) Ensure 'Print list' is a button in year tables below 6) Edit subscription to have a later end-date (will not expire soon) 7) Go back to serials collection 8) Ensure that table cell is no longer highlighted and buttons not bolded. Renew button should not be there. 9) Ensure that both buttons do not wrap when the browser is narrow Sponsored-by: Catalyst IT Signed-off-by: Lisette Scheer --- koha-tmpl/intranet-tmpl/prog/css/staff-global.css | 2 +- .../prog/en/modules/serials/serials-collection.tt | 29 +++++++++----------- 2 files changed, 14 insertions(+), 17 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css index 525cd0e..de0e492 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css @@ -305,7 +305,7 @@ caption { margin : .3em 0; } -span.problem { +.problem { background-color : #FFFFCC; color : #990000; font-weight : bold; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt index 9a79eca..f723626 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt @@ -113,8 +113,7 @@ $(document).ready(function() { Library Call number Notes - [% IF ( routing && CAN_user_serials_routing ) %]Routing[% END %] - Renew +   [% FOREACH subscription IN subscriptions %] @@ -132,28 +131,26 @@ $(document).ready(function() {
Subscription closed [% END %] - [% IF ( routing && CAN_user_serials_routing ) %] - - [% UNLESS subscription.closed %] + [% IF ( subscription.abouttoexpire ) || ( subscription.subscriptionexpired ) %][% ELSE %][% END %] + [% UNLESS subscription.closed %] + [% IF ( routing && CAN_user_serials_routing ) %] [% IF ( subscription.hasRouting ) %] - Edit routing list + Edit routing list [% ELSE %] - Create routing list + Create routing list [% END %] [% END %] - - [% END %] - [% UNLESS subscription.closed %] - [% IF ( subscription.abouttoexpire ) %] Renew + [% IF ( subscription.abouttoexpire ) %] Renew [% ELSE %] - [% IF ( subscription.subscriptionexpired ) %] Renew + [% IF ( subscription.subscriptionexpired ) %] Renew [% ELSE %] -   +   [% END %] [% END %] [% ELSE %] -   +   [% END %] + [% END %] [% IF ( subscr ) %] @@ -303,8 +300,8 @@ $(document).ready(function() { [% Branches.GetName( serial.branchcode ) %] [% IF ( routing ) %] - - Print list + + Print list [% END %] -- 1.7.10.4