Attachment 10879 Details for Bug 8449 – Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen

[patch] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen

0001-Bug-8449-Improve-wording-and-styling-for-cancelled-h.patch (text/plain), 2.93 KB, created by David Cook on 2012-07-16 23:53:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Cook

Created: 2012-07-16 23:53:27 UTC

Size: 2.93 KB

patch

obsolete

>From e00a3bfd44c5ce6a24490ed77b32c54fdd931db6 Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Tue, 17 Jul 2012 09:48:16 +1000
>Subject: [PATCH] Bug 8449 - Improve wording and styling for "cancelled holds"
> on Holds Awaiting Pickup screen
>Content-Type: text/plain; charset="utf-8"
>
>This patch changes the div classes, so that the prompts for action stand out and actually grab the attention of the user.
>
>It also adds more precise, descriptive language for the prompt messages and fixes the grammar (e.g. comma splices). The most important part being the second instruction where staff are prompted to check-in the item to continue processing the hold for the next patron in the holds queue.
>---
> .../prog/en/modules/circ/waitingreserves.tt        |   14 +++++++-------
> 1 files changed, 7 insertions(+), 7 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt
>index 0101d7f..24027e2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt
>@@ -46,9 +46,9 @@ $.tablesorter.addParser({
>         </h2>
>     [% IF ( cancel_result ) %]
>         [% IF ( cancel_result.messagetransfert ) %]
>-            <div>
>-                <h2>Hold find for ([% nextreservtitle %]), must be transferred</h2>
>-                <p>This hold placed by : <b> [% nextreservsurname %] [% nextreservfirstname %]</b> at the library : <b> [% branchname %] </b>, Please transfer this hold.
>+            <div class="dialog alert">
>+                <h2>This item is on hold for pick-up at another library</h2>
>+                <p>This hold ([% nextreservtitle %]) was placed by : <b> [% nextreservsurname %], [% nextreservfirstname %]</b> for pick-up at the library : <b> [% branchname %] </b>. Please transfer this hold.
>                 </p>
>                 <form name="cancelReservewithtransfert" action="waitingreserves.pl" method="post">
>                     <input type="submit" class="button" />
>@@ -56,10 +56,10 @@ $.tablesorter.addParser({
>             </div>
>         [% END %]
>         [% IF ( cancel_result.waiting ) %]
>-            <div id="bloc25">
>-                <h2>This hold is waiting</h2>
>-                <p>This hold ([% nextreservtitle %]) was placed by <b> : [% nextreservsurname %] [% nextreservfirstname %]</b>,
>-                Please retain this hold.
>+            <div class="dialog alert">
>+                <h2>This item is still on hold for pick-up at your library</h2>
>+                <p>This hold ([% nextreservtitle %]) was placed by : <b> [% nextreservsurname %], [% nextreservfirstname %]</b>.
>+                Please retain this hold, and check in the item to process the hold for this patron.
>                 </p>
>                 <form name="cancelReservewithwaiting" action="waitingreserves.pl" method="post">
>                     <input type="submit" />
>-- 
>1.7.7.4
>

Actions: View | Diff | Splinter Review

Attachments on bug 8449: 10879 | 11794 | 11795 | 12637 | 12638