This is a followup report on bug 9722.
Created attachment 18364 [details] [review] Bug 10321: Followup for hold notes Based on work for report 9722. This patch resolves a small display problem with the number of columns of the table on opac-reserve. Test plan: Enable OPACShowHoldQueueDetails and OpacShowHoldNotes. Without this patch, the hold notes end at the one-but-last column. Test this by placing a hold in opac and click on Edit notes. Apply this patch: repeat what you did, the display should be correct. Toggle the value of OPACShowHoldQueueDetails. And check again.
Created attachment 18365 [details] [review] Bug 10321: QA Followup on report 9722 As Jonathan suggested, use jQuery toggle function. Test plan: Enable OPACShowHoldNotes. Place one hold. Repeat clicking on Edit notes to show and hide notes. Place a multi hold (two biblios at the same time). Check again with both.
Created attachment 18594 [details] [review] [SIGNED-OFF] Bug 10321: Followup for hold notes Based on work for report 9722. This patch resolves a small display problem with the number of columns of the table on opac-reserve. Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Created attachment 18595 [details] [review] [SIGNED-OFF] Bug 10321: QA Followup on report 9722 As Jonathan suggested, use jQuery toggle function. Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Created attachment 18759 [details] [review] Bug 10321: Followup for hold notes Based on work for report 9722. This patch resolves a small display problem with the number of columns of the table on opac-reserve. Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 18760 [details] [review] Bug 10321: QA Followup on report 9722 As Jonathan suggested, use jQuery toggle function. Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Pushed to master. Thanks, Marcel!