Merge pull request imakewebthings#365 from lsthornt/master

Prevents console warnings from disabled waypoints

Author: imakewebthings

lib/waypoints.debug.js

@@ -24,7 +24,11 @@ https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
     window.Waypoint.Context.prototype.refresh = function() {
       for (var axis in this.waypoints) {
         for (var key in this.waypoints[axis]) {
-          var style = this.waypoints[axis][key].element.style
+          var waypoint = this.waypoints[axis][key]
+          var style = waypoint.element.style
+          if (!waypoint.enabled) {
+            continue
+          }
           if (style && style.display === 'none') {
             console.error(displayNoneMessage)
           }

src/debug.js

@@ -18,7 +18,11 @@
     window.Waypoint.Context.prototype.refresh = function() {
       for (var axis in this.waypoints) {
         for (var key in this.waypoints[axis]) {
-          var style = this.waypoints[axis][key].element.style
+          var waypoint = this.waypoints[axis][key]
+          var style = waypoint.element.style
+          if (!waypoint.enabled) {
+            continue
+          }
           if (style && style.display === 'none') {
             console.error(displayNoneMessage)
           }

test/debug-spec.js

@@ -54,4 +54,22 @@ describe('Waypoints debug script', function() {
   describe('fixed position detection', function() {
 
   })
+
+  describe('respect waypoint disabling', function() {
+    beforeEach(function() {
+      element = document.getElementById('same1')
+      waypoint = new Waypoint({
+        element: element,
+        handler: function() {}
+      })
+      element.style.display = 'none'
+      waypoint.disable()
+    })
+
+    it('does not log a console error', function() {
+      spyOn(console, 'error')
+      waypoint.context.refresh()
+      expect(console.error.calls.length).toEqual(0)
+    })
+  })
 })