Add context option to Inview, fixes imakewebthings#433

Author: imakewebthings

CHANGELOG.md

@@ -7,6 +7,8 @@
 - Pass the jQuery object of items added during an Infinite page load to the `onAfterPageLoad` callback. (Pull #398)
 - Add `enabled` option, `enable` and `disable` methods to the Inview shortcut (Pull #406)
 - Make the Inview instance `this` within the callbacks, rather than the invdividual underlying waypoints. (Issue #412)
+- Account for changes to jQuery 3 around calling `offset` on the window. (Pull #430)
+- Add `context` option to Inview. (Issue #433)
 
 ## v3.1.1
 

lib/jquery.waypoints.js

@@ -348,7 +348,7 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
     /*eslint-disable eqeqeq */
     var isWindow = this.element == this.element.window
     /*eslint-enable eqeqeq */
-    var contextOffset = this.adapter.offset()
+    var contextOffset = isWindow ? undefined : this.adapter.offset()
     var triggeredGroups = {}
     var axes
 

lib/jquery.waypoints.min.js

@@ -348,7 +348,7 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
     /*eslint-disable eqeqeq */
     var isWindow = this.element == this.element.window
     /*eslint-enable eqeqeq */
-    var contextOffset = this.adapter.offset()
+    var contextOffset = isWindow ? undefined : this.adapter.offset()
     var triggeredGroups = {}
     var axes
 

lib/noframework.waypoints.js

@@ -72,6 +72,7 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
   Inview.prototype.createWaypoint = function(config) {
     var self = this
     this.waypoints.push(new Waypoint({
+      context: this.options.context,
       element: this.options.element,
       enabled: this.options.enabled,
       handler: (function(config) {
@@ -105,6 +106,7 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
   }
 
   Inview.defaults = {
+    context: window,
     enabled: true,
     enter: noop,
     entered: noop,

lib/noframework.waypoints.min.js

@@ -348,7 +348,7 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
     /*eslint-disable eqeqeq */
     var isWindow = this.element == this.element.window
     /*eslint-enable eqeqeq */
-    var contextOffset = this.adapter.offset()
+    var contextOffset = isWindow ? undefined : this.adapter.offset()
     var triggeredGroups = {}
     var axes
 

lib/shortcuts/inview.js

@@ -66,6 +66,7 @@
   Inview.prototype.createWaypoint = function(config) {
     var self = this
     this.waypoints.push(new Waypoint({
+      context: this.options.context,
       element: this.options.element,
       enabled: this.options.enabled,
       handler: (function(config) {
@@ -99,6 +100,7 @@
   }
 
   Inview.defaults = {
+    context: window,
     enabled: true,
     enter: noop,
     entered: noop,