correct e.clientX

react-component · Apr 22, 2015 · b61f0bb · b61f0bb
1 parent 13c4e13
commit b61f0bb
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/Slider.js b/lib/Slider.js
@@ -137,6 +137,10 @@ var Slider = React.createClass({
     this.setState({value: nextValue, active: 'active'}, callback);
   },
 
+  _getMousePosition: function(e) {
+    return e.pageX || (e.clientX + document.documentElement.scrollLeft);
+  },
+
   _triggerEvents: function(event) {
     var props = this.props;
     var hasMarks = props.marks && props.marks.length > 0;
@@ -184,7 +188,7 @@ var Slider = React.createClass({
 
   _onMouseMove: function(e) {
     pauseEvent(e);
-    var position = e.pageX || e.clientX;
+    var position = this._getMousePosition(e);
     var props = this.props;
     var state = this.state;
 
@@ -222,7 +226,7 @@ var Slider = React.createClass({
       if (this.props.disabled) {
         return;
       }
-      var position = e.pageX || e.clientX;
+      var position = this._getMousePosition(e);
       this._start(position);
       this._addEventHandles();
       pauseEvent(e);
@@ -233,7 +237,7 @@ var Slider = React.createClass({
     if (this.props.disabled) {
       return;
     }
-    var position = e.pageX || e.clientX;
+    var position = this._getMousePosition(e);
     this._calValueByPos(position, 
       () => {
         this._triggerEvents('onChange');