From 8acc21b02f11534632daf4b43f21ebce5b8849ea Mon Sep 17 00:00:00 2001
From: Ruy Diaz <diazruy@gmail.com>
Date: Thu, 20 Nov 2014 11:26:26 -0800
Subject: [PATCH] Filter children to slides only when determining number of
 slides

When using MVC frameworks (Ember) to generate the HTML, additional
`<script>` nodes are inserted inside the `.slide-group` element. These
extra nodes get returned by `slider.children`, causing an incorrect
number of slides to be calculated, and hence when scrolling to the end
of the slides, you scroll into nothingness.

This patch counts only the `.slide` nodes within the slider to get the
correct slide count.
---
 js/sliders.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/js/sliders.js b/js/sliders.js
index e8bdf7382..011a09c44 100644
--- a/js/sliders.js
+++ b/js/sliders.js
@@ -47,11 +47,12 @@
   };
 
   var setSlideNumber = function (offset) {
+    var slideCount = slider.querySelectorAll('.slide').length;
     var round = offset ? (deltaX < 0 ? 'ceil' : 'floor') : 'round';
-    slideNumber = Math[round](getScroll() / (scrollableArea / slider.children.length));
+    slideNumber = Math[round](getScroll() / (scrollableArea / slideCount));
     slideNumber += offset;
     slideNumber = Math.min(slideNumber, 0);
-    slideNumber = Math.max(-(slider.children.length - 1), slideNumber);
+    slideNumber = Math.max(-(slideCount - 1), slideNumber);
   };
 
   var onTouchStart = function (e) {
@@ -60,14 +61,15 @@
     if (!slider) {
       return;
     }
+    var slideCount = slider.querySelectorAll('.slide').length;
 
     var firstItem  = slider.querySelector('.slide');
 
-    scrollableArea = firstItem.offsetWidth * slider.children.length;
+    scrollableArea = firstItem.offsetWidth * slideCount;
     isScrolling    = undefined;
     sliderWidth    = slider.offsetWidth;
     resistance     = 1;
-    lastSlide      = -(slider.children.length - 1);
+    lastSlide      = -(slideCount - 1);
     startTime      = +new Date();
     pageX          = e.touches[0].pageX;
     pageY          = e.touches[0].pageY;