Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 34724: (QA follow-up):

Browse source 
Replace href from thumbnail link with a # to keep pointer event. The href link gets reconstructed by showCover anyway
Remove return false from on click event, its not triggering a location change anymore because of the above
Remove check for imagenumber, its a non-null primary key, if the image exists, then the imagenumber must also exist

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit bc8a062001)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 167f5aa3bb)
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
This commit is contained in:
Pedro Amorim 2023-09-08 16:01:03 +00:00 committed by Matt Blenkinsop
parent 1c19a1cc14
commit 8e7b3709c6
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-imageviewer.tt
View file

@ -87,7 +87,7 @@
<div class="col-lg-3">
<div id="thumbnails">
[% FOREACH image IN images %]
<a class="cover_thumbnail" data-imagenumber="[% image.imagenumber| html %]" href="/cgi-bin/koha/opac-imageviewer.pl?biblionumber=[% biblionumber | url %]&amp;imagenumber=[% image.imagenumber| uri %]">
<a class="cover_thumbnail" data-imagenumber="[% image.imagenumber| html %]" href="#">
[% IF loop.first %]
<img class="thumbnail selected" id="[% image.imagenumber| html %]" src="/cgi-bin/koha/opac-image.pl?imagenumber=[% image.imagenumber | html %]&amp;thumbnail=1" alt="Thumbnail"/>
[% ELSE %]
@ -114,11 +114,7 @@
$("#largeCoverImg").attr("src","[% interface | html %]/[% theme | html %]/images/loading.gif");
showCover([% imagenumber | html %]);
$("a.cover_thumbnail").on('click',function(){
let imagenumber = this.dataset.imagenumber;
if (imagenumber){
showCover(imagenumber);
}
return false;
showCover(this.dataset.imagenumber);
});
});