fix paper-slider on ios. closes #645 (for real).

miguelcobain · Feb 23, 2017 · a86bb64 · a86bb64
1 parent 41fe454
commit a86bb64
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/addon/components/paper-slider.js b/addon/components/paper-slider.js
@@ -132,16 +132,23 @@ export default Component.extend(FocusableMixin, ColorMixin, {
     event.preventDefault();
   },
 
+  // needed for iOS
+  // iOS doesn't trigger a click event on normal divs
+  // unless we use `cursor: pointer` css
+  touchEnd(e) {
+    this.click(e);
+  },
+
   click(event) {
     if (this.get('disabled')) {
       return;
     }
 
-    this.setValueFromEvent(event);
+    this.setValueFromEvent(event.pageX || event.clientX);
   },
 
-  setValueFromEvent(event) {
-    let exactVal = this.percentToValue(this.positionToPercent(event.clientX || event.srcEvent.clientX));
+  setValueFromEvent(value) {
+    let exactVal = this.percentToValue(this.positionToPercent(value));
     let closestVal = this.minMaxValidator(this.stepValidator(exactVal));
 
     this.sendAction('onChange', closestVal);
@@ -156,18 +163,18 @@ export default Component.extend(FocusableMixin, ColorMixin, {
     this.set('dragging', true);
     this.$().focus();
 
-    this.setValueFromEvent(event);
+    this.setValueFromEvent(event.center.x);
   },
 
   drag(event) {
     if (this.get('disabled') || !this.get('dragging')) {
       return;
     }
 
-    this.setValueFromEvent(event);
+    this.setValueFromEvent(event.center.x);
   },
 
-  dragEnd() {
+  dragEnd(event) {
     if (this.get('disabled')) {
       return;
     }