Created attachment 10879 [details] [review] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen 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.
Patch doesn't apply anymore Applying: Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt Failed to merge in the changes. Patch failed at 0001 Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen
Created attachment 11794 [details] [review] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen This patch re-implements and adapts the changes dcook@prosentient.com.au submitted in a patch which no longer applies. I have changed the "alert" style messages to "message" style to match the style which is applied to "hold found" messages elsewhere. I have also revised the language in a way that I think is more clear and accurate.
Created attachment 11795 [details] [review] [SIGNED-OFF] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen This patch re-implements and adapts the changes dcook@prosentient.com.au submitted in a patch which no longer applies. I have changed the "alert" style messages to "message" style to match the style which is applied to "hold found" messages elsewhere. I have also revised the language in a way that I think is more clear and accurate. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
String changes only. Marked as Passed QA.
Question: Owen, you replace: - <p>This hold placed <snip> at the library : <b> [% branchname %] </b>, Please transfer this hold. by + Please retain this item and check it in to process the hold. IE: you loose the information of the branch you must transfer it to. Was it intended ? Woudln't it be better to have + <h2>This item is on hold for pick-up at another library ([% branchname %])</h2> or something like that?
(In reply to comment #6) > Question: > > Owen, you replace: > - <p>This hold placed <snip> at the library : <b> [% branchname %] </b>, > Please transfer this hold. > > by > + Please retain this item and check it in to process the hold. > > IE: you loose the information of the branch you must transfer it to. Was it > intended ? > > Woudln't it be better to have > + <h2>This item is on hold for pick-up at another library ([% branchname > %])</h2> > > or something like that? I don't mean to speak for Owen, but the branchname information isn't particularly useful or relevant at this stage. Really, the staff member just needs to know that the item needs to be transferred (rather than shelved at their location), and that they process the transfer by checking in the item. Once they check it in, they'll see the branch to transfer it to and have the option of printing a slip, I believe.
Created attachment 12637 [details] [review] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen This patch re-implements and adapts the changes dcook@prosentient.com.au submitted in a patch which no longer applies. I have changed the "alert" style messages to "message" style to match the style which is applied to "hold found" messages elsewhere. I have also revised the language in a way that I think is more clear and accurate. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Created attachment 12638 [details] [review] [FOLLOW-UP] Bug 8449 - Improve wording and styling for "cancelled holds" on Holds Awaiting Pickup screen (In reply to comment #6) > Question: > > IE: you lose the information of the branch you must transfer it to. Was it > intended ? I don't recall my reasoning when I submitted the patch but I agree that having the library name is useful. This follow-up returns the library name to message about hold requiring transfer
Patch pushed to master (would it be considered by chris_c as a bugfix I would not object)