Merge pull request #9 from benjycui/fix-bug

修复动画导致 Slider 滑块偏移的 Bug。

HISTORY.md

@@ -0,0 +1,9 @@
+# History
+----
+
+
+## 1.2.5 / 2015-07-13
+
+[#8](https://github.com/react-component/slider/issues/8) add `isIncluded` props   ([@simaQ](https://github.com/simaQ))
+
+[#7](https://github.com/react-component/slider/issues/7) add tooltip for handler when slider has no `marks` props   ([@simaQ](https://github.com/simaQ))

README.md

@@ -106,6 +106,12 @@ React.render(<Rcslider />, container);
           <td>[]</td>
           <td>mark every step for the slider, it will ignore the `step` parameter if it has been defined</td>
         </tr>
+        <tr>
+          <td>isIncluded</td>
+          <td>boolean</td>
+          <td>true</td>
+          <td>if the value is true, it means a continuous value interval, otherwise, it is a independent value.</td>
+        </tr>
         <tr>
           <td>index</td>
           <td>number</td>

assets/index.less

@@ -1,26 +1,32 @@
 @prefixClass: rc-slider;
 
-// color
-@disabledColor: #e7eaec;
+@disabledColor: #ccc;
+@border-radius-base: 6px;
+@primary-color: #2db7f5;
 
 * {
-  -webkit-tap-highlight-color: rgba(0, 0, 0, 0); //remove tap highlight color for mobile safari
+  box-sizing: border-box;
+  -webkit-tap-highlight-color: rgba(0, 0, 0, 0); //  remove tap highlight color for mobile safari
+}
+
+*:before,
+*:after {
+  box-sizing: border-box;
 }
 
 .@{prefixClass} {
   position: relative;
-  margin: 6px auto;
   height: 4px;
   width: 100%;
-  border-radius: 2px;
+  border-radius: @border-radius-base;
   background-color: #e9e9e9;
 
   &-track {
     position: absolute;
     left: 0;
     height: 4px;
-    border-radius: 2px;
-    background-color: #b2e9fd;
+    border-radius: @border-radius-base;
+    background-color: tint(@primary-color, 60%);
     z-index: 1;
   }
 
@@ -28,26 +34,21 @@
     position: absolute;
     margin-left: -7px;
     margin-top: -5px;
-    width: 10px;
-    height: 10px;
+    width: 14px;
+    height: 14px;
     cursor: pointer;
     border-radius: 50%;
-    border: solid 2px #d9d9d9;
+    border: solid 2px tint(@primary-color, 50%);
     background-color: #fff;
     z-index: 2;
 
     &:hover {
-      border-color: #999;
+      border-color: tint(@primary-color, 20%);
     }
     &-active {
-      border-color: #8cddfc;
       &:active {
-        border-color: #2db7f5;
-        background-color: #2db7f5;
-        box-shadow: 0 0 3px rgba(45, 183, 245, .75);
-      }
-      &:hover {
-        border-color: #23c0fa;
+        border-color: tint(@primary-color, 20%);
+        box-shadow: 0 0 5px tint(@primary-color, 20%);
       }
     }
   }
@@ -55,9 +56,8 @@
   &-mark {
     position: absolute;
     top: 10px;
-    left: 0px;
+    left: 0;
     width: 100%;
-    height: 20px;
     font-size: 12px;
     z-index: 3;
   }
@@ -65,17 +65,13 @@
   &-mark-text {
     position: absolute;
     display: inline-block;
-    line-height: 1.5;
-    height: 20px;
     vertical-align: middle;
     text-align: center;
     cursor: pointer;
-    color: #ccc;
-    &:first-child {
-      text-align: left;
-    }
+    color: #999;
+
     &-active {
-      color: #999;
+      color: #666;
     }
   }
 
@@ -91,21 +87,21 @@
     position: absolute;
     top: -2px;
     margin-left: -4px;
-    width: 4px;
-    height: 4px;
-    border: 2px solid #fff;
-    background-color: #bcbcbc;
+    width: 8px;
+    height: 8px;
+    border: 2px solid #e9e9e9;
+    background-color: #fff;
     cursor: pointer;
     border-radius: 50%;
     vertical-align: middle;
     &:first-child {
-      margin-left: -2px;
+      margin-left: -4px;
     }
     &:last-child {
-      margin-left: -6px;
+      margin-left: -4px;
     }
     &-active {
-      background-color: #8cddfc;
+      border-color: tint(@primary-color, 50%);
     }
   }
 

examples/simple.js

@@ -1,6 +1,7 @@
 'use strict';
 
 require('rc-slider/assets/index.css');
+require('rc-tooltip/assets/bootstrap.css');
 var Slider = require('rc-slider');
 var React = require('react');
 // React.render(<Slider marks={["一","二","三","四","五"]} index={3}/>, document.getElementById('__react-content'));

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rc-slider",
-  "version": "1.2.3",
+  "version": "1.2.5",
   "description": "slider ui component for react",
   "keywords": [
     "react",
@@ -18,7 +18,7 @@
     "url": "http://github.com/react-component/slider/issues"
   },
   "licenses": "MIT",
-  "main":"./lib/index.js",
+  "main": "./lib/index.js",
   "config": {
     "port": 8000
   },
@@ -48,6 +48,7 @@
     "precommit"
   ],
   "dependencies": {
+    "rc-tooltip": "^2.4.0",
     "rc-util": "^2.0.2"
   }
 }

src/Slider.jsx

@@ -1,6 +1,7 @@
 'use strict';
 
 var React = require('react');
+var Tooltip = require('rc-tooltip');
 var DomUtils = require('rc-util').Dom;
 
 function pauseEvent(e) {
@@ -10,9 +11,6 @@ function pauseEvent(e) {
   if (e.preventDefault) {
     e.preventDefault();
   }
-  e.cancelBubble = true;
-  e.returnValue = false;
-  return false;
 }
 
 function prefixClsFn(prefixCls) {
@@ -30,6 +28,7 @@ var Slider = React.createClass({
     value: React.PropTypes.number,
     index: React.PropTypes.number,
     marks: React.PropTypes.array,
+    isIncluded: React.PropTypes.bool,
     className: React.PropTypes.string,
     disabled: React.PropTypes.bool,
     onBeforeChange: React.PropTypes.func,
@@ -44,6 +43,7 @@ var Slider = React.createClass({
       step: 1,
       value: 0,
       marks: [],
+      isIncluded: true,
       className: 'rc-slider',
       disabled: false,
       index: 0
@@ -93,7 +93,7 @@ var Slider = React.createClass({
       return 0;
     }
     var value = this.state.value;
-    var unit = (props.max - props.min) / (props.marks.length - 1);
+    var unit = ((props.max - props.min) / (props.marks.length - 1)).toFixed(5);
     return Math.floor(value / unit);
   },
 
@@ -130,7 +130,7 @@ var Slider = React.createClass({
   },
 
   _calValueByPos: function (position, callback) {
-    var pixelOffset = position - this.state.sliderStart;
+    var pixelOffset = position - this.getSliderStart();
     // pixelOffset -= (this.state.handleSize / 2);
 
     var nextValue = this._trimAlignValue(this._calcValue(pixelOffset));
@@ -251,11 +251,17 @@ var Slider = React.createClass({
 
     this.setState({
       upperBound: slider.clientWidth,
-      sliderLength: Math.abs(sliderMax - sliderMin),
-      sliderStart: sliderMin
+      sliderLength: Math.abs(sliderMax - sliderMin)
     });
   },
 
+  getSliderStart: function() {
+    var slider = this.refs.slider.getDOMNode();
+    var rect = slider.getBoundingClientRect();
+
+    return rect.left;
+  },
+
   handleTouchStart: function(e) {
     if (this.props.disabled || e.touches.length > 1 || (e.type === 'touchend' && e.touches.length > 0)) {
       return;
@@ -292,6 +298,7 @@ var Slider = React.createClass({
         this._addEventHandles('mouse');
       }
     );
+
     pauseEvent(e);
   },
 
@@ -311,9 +318,14 @@ var Slider = React.createClass({
         left: offset.toFixed(5)
       };
       var className = prefixClsFn(prefixCls, 'dot');
-      if (i <= this.getIndex() || (this._calcValue(offset) <= this.getValue())) {
-        className = prefixClsFn(prefixCls, 'dot', 'dot-active');
+      if (props.isIncluded) {
+        if (i <= this.getIndex() || (this._calcValue(offset) <= this.getValue())) {
+          className = prefixClsFn(prefixCls, 'dot', 'dot-active');
+        }
+      } else {
+        className = (i === this.getIndex()) ? prefixClsFn(prefixCls, 'dot', 'dot-active') : className;
       }
+
       elements[i] = (
         <span className={className} style={style} ref={'step' + i}></span>
       );
@@ -336,18 +348,21 @@ var Slider = React.createClass({
       width: (unit / 2).toFixed(5)
     };
 
-    if (i === (marksLen - 1)) {
-      style.right = '0';
-      style.width = 'auto';
-    }else {
-      style.left = (i > 0 ? (offset - (unit / 4)).toFixed(5) : offset);
+    if (i === marksLen - 1) {
+      style.right = -(style.width / 2).toFixed(5);
+    } else {
+      style.left = i > 0 ? (offset - unit / 4).toFixed(5) : -(style.width / 2).toFixed(5);
     }
 
     var prefixCls = this.props.className;
     var className = prefixClsFn(prefixCls, 'mark-text');
 
-    if (i <= this.getIndex()) {
-      className = prefixClsFn(prefixCls, 'mark-text', 'mark-text-active');
+    if (this.props.isIncluded) {
+      if (i <= this.getIndex()) {
+        className = prefixClsFn(prefixCls, 'mark-text', 'mark-text-active');
+      }
+    } else {
+      className = (i === this.getIndex()) ? prefixClsFn(prefixCls, 'mark-text', 'mark-text-active') : className;
     }
 
     return (
@@ -385,14 +400,22 @@ var Slider = React.createClass({
       className =  prefixClsFn(prefixCls, 'handle', 'handle-active');
     }
 
-    return (
-      <a className={className}
+    var handle =  <a className={className}
         ref = "handle"
         style = {handleStyle}
         href = "#"
         onMouseDown = {this.handleMouseDown}
-        onTouchStart = {this.handleTouchStart}></a>
-    );
+        onTouchStart = {this.handleTouchStart}></a>;
+
+    if (this.props.marks.length > 0) {
+      return handle;
+    } else {
+      return (
+        <Tooltip placement="top" overlay={<span>{this.state.value}</span>} delay={0} >
+          {handle}
+        </Tooltip>
+      );
+    }
   },
 
   renderTrack: function(offset) {
@@ -415,7 +438,7 @@ var Slider = React.createClass({
     var value = state.value;
     var offset = this._calcOffset(value);
 
-    var track = this.renderTrack(offset);
+    var track = this.props.isIncluded ? this.renderTrack(offset) : null;
     var handles = this.renderHandle(offset);
     var steps = (props.step > 1 || props.marks.length > 0) ? this.renderSteps() : null;
     var sliderMarks = (props.marks.length > 0) ? this.renderMarks() : null;
@@ -429,6 +452,7 @@ var Slider = React.createClass({
         {handles}
         {steps}
         {sliderMarks}
+        {this.props.children}
       </div>
     );
   }