Fix: loadFullSizeImageImageTimeout not clearing when mouse leaves ima…

…ge (#1516) - This fixes what #1515 was trying to fix - setLoadImage used to set loadHoveredImage within callback so timeout can be set outside of callback. This makes clearTimeout use the correct timeout ID - There is probably a smarter way of fixing this, but this works --------- Co-authored-by: Oleg Anashkin <[email protected]>
extesy · Dec 31, 2024 · 5671342 · 5671342
1 parent 4cb6b61
commit 5671342
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/js/hoverzoom.js b/js/hoverzoom.js
@@ -1070,8 +1070,10 @@ var hoverZoom = {
 
                             // if the action key has been pressed over an image, no delay is applied
                             const delay = actionKeyDown || explicitCall ? 0 : (isVideoLink(srcDetails.url) ? options.displayDelayVideo : options.displayDelay);
+                            loadFullSizeImageTimeout = setTimeout(loadFullSizeImage, delay);
 
-                            if (audioSrc) {
+                            // Temporarily removing until a better fix is found: sendMessage is async so it can't be used to set local variables
+                            /*if (audioSrc) {
                                 chrome.runtime.sendMessage({action:'isImageBanned', url:audioSrc}, function (result) {
                                     if (!result) {
                                         loadFullSizeImageTimeout = setTimeout(loadFullSizeImage, delay);
@@ -1083,7 +1085,7 @@ var hoverZoom = {
                                         loadFullSizeImageTimeout = setTimeout(loadFullSizeImage, delay);
                                     }
                                 });                               
-                            }
+                            }*/
 
                             loading = true;
                         }