Run fresh waypoint triggers on next animation frame, fixes imakewebthings#384

Author: imakewebthings

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v4.0.0
+
+- Allow Sticky option `wrapper` to accept false, which will not create a wrapper and instead use the preexisting parent element.   (Pull #416)
+
 ## v3.1.1
 
 - Fix bad `isWindow` checks causing errors in IE8-. (Issue #372)

lib/jquery.waypoints.js

@@ -421,9 +421,11 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
       }
     }
 
-    for (var groupKey in triggeredGroups) {
-      triggeredGroups[groupKey].flushTriggers()
-    }
+    Waypoint.requestAnimationFrame(function() {
+      for (var groupKey in triggeredGroups) {
+        triggeredGroups[groupKey].flushTriggers()
+      }
+    })
 
     return this
   }

lib/jquery.waypoints.min.js

@@ -421,9 +421,11 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
       }
     }
 
-    for (var groupKey in triggeredGroups) {
-      triggeredGroups[groupKey].flushTriggers()
-    }
+    Waypoint.requestAnimationFrame(function() {
+      for (var groupKey in triggeredGroups) {
+        triggeredGroups[groupKey].flushTriggers()
+      }
+    })
 
     return this
   }

lib/noframework.waypoints.js

@@ -41,7 +41,9 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
 
   /* Private */
   Sticky.prototype.createWrapper = function() {
-    this.$element.wrap(this.options.wrapper)
+    if (this.options.wrapper) {
+      this.$element.wrap(this.options.wrapper)
+    }
     this.$wrapper = this.$element.parent()
     this.wrapper = this.$wrapper[0]
   }
@@ -50,7 +52,10 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
   Sticky.prototype.destroy = function() {
     if (this.$element.parent()[0] === this.wrapper) {
       this.waypoint.destroy()
-      this.$element.removeClass(this.options.stuckClass).unwrap()
+      this.$element.removeClass(this.options.stuckClass)
+      if (this.options.wrapper) {
+        this.$element.unwrap()
+      }
     }
   }
 

lib/noframework.waypoints.min.js

@@ -421,9 +421,11 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
       }
     }
 
-    for (var groupKey in triggeredGroups) {
-      triggeredGroups[groupKey].flushTriggers()
-    }
+    Waypoint.requestAnimationFrame(function() {
+      for (var groupKey in triggeredGroups) {
+        triggeredGroups[groupKey].flushTriggers()
+      }
+    })
 
     return this
   }

lib/shortcuts/sticky.js

@@ -255,9 +255,11 @@
       }
     }
 
-    for (var groupKey in triggeredGroups) {
-      triggeredGroups[groupKey].flushTriggers()
-    }
+    Waypoint.requestAnimationFrame(function() {
+      for (var groupKey in triggeredGroups) {
+        triggeredGroups[groupKey].flushTriggers()
+      }
+    })
 
     return this
   }