check for element on resize; Ref #38

Possible fix for IE8, which triggers resize when body size changes. See http://snook.ca/archives/javascript/ie6_fires_onresize
metafizzy · May 14, 2013 · 0546047 · 0546047
1 parent b74b853
commit 0546047
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "packery",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "author": "David DeSandro / Metafizzy",
   "description": "bin-packing layout library",
   "main": [

diff --git a/js/packery.js b/js/packery.js
@@ -1,5 +1,5 @@
 /*!
- * Packery v1.0.3
+ * Packery v1.0.4
  * bin-packing layout library
  * http://packery.metafizzy.co
  *
@@ -612,7 +612,10 @@ Packery.prototype.onresize = function() {
 Packery.prototype.resize = function() {
   // don't trigger if size did not change
   var size = getSize( this.element );
-  if ( size.innerWidth === this.elementSize.innerWidth ) {
+  // check that elementSize and size are there
+  // IE8 triggers resize on body size change, so they might not be
+  var hasSizes = this.elementSize && size;
+  if ( hasSizes && size.innerWidth === this.elementSize.innerWidth ) {
     return;
   }