Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upresponsive.org - Runaway javascript #145
Comments
miketaylr
added
firefox
labels
Jun 24, 2014
This comment has been minimized.
This comment has been minimized.
Confirmed from here. Here's the offending code, needs analysis: /* FlexSlider
/* ------------------------------------ */
var firstImage = jQuery('.flexslider').find('img').filter(':first'),
checkforloaded = setInterval(function() {
var image = firstImage.get(0);
if (image.complete || image.readyState == 'complete' || image.readyState == 4) {
clearInterval(checkforloaded);
jQuery('.flexslider').flexslider({
animation: "slide",
useCSS: false, // Fix iPad flickering issue
slideshow: false,
directionNav: true,
controlNav: true,
pauseOnHover: true,
slideshowSpeed: 7000,
animationSpeed: 400,
smoothHeight: true,
touch: false
});
}
}, 20); |
This comment has been minimized.
This comment has been minimized.
OK, so the problem is
Inside of checkforloaded, they should first check for the existence of that DOM element before trying to get its properties (by checking the length property of the jQuery property like above. |
miketaylr
added
the
contactready
label
Jun 24, 2014
This comment has been minimized.
This comment has been minimized.
Tweeted at them: https://twitter.com/miketaylr/status/481554544086634496 |
miketaylr
added
sitewait
and removed
contactready
labels
Jun 24, 2014
This comment has been minimized.
This comment has been minimized.
It seems like either the site was rebuilt (the URL 404s), or they fixed this--I don't see similar errors anywhere else. |
hoosteeno commentedJun 24, 2014
URL: http://responsive.org/2014/03/the-operating-model/
Browser: Firefox
Version: 31.0
Problem type: Looks like the website has a bug.
Site owner: Unknown
Steps to Reproduce