From 2068d19cd3899c78909bfb0ec3edf14ab1c86b4d Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Fri, 27 Sep 2013 15:55:29 +0000 Subject: [PATCH] [SIGNED-OFF] Bug 10836: (follow-up) improve display toggles on the OPAC hold request form Content-Type: text/plain; charset="utf-8" This patch improves the the toggles for displaying and hiding the hold options div and the toggles for displaying the table of specific copies to request. It also moves the copy table into the hold options div. To test: [1] Select a single title in the OPAC and click on the place hold button. [2] Click 'Show more options', and select a specific copy to request. [3] Click 'Hide options'. Verify that the hold options, including the copy table, are hidden. [4] Click 'Show more options'. Verify that the specific copy radio button is still selected and that the copy selected in step to is selected. [5] Enable DisplayMultiPlaceHold and select several titles to request. [6] Do steps 2, 3, and 4 on various titles on the request form. Verify that toggling the options link for one title on the form affects the visibility only for that title. Signed-off-by: Galen Charlton Signed-off-by: Owen Leonard While signing off I removed one
tag and corrected a float in the CSS. This helps keep the border between individual hold listings. --- koha-tmpl/opac-tmpl/prog/en/css/opac.css | 1 + .../opac-tmpl/prog/en/modules/opac-reserve.tt | 13 +++---------- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css index 7d40ba9..c2902c1 100644 --- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css +++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css @@ -3064,6 +3064,7 @@ padding: 0.1em 0; .holdrow fieldset { border : 0; margin : 0; + float: none; } .hold-options { clear : both; diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt index 46005ed..bf334fa 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt @@ -108,7 +108,6 @@ if (!changeSelection(newCopiesRowId, true)) { return false; } - $(".copiesrow:not(" + newCopiesRowId + ")").hide(); // Show the specific copy table for this radio button. $(newCopiesRowId).show(); @@ -126,7 +125,7 @@ } // Hide the copies table row - $(".copiesrow").hide(); + $(newCopiesRowId).hide(); }); // When 'Place Hold' button is clicked @@ -182,12 +181,7 @@ e.preventDefault(); toggleLink = $(this); var optionsID = this.id.replace("toggle-hold-options-",""); - var copiesRow = $("#copiesrow_"+optionsID) $("#hold-options-"+optionsID).toggle(0, function() { - if( copiesRow.is(':visible') ){ - $("#reqany_"+optionsID).prop("checked",true); - copiesRow.hide(); - } toggleLink.text($(this).is(':visible') ? _("Hide options") : _("Show more options")); }); }); @@ -468,9 +462,6 @@ [% END %] [% END %] - - -
[% IF ( OPACItemHolds ) %] [% IF ( bibitemloo.holdable ) %] @@ -552,6 +543,8 @@ [% END %] [% END %] + + [% END %] [% END %] -- 1.7.9.5