Bugzilla – Attachment 149247 Details for
Bug 32642
Loading spinner always visible when cover image is short (OPAC)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 32642: Loading spinner always visible when cover image is short (OPAC)
Bug-32642-Loading-spinner-always-visible-when-cove.patch (text/plain), 2.55 KB, created by
Hammat wele
on 2023-04-06 23:39:31 UTC
(
hide
)
Description:
Bug 32642: Loading spinner always visible when cover image is short (OPAC)
Filename:
MIME Type:
Creator:
Hammat wele
Created:
2023-04-06 23:39:31 UTC
Size:
2.55 KB
patch
obsolete
>From c3b167cc0a4e260cf6cc6a687618de6ab701ce0f Mon Sep 17 00:00:00 2001 >From: Hammat Wele <hammat.wele@inlibro.com> >Date: Thu, 6 Apr 2023 23:38:22 +0000 >Subject: [PATCH] Bug 32642: Loading spinner always visible when cover image is > short (OPAC) > >We noticed that if the cover image is somewhat short (i.e. the height is under 80-90 px), the loading spinner indicating that the image is currently loading still appears after the image is loaded. > >this patch remove the spinner when the image is loaded,. > >To test: >1. Make sure the OPACLocalCoverImages system preference is enabled > 1.1. Go to Administration > Global system preferences > 1.2. Search for Coverimages > 1.3. Set OPACLocalCoverImages to 'Show' > 1.4. Click 'Save all Enhanced content preferences' >2. Add a short local cover image to a record (I added one which is just a grey rectangle with the size printed on it, to facilitate the test) > 2.1. Download the attached image > 2.2. Find a record without an image > 2.3. Click the 'Images' tab > 2.4. Click 'Upload' > 2.5. Click 'Drop files here or click to select a file' and choose the downloaded image > 2.6. Click 'Process images' >3. View the record in the OPAC > 3.1. From the image upload page, click on the title of the record in the page heading to access the detailed record in the staff interface > 3.2. Click on 'OPAC view: Open in new window.' > --> Note that the image is displayed normally, but the spinner is also displayed >4. Apply the patch >5. View the record in the OPAC > --> Note that the spinner is no more displayed >--- > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 9 +++++++++ > 1 file changed, 9 insertions(+) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >index ab2f28e615..1f9eb3f2fa 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >@@ -1607,6 +1607,15 @@ > $(coverSlide).remove(); > } else { > $(coverSlide).addClass("cover-slides"); >+ $(coverSlide).find("img").one("load", function(index) { >+ if(this.complete && this.naturalHeight !== 0){ >+ $(coverSlide).css({"background-image":"none"}); >+ } >+ }).each(function() { >+ if(this.complete) { >+ $(this).trigger('load'); >+ } >+ }); > } > }); > >-- >2.34.1
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 32642
:
145334
|
145472
|
145473
|
145976
|
148658
|
148661
|
148762
|
149247
|
149249
|
149250
|
149251
|
150022
|
150023