Bug 13315 [Follow-up] Add feedback for last item checked out to circulation.pl

This patch changes the position and styling of the information about the
last checkout so that it is grouped with the checkout form itself,
placing the message in a more visible position, especially for patrons
with a lot of information in the right-hand messages column which might
push the checkouts table down offscreen.

To test, apply the patch and clear your browser cache in order to
refresh the main CSS file.

Check out an item and confirm that a message is prominently displayed.

Signed-off-by: Jason Burds  <jburds@dubuque.lib.ia.us>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>

diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
index 1759f46..b92f490 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
+++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
@@ -333,6 +333,13 @@ fieldset {
     border-radius:5px;
 }
 
+fieldset.lastchecked {
+    margin-bottom : 0;
+    border-bottom-width: 0;
+    border-bottom-left-radius: 0;
+    border-bottom-right-radius: 0;
+}
+
 legend {
        font-size : 123.1%;
        font-weight : bold;
@@ -2764,7 +2771,12 @@ span.onsite_checkout {
     border-radius: 4px;
     border : 1px solid #FFF2CE;
 }
-
+div.lastchecked {
+    padding : .2em 1em;
+    border: 2px solid #BCDB89;
+    border-bottom-left-radius: 5px;
+    border-bottom-right-radius: 5px;
+}
 .branchgriditem {
     background-color: #fff;
     border: 1px solid #b9d8d9;

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
index e845ee0..233114d 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -560,7 +560,11 @@ No patron matched <span class="ex">[% message %]</span>
 
 
 <form method="post" action="/cgi-bin/koha/circ/circulation.pl" id="mainform" name="mainform" autocomplete="off">
-<fieldset id="circ_circulation_issue">
+[% IF ( issue ) %]
+    <fieldset id="circ_circulation_issue" class="lastchecked">
+[% ELSE %]
+    <fieldset id="circ_circulation_issue">
+[% END %]
     [% IF ( DisplayClearScreenButton ) %]
         <span id="clearscreen"><a href="/cgi-bin/koha/circ/circulation.pl" title="Clear screen">x</a></span>
     [% END %]
@@ -631,7 +635,14 @@ No patron matched <span class="ex">[% message %]</span>
                         <input type="hidden" name="oldamount" value="[% amountold %]" />
                 [% END %]
 </fieldset>
-</form></div>[% END %]<!-- /unless noissues -->
+[% IF ( issue ) %]
+    <div class="lastchecked">
+        <p><strong>Checked out: </strong>[% issue.item.biblio.title %] ([% issue.item.barcode %]). Due on [% issue.date_due | $KohaDates %]</p>
+    </div>
+[% END %]
+</form></div>
+
+[% END %]<!-- /unless noissues -->
 
 [% IF ( noissues ) %]
     [% IF ( Koha.Preference('OnSiteCheckouts') && Koha.Preference('OnSiteCheckoutsForce') ) %]
@@ -795,10 +806,6 @@ No patron matched <span class="ex">[% message %]</span>
 
 </div>
 </div>
-[% IF ( issue ) %]
-    <div class="dialog message">[% issue.item.biblio.title %] ([% issue.item.barcode %]) due on [% issue.date_due | $KohaDates %]</div>
-[% END %]
-
 
 <div class="yui-g"><div id="patronlists" class="toptabs">