Bugzilla – Attachment 21518 Details for
Bug 10836
New layout for placing holds in the OPAC
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10836: (follow-up) improve display toggles on the OPAC hold request form
Bug-10836-follow-up-improve-display-toggles-on-the.patch (text/plain), 3.07 KB, created by
Galen Charlton
on 2013-09-27 15:57:17 UTC
(
hide
)
Description:
Bug 10836: (follow-up) improve display toggles on the OPAC hold request form
Filename:
MIME Type:
Creator:
Galen Charlton
Created:
2013-09-27 15:57:17 UTC
Size:
3.07 KB
patch
obsolete
>From 118a27c33f8a3aebc277c398a9469f5dfed4cf5e Mon Sep 17 00:00:00 2001 >From: Galen Charlton <gmc@esilibrary.com> >Date: Fri, 27 Sep 2013 15:55:29 +0000 >Subject: [PATCH] Bug 10836: (follow-up) improve display toggles on the OPAC > hold request form > >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 <gmc@esilibrary.com> >--- > koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt | 12 +++--------- > 1 file changed, 3 insertions(+), 9 deletions(-) > >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..3936be9 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,8 +462,6 @@ > [% END %] > [% END %] > </ul> >- </div> >- </fieldset> > <br style="clear:both" /> > [% IF ( OPACItemHolds ) %] > [% IF ( bibitemloo.holdable ) %] >@@ -552,6 +544,8 @@ > [% END %]<!-- bib_available --> > > [% END %]<!-- OPACItemHolds --> >+ </div> >+ </fieldset> > [% END %] > > [% END %] <!-- if message --> >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10836
:
20837
|
20920
|
21412
|
21416
|
21517
|
21518
|
21519