Bugzilla – Attachment 21211 Details for
Bug 10856
Improve the previous and next items on the shelf browser
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10856 [Follow-up] Improve the previous and next items on the shelf browser
Bug-10856-Follow-up-Improve-the-previous-and-next-.patch (text/plain), 4.74 KB, created by
Jonathan Druart
on 2013-09-18 14:53:25 UTC
(
hide
)
Description:
Bug 10856 [Follow-up] Improve the previous and next items on the shelf browser
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-09-18 14:53:25 UTC
Size:
4.74 KB
patch
obsolete
>From d270d9170ac306ca65195d499ddbdc06d371d65f Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Tue, 17 Sep 2013 08:33:01 -0400 >Subject: [PATCH] Bug 10856 [Follow-up] Improve the previous and next items on > the shelf browser > >In Firefox at least, the shelf browser cannot be reopened after >hiding it with the "close shelf browser" link. This followup improves >the behavior of the "close shelf browser" link so that the shelf browser >can be redisplayed. > >To test, open a bibliographic detail page in the OPAC and click a >"browse shelf" link. Click the "close shelf browser" link--the shelf >browser should be hidden. Click the original "browse shelf" link and the >shelf browser should reappear without reloading the page. > >Test with Firefox and Chrome (at least). > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc | 2 +- > koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 15 +++++++++++++-- > opac/opac-detail.pl | 1 + > 3 files changed, 15 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc b/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc >index f5672f9..b910592 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc >+++ b/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc >@@ -87,7 +87,7 @@ > $(document).ready(function(){ > $(".close_shelf").click(function(e){ > e.preventDefault(); >- $("#shelfbrowser").hide(); >+ $("#shelfbrowser").toggle(); > }); > [% IF shelfbrowser_prev_item.itemnumber %] > $("#browser_previous a").click(function(e){ >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >index 3f7053d..e5d9c3c 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -1540,12 +1540,23 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > <span class="shelvingloc">[% ITEM_RESULT.location_description %]</span> > </td> > [% IF ( itemdata_ccode ) %]<td class="collection">[% ITEM_RESULT.ccode %]</td>[% END %] >- <td class="call_no" property="sku">[% IF ( ITEM_RESULT.itemcallnumber ) %] [% ITEM_RESULT.itemcallnumber %][% IF ( OPACShelfBrowser ) %] (<a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#[% tab %]">Browse shelf</a>)[% END %][% END %]</td> >+ <td class="call_no" property="sku"> >+ [% IF ( ITEM_RESULT.itemcallnumber ) %] >+ [% ITEM_RESULT.itemcallnumber %] >+ [% IF ( OPACShelfBrowser ) %] >+ [% IF ( ITEM_RESULT.itemnumber == starting_itemnumber ) %] >+ (<a class="close_shelf" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#[% tab %]">Browse shelf</a>) >+ [% ELSE %] >+ (<a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#[% tab %]">Browse shelf</a>) >+ [% END %] >+ [% END %] >+ [% END %] >+ </td> > [% IF ( itemdata_enumchron ) %]<td class="vol_info">[% ITEM_RESULT.enumchron %]</td>[% END %] > [% IF ( itemdata_uri ) %]<td class="url"> > [% IF ITEM_RESULT.uri %] > [% IF trackclicks == 'track' || trackclicks == 'anonymous' %] >- <a href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | url %]&biblionumber=[% biblionumber |url %]&itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url"> Link to resource </a> >+ <a href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | url %]&biblionumber=[% biblionumber |url %]&itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url"> Link to resource </a> > [% ELSE %] > <a href="[% ITEM_RESULT.uri %]" property="url">[% ITEM_RESULT.uri %]</a> > [% END %] >diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl >index f914527..72e93e2 100755 >--- a/opac/opac-detail.pl >+++ b/opac/opac-detail.pl >@@ -936,6 +936,7 @@ if (C4::Context->preference("OPACShelfBrowser")) { > my $nearby = GetNearbyItems($starting_itemnumber); > > $template->param( >+ starting_itemnumber => $starting_itemnumber, > starting_homebranch => $nearby->{starting_homebranch}->{description}, > starting_location => $nearby->{starting_location}->{description}, > starting_ccode => $nearby->{starting_ccode}->{description}, >-- >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 10856
:
20964
|
20965
|
21002
|
21071
|
21136
|
21137
|
21138
|
21211
|
21319
|
21320
|
21321
|
21508
|
21666
|
21670
|
21671
|
21672
|
21673