Moved innerheight into picturefill function

theonion · Oct 14, 2014 · e9d9ac1 · e9d9ac1
1 parent be2cd09
commit e9d9ac1
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/betty/cropper/templates/image.js b/betty/cropper/templates/image.js
@@ -7,8 +7,6 @@
       ASPECT_RATIO_TOLERANCE = .1, // 10% tolerance.
       breakpoints = [{{ BETTY_WIDTHS|join:","}}];
 
-  w.innerHeight = w.innerHeight || w.document.documentElement.clientHeight;
-
   w.picturefill = function(elements, forceRerender) {
 
     // get elements to picturefill
@@ -33,7 +31,8 @@
       }
 
       // check if image is in viewport for lazy loading
-      var visible = el.getBoundingClientRect().top <= w.innerHeight;
+      var innerHeight = w.innerHeight || w.document.documentElement.clientHeight,
+          visible = el.getBoundingClientRect().top <= innerHeight;
 
       // this is a div to picturefill, start working on it if it hasn't been rendered yet
       if (el.getAttribute("data-image-id") !== null