fixed onMove issue prajwalkman#22, where eventX would return NaN if e…

…vent.clientX was 0
juliankrispel · Nov 8, 2013 · 928103e · 928103e
1 parent e926b07
commit 928103e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 20 deletions.
diff --git a/angular-slider.js b/angular-slider.js
diff --git a/angular-slider.min.js b/angular-slider.min.js
diff --git a/src/angular-slider.coffee b/src/angular-slider.coffee
@@ -174,7 +174,10 @@ sliderDirective = ($timeout) ->
                         ngDocument.unbind events.move
                         ngDocument.unbind events.end
                     onMove = (event) ->
-                        eventX = event.clientX || event.touches[0].clientX
+                        if event['touches']
+                            eventX = event.touches?[0].clientX
+                        else
+                            eventX = event.clientX
                         newOffset = eventX - element[0].getBoundingClientRect().left - pointerHalfWidth
                         newOffset = Math.max(Math.min(newOffset, maxOffset), minOffset)
                         newPercent = percentOffset newOffset
@@ -227,4 +230,4 @@ module = (window, angular) ->
         .module(MODULE_NAME, [])
         .directive(SLIDER_TAG, qualifiedDirectiveDefinition)
 
-module window, window.angular
+module window, window.angular