update version.

refactor: extract sub components from Slider

.travis.yml

@@ -7,7 +7,7 @@ notifications:
   - yiminghe@gmail.com
 
 node_js:
-- 0.12
+- 4.0.0
 
 before_install:
 - |
@@ -16,7 +16,6 @@ before_install:
       echo "Only docs were updated, stopping build process."
       exit
     fi
-    npm install mocha-phantomjs -g
     phantomjs --version
 
 script:
@@ -36,3 +35,8 @@ env:
   global:
   - secure: S1VwbaPzLnSH/IUT/wlJulxAX5VHRIDmSt53h/ycHcZsszUpWcLCJRQAe0fTVB2dAx5MdBbSZ+o+tr3tRwVB5TRAYm0oTCsYAkOZaWOB28RuUQtdGt3wf9xxTG1UiPiaLLUW3waX9zAaf3yqKBcJGf1op0RD8dksxbCFw/7xVbU=
   - secure: EBEDg8k///IlEsnx0AE8X3mbFl0QE5+xGKbG4AxXlGZda12uTIPUSMKJzdZQ2hVbZXduTzf1cQl9rcu9nGoSnkL/DWnIax9cvHi+1orx5+YPlxPHNWAwWByTnHosBn2MJhfy1s5paJfHC9cUzmmEL6x4fYthWxjsPUo+irEZH6E=
+
+
+matrix:
+  allow_failures:
+  - env: "TEST_TYPE=saucelabs"

README.md

@@ -4,7 +4,6 @@
 slider ui component for react
 
 [![NPM version][npm-image]][npm-url]
-[![SPM version](http://spmjs.io/badge/rc-slider)](http://spmjs.io/package/rc-slider)
 [![build status][travis-image]][travis-url]
 [![Test coverage][coveralls-image]][coveralls-url]
 [![gemnasium deps][gemnasium-image]][gemnasium-url]
@@ -51,9 +50,10 @@ slider ui component for react
 ## Usage
 
 ```js
-var Rcslider = require('rc-slider');
 var React = require('react');
-React.render(<Rcslider />, container);
+var ReactDOM = require('react-dom');
+var Rcslider = require('rc-slider');
+ReactDOM.render(<Rcslider />, container);
 ```
 
 ## API
@@ -142,6 +142,12 @@ React.render(<Rcslider />, container);
           <td>''</td>
           <td>Set the animation for tooltip if it shows.</td>
         </tr>
+        <tr>
+          <td>withDots</td>
+          <td>bool</td>
+          <td>false</td>
+          <td>For linear slider, when the `step` value is greater than 1, you can set the `withDots` to  `true` if you want to render the slider bar with dots.</td>
+        </tr>
     </tbody>
 </table>
 

assets/index.less

@@ -11,14 +11,14 @@
 @ease-out-quint      : cubic-bezier(0.23, 1, 0.32, 1);
 @ease-in-quint       : cubic-bezier(0.755, 0.05, 0.855, 0.06);
 
-* {
+.borderBox() {
   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;
+  * {
+    box-sizing: border-box;
+    -webkit-tap-highlight-color: rgba(0, 0, 0, 0); //  remove tap highlight color for mobile safari
+  }
 }
 
 .@{prefixClass} {
@@ -28,6 +28,8 @@
   border-radius: @border-radius-base;
   background-color: #e9e9e9;
 
+  .borderBox();
+
   &-track {
     position: absolute;
     left: 0;
@@ -119,13 +121,13 @@
       background-color: @disabledColor;
     }
 
-    .@{prefixClass}-handle {
+    .@{prefixClass}-handle, .@{prefixClass}-dot {
       border-color: @disabledColor;
       background-color: #fff;
       cursor: not-allowed;
     }
 
-    .@{prefixClass}-mark-text, .dot {
+    .@{prefixClass}-mark-text, .@{prefixClass}-dot {
       cursor: not-allowed!important;
     }
   }
@@ -138,6 +140,8 @@
     z-index: 4;
     visibility: visible;
 
+    .borderBox();
+
     &-hidden {
       display: none;
     }
@@ -212,9 +216,9 @@
     animation-timing-function: @ease-in-quint;
   }
 }
-.zoom-motion(zoom-down, zoomDown);
+.zoom-motion(rc-slider-tooltip-zoom-down, rcSliderTooltipZoomDown);
 
-@keyframes zoomDownIn {
+@keyframes rcSliderTooltipZoomDownIn {
   0% {
     opacity: 0;
     transform-origin: 50% 100%;
@@ -226,7 +230,7 @@
   }
 }
 
-@keyframes zoomDownOut {
+@keyframes rcSliderTooltipZoomDownOut {
   0% {
     transform-origin: 50% 100%;
     transform: scale(1, 1);
@@ -236,4 +240,4 @@
     transform-origin: 50% 100%;
     transform: scale(0, 0);
   }
-}
+}

examples/simple.js

@@ -2,13 +2,34 @@
 
 require('rc-slider/assets/index.less');
 
-var Slider = require('rc-slider');
 var React = require('react');
+var Slider = require('rc-slider');
 
-function onChange(v){
+function onChange(v) {
   console.log(v);
 }
 
-// React.render(<div style={{width:400,margin:100}}><Slider marks={["一","二","三","四","五"]} defaultIndex={2} /></div>, document.getElementById('__react-content'));
-// React.render(<div style={{width:400,margin:100}}><Slider className='rc-slider' step={20}/></div>, document.getElementById('__react-content'));
-React.render(<div style={{width:400,margin:100}}><Slider onChange={onChange}  tipTransitionName='zoom-down'/></div>, document.getElementById('__react-content'));
+React.render(
+  <div>
+    <div style={{width:400,margin:100}}>
+      <Slider onChange={onChange} tipTransitionName='rc-slider-tooltip-zoom-down'/>
+    </div>
+    <div style={{width:400,margin:100}}>
+      <Slider marks={["一","二","三","四","五"]} defaultIndex={2}/>
+    </div>
+    <div style={{width:400,margin:100}}>
+      <Slider withDots step={20}/>
+    </div>
+    <div style={{width:400,margin:100}}>
+      <Slider step={20}/>
+    </div>
+    <div style={{width:400,margin:100}}>
+      <p>包含关系</p>
+      <Slider marks={["状态1","状态2","状态3","状态4"]} defaultIndex={1} />
+    </div>
+    <div style={{width:400,margin:100}}>
+      <p>并列关系</p>
+      <Slider marks={["状态1","状态2","状态3","状态4"]} isIncluded={false} defaultIndex={1} />
+    </div>
+  </div>
+  , document.getElementById('__react-content'));

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rc-slider",
-  "version": "1.4.4",
+  "version": "1.5.2",
   "description": "slider ui component for react",
   "keywords": [
     "react",
@@ -29,22 +29,23 @@
   "scripts": {
     "build": "rc-tools run build",
     "gh-pages": "rc-tools run gh-pages",
-    "start": "node --harmony node_modules/.bin/rc-server",
+    "start": "rc-server",
     "pub": "rc-tools run pub",
     "lint": "rc-tools run lint",
     "karma": "rc-tools run karma",
-    "saucelabs": "node --harmony node_modules/.bin/rc-tools run saucelabs",
-    "browser-test": "node --harmony node_modules/.bin/rc-tools run browser-test",
-    "browser-test-cover": "node --harmony node_modules/.bin/rc-tools run browser-test-cover"
+    "saucelabs": "rc-tools run saucelabs",
+    "browser-test": "rc-tools run browser-test",
+    "browser-test-cover": "rc-tools run browser-test-cover"
   },
   "devDependencies": {
     "expect.js": "0.3.x",
+    "jquery": "^1.11.2",
     "precommit-hook": "^1.0.7",
     "rc-server": "3.x",
     "rc-tools": "4.x",
-    "react": "^0.13.x",
-    "node-dev": "2.x",
-    "jquery": "^1.11.2"
+    "react": "~0.14.0",
+    "react-addons-test-utils": "^0.14.0",
+    "react-dom": "~0.14.0"
   },
   "precommit": [
     "lint"

src/Handle.jsx

@@ -0,0 +1,60 @@
+import React from 'react';
+import Tooltip from 'rc-tooltip';
+
+export default class Handle extends React.Component {
+  constructor(props) {
+    super(props);
+
+    this.state = {
+      isTooltipVisible: false,
+    };
+  }
+
+  render() {
+    const props = this.props;
+    const {className, prefixCls, offset, tipTransitionName, value} = props;
+    const {dragging, noTip} = props;
+
+    const style = { left: offset + '%' };
+    const handle = (<div className={className} style={style}
+                      onMouseUp={this.showTooltip.bind(this)}
+                      onMouseEnter={this.showTooltip.bind(this)}
+                      onMouseLeave={this.hideTooltip.bind(this)}/>);
+
+    if (noTip) {
+      return handle;
+    }
+
+    const isTooltipVisible = dragging || this.state.isTooltipVisible;
+    return (<Tooltip
+              prefixCls={prefixCls + '-tooltip'}
+              placement={{points: ['bc', 'tc']}}
+              visible={isTooltipVisible}
+              overlay={<span>{value}</span>}
+              delay={0}
+              transitionName={tipTransitionName}>
+              {handle}
+            </Tooltip>);
+  }
+
+  showTooltip() {
+    this.setState({
+      isTooltipVisible: true,
+    });
+  }
+  hideTooltip() {
+    this.setState({
+      isTooltipVisible: false,
+    });
+  }
+}
+
+Handle.propTypes = {
+  className: React.PropTypes.string,
+  prefixCls: React.PropTypes.string,
+  offset: React.PropTypes.number,
+  tipTransitionName: React.PropTypes.string,
+  value: React.PropTypes.number,
+  dragging: React.PropTypes.bool,
+  noTip: React.PropTypes.bool,
+};

src/Marks.jsx

@@ -0,0 +1,35 @@
+import React from 'react';
+import rcUtil from 'rc-util';
+
+const Marks = ({className, marks, index, isIncluded}) => {
+  const marksLen = marks.length;
+  const unit = 100 / (marksLen - 1);
+  const markWidth = unit / 2 + '%';
+
+  const elements = [];
+  for (let i = 0; i < marksLen; i++) {
+    const isActived = (isIncluded && i <= index) || (!isIncluded && i === index);
+    const markClassName = rcUtil.classSet({
+      [className + '-text']: true,
+      [className + '-text-active']: isActived,
+    });
+
+    const style = { width: markWidth };
+    const offset = unit * i;
+    if (i === marksLen - 1) {
+      style.right = -unit / 4 + '%';
+    } else {
+      style.left = (i > 0 ? offset - unit / 4 : -unit / 4) + '%';
+    }
+
+    elements.push(<span className={markClassName} style={style} key={i}>
+                    {marks[i]}
+                  </span>);
+  }
+
+  return (<div className={className}>
+            {elements}
+          </div>);
+};
+
+export default Marks;

src/Slider.jsx

@@ -1,6 +1,9 @@
 import React from 'react';
-import Tooltip from 'rc-tooltip';
 import rcUtil, {Dom as DomUtils} from 'rc-util';
+import Track from './Track';
+import Handle from './Handle';
+import Steps from './Steps';
+import Marks from './Marks';
 
 function noop() {
 }
@@ -16,12 +19,6 @@ function pauseEvent(e) {
   }
 }
 
-function prefixClsFn(prefixCls, ...args) {
-  return args.map((s)=> {
-    return prefixCls + '-' + s;
-  }).join(' ');
-}
-
 function getValueFromIndex(props) {
   let value;
   const marksLen = props.marks.length;
@@ -57,6 +54,7 @@ const Slider = React.createClass({
     onChange: React.PropTypes.func,
     onAfterChange: React.PropTypes.func,
     tipTransitionName: React.PropTypes.string,
+    withDots: React.PropTypes.bool,
   },
 
   getDefaultProps() {
@@ -72,6 +70,7 @@ const Slider = React.createClass({
       disabled: false,
       defaultIndex: 0,
       tipTransitionName: '',
+      withDots: false,
     };
   },
 
@@ -88,7 +87,6 @@ const Slider = React.createClass({
     }
     return {
       dragging: false,
-      showTooltip: false,
       value: value,
     };
   },
@@ -105,14 +103,8 @@ const Slider = React.createClass({
     }
   },
 
-  onMouseUp(e) {
-    const m = e.target;
-    const handleDom = React.findDOMNode(this.refs.handle);
-    let showToolTip = false;
-    if (m === handleDom) {
-      showToolTip = true;
-    }
-    this._end('mouse', showToolTip);
+  onMouseUp() {
+    this._end('mouse');
   },
 
   onTouchUp() {
@@ -164,7 +156,7 @@ const Slider = React.createClass({
     const position = this._getTouchPosition(e);
     const value = this._calValueByPos(position);
     this._triggerEvents('onChange', value);
-    this._start(position);
+    this._start(position, value);
     this._addDocumentEvents('touch');
     pauseEvent(e);
   },
@@ -173,7 +165,7 @@ const Slider = React.createClass({
     const position = e.pageX || (e.clientX + document.documentElement.scrollLeft); // to compat ie8
     const value = this._calValueByPos(position);
     this._triggerEvents('onChange', value);
-    this._start(position);
+    this._start(position, value);
     this._addDocumentEvents('mouse');
     pauseEvent(e);
   },
@@ -195,195 +187,70 @@ const Slider = React.createClass({
       return 0;
     }
 
-    return slider.getDOMNode().clientWidth;
+    return slider.clientWidth;
   },
 
   getSliderStart() {
-    const slider = this.refs.slider.getDOMNode();
+    const slider = this.refs.slider;
     const rect = slider.getBoundingClientRect();
 
     return rect.left;
   },
 
-  renderSteps() {
-    const props = this.props;
-    const marksLen = props.marks.length;
-    const stepNum = marksLen > 0 ? marksLen : Math.floor((props.max - props.min) / props.step) + 1;
-    const unit = 100 / (stepNum - 1);
-
-    const prefixCls = props.prefixCls;
-    const stepClassName = prefixClsFn(prefixCls, 'step');
-
-    const elements = [];
-    for (let i = 0; i < stepNum; i++) {
-      const offset = unit * i + '%';
-      const style = {
-        left: offset,
-      };
-      let className = prefixClsFn(prefixCls, 'dot');
-      if (props.isIncluded) {
-        if (i <= this.getIndex()) {
-          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} key={'step' + i}></span>
-      );
-    }
-
-    return (
-      <div className={stepClassName}>
-        {elements}
-      </div>
-    );
-  },
-
-  renderMark(i) {
-    const marks = this.props.marks;
-    const marksLen = marks.length;
-    const unit = 100 / (marksLen - 1);
-    const offset = unit * i;
-
-    const style = {
-      width: unit / 2 + '%',
-    };
-
-    if (i === marksLen - 1) {
-      style.right = -unit / 4 + '%';
-    } else {
-      style.left = i > 0 ? offset - unit / 4 + '%' : -unit / 4 + '%';
-    }
-
-    const prefixCls = this.props.prefixCls;
-    let className = prefixClsFn(prefixCls, 'mark-text');
-
-    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 (
-      <span className={className} style={style} key={i}>{this.props.marks[i]}</span>
-    );
-  },
-
-  renderMarks() {
-    const marks = this.props.marks;
-    const marksLen = marks.length;
-    const elements = [];
-    for (let i = 0; i < marksLen; i++) {
-      elements[i] = this.renderMark(i);
-    }
-
-    const prefixCls = this.props.prefixCls;
-    const className = prefixClsFn(prefixCls, 'mark');
-
-    return (
-      <div className={className}>
-        {elements}
-      </div>
-    );
-  },
-
-  renderHandler(offset) {
-    const onStyle = {
-      left: offset,
-    };
-
-    const prefixCls = this.props.prefixCls;
-    const className = prefixClsFn(prefixCls, 'handle');
-
-    let events = {};
-
-    let tooltipVisible;
-
-    if (this.state.dragging) {
-      tooltipVisible = true;
-    } else {
-      events = {
-        onClick: this.showTooltip.bind(this, true),
-        onMouseEnter: this.showTooltip.bind(this, true),
-        onMouseLeave: this.showTooltip.bind(this, false),
-      };
-      tooltipVisible = this.state.showTooltip;
-    }
-
-    const handle = (<div className={className}
-      {...events}
-      ref="handle"
-      style={onStyle}></div>);
-
-    if (this.props.marks.length > 0) {
-      return handle;
-    }
-    return (
-      <Tooltip
-        placement={{points: ['bc', 'tc']}}
-        visible={tooltipVisible}
-        overlay={<span>{this.state.value}</span>}
-        delay={0}
-        transitionName={this.props.tipTransitionName}
-        prefixCls={prefixClsFn(prefixCls, 'tooltip')}>
-        {handle}
-      </Tooltip>
-    );
-  },
-
-  renderTrack(offset) {
-    const style = {
-      width: offset,
-    };
-
-    const prefixCls = this.props.prefixCls;
-    const trackClassName = prefixClsFn(prefixCls, 'track');
-
-    return (
-      <div className={trackClassName} ref="track" style={style}></div>
-    );
-  },
-
   render() {
     const state = this.state;
+    const {value, dragging} = state;
     const props = this.props;
-    const value = state.value;
+    const {className, prefixCls, disabled, isIncluded, withDots} = props;
+    const {marks, step, max, min, tipTransitionName, children} = props;
+    const marksLen = marks.length;
+
+    const sliderClassName = rcUtil.classSet({
+      [prefixCls]: true,
+      [prefixCls + '-disabled']: disabled,
+      [className]: !!className,
+    });
+
     const offset = this._calcOffset(value);
-    const track = this.props.isIncluded ? this.renderTrack(offset) : null;
-    const ons = this.renderHandler(offset);
-    const steps = (props.step > 1 || props.marks.length > 0) ? this.renderSteps() : null;
-    const sliderMarks = (props.marks.length > 0) ? this.renderMarks() : null;
-    const prefixCls = props.prefixCls;
-    const disabled = props.disabled;
-    const sliderClassName = {
-      [prefixCls]: 1,
-      [props.className]: !!props.className,
-      [`${prefixCls}-disabled`]: disabled,
-    };
+
+    let track = null;
+    if (isIncluded) {
+      const trackClassName = prefixCls + '-track';
+      track = <Track className={trackClassName} offset={0} length={offset - 0} />;
+    }
+
+    const handleClassName = prefixCls + '-handle';
+
+    let steps = null;
+    if (marksLen > 0 || (step > 1 && withDots)) {
+      const stepsClassName = prefixCls + '-step';
+      const stepNum = marksLen > 0 ? marksLen : Math.floor((max - min) / step) + 1;
+      steps = (<Steps className={stepsClassName} stepNum={stepNum}
+                 index={this.getIndex()} isIncluded={isIncluded} />);
+    }
+
+    let mark = null;
+    if (marksLen > 0) {
+      const markClassName = prefixCls + '-mark';
+      mark = (<Marks className={markClassName} marks={marks}
+                 index={this.getIndex()} isIncluded={isIncluded} />);
+    }
 
     return (
-      <div className={rcUtil.classSet(sliderClassName)} ref="slider"
+      <div ref="slider" className={sliderClassName}
         onTouchStart={disabled ? noop : this.onTouchStart}
         onMouseDown={disabled ? noop : this.onSliderMouseDown}>
         {track}
-        {ons}
+        <Handle className={handleClassName} prefixCls={prefixCls}
+          offset={offset} tipTransitionName={tipTransitionName} value={value}
+          dragging={dragging} noTip={marksLen > 0} />
         {steps}
-        {sliderMarks}
-        {this.props.children}
+        {mark}
+        {children}
       </div>
     );
   },
 
-  showTooltip(show) {
-    this.setState({
-      showTooltip: show,
-    });
-  },
-
   _trimAlignValue(v, propsArg) {
     let val = v;
     const props = propsArg || this.props;
@@ -408,7 +275,7 @@ const Slider = React.createClass({
 
   _calcOffset(value) {
     const ratio = (value - this.props.min) / (this.props.max - this.props.min);
-    return ratio * 100 + '%';
+    return ratio * 100;
   },
 
   _calcValue(offset) {
@@ -468,21 +335,20 @@ const Slider = React.createClass({
     }
   },
 
-  _start(position) {
+  _start(position, value) {
     this._triggerEvents('onBeforeChange');
-    this.startValue = this.state.value;
+    this.startValue = value;
     this.startPosition = position;
     this.setState({
       dragging: true,
     });
   },
 
-  _end(type, showToolTip) {
+  _end(type) {
     this._removeEventons(type);
     this._triggerEvents('onAfterChange');
     this.setState({
       dragging: false,
-      showTooltip: !!showToolTip,
     });
   },
 });

src/Steps.jsx

@@ -0,0 +1,27 @@
+import React from 'react';
+import rcUtil from 'rc-util';
+
+const Steps = ({className, stepNum, isIncluded, index}) => {
+  const dotClassName = className.replace('step', 'dot');
+  const unit = 100 / (stepNum - 1);
+
+  const elements = [];
+  for (let i = 0; i < stepNum; i++) {
+    const offset = unit * i + '%';
+    const style = { left: offset };
+
+    const isActived = (isIncluded && i <= index) || (!isIncluded && i === index);
+    const stepClassName = rcUtil.classSet({
+      [dotClassName]: true,
+      [dotClassName + '-active']: isActived,
+    });
+
+    elements.push(<span className={stepClassName} style={style} key={i} />);
+  }
+
+  return (<div className={className}>
+            {elements}
+          </div>);
+};
+
+export default Steps;

src/Track.jsx

@@ -0,0 +1,11 @@
+import React from 'react';
+
+const Track = ({className, offset, length}) => {
+  const style = {
+    left: offset + '%',
+    width: length + '%',
+  };
+  return <div className={className} style={style} />;
+};
+
+export default Track;

tests/index.spec.js

@@ -2,7 +2,8 @@
 var expect = require('expect.js');
 var Slider = require('../index.js');
 var React = require('react');
-var TestUtils = React.addons.TestUtils;
+var ReactDOM = require('react-dom');
+var TestUtils = require('react-addons-test-utils');
 var Simulate = TestUtils.Simulate;
 var $ = require('jquery');
 require('../assets/index.less');
@@ -17,7 +18,7 @@ describe('rc-slider', function () {
   });
 
   it('should render a simple slider with value correctly!', function () {
-    var slider = React.render(
+    var slider = ReactDOM.render(
       <Slider className='rc-slider' defaultValue={40}/>,
       div
     );
@@ -31,7 +32,7 @@ describe('rc-slider', function () {
   });
 
   it('should render a slider with correct numbers of step!', function () {
-    var slider = React.render(
+    var slider = ReactDOM.render(
       <Slider className='rc-slider' step={20}/>,
       div
     );
@@ -39,12 +40,18 @@ describe('rc-slider', function () {
     expect(node.find('.rc-slider').length).to.be(1);
     expect(node.find('.rc-slider-handle').length).to.be(1);
     expect(node.find('.rc-slider-track').length).to.be(1);
-    expect(node.find('.rc-slider-dot').length).to.be(6);
     expect(slider.state.value).to.be(0);
+
+    var sliderWithDots = React.render(
+      <Slider className='rc-slider' step={20} withDots/>,
+      div
+    );
+    var node1 = $(div);
+    expect(node1.find('.rc-slider-dot').length).to.be(6);
   });
 
   it('should render a slider with marks correctly!', function () {
-    var slider = React.render(
+    var slider = ReactDOM.render(
       <Slider marks={["一","二","三","四","五"]} defaultIndex={3} />,
       div
     );
@@ -59,12 +66,12 @@ describe('rc-slider', function () {
   });
 
   // it('should mouseDown works!', function (done) {
-  //   var slider = React.render(
+  //   var slider = ReactDOM.render(
   //     <Slider marks={["一","二","三","四","五"]} />,
   //     div
   //   );
   //   var selectedStep = slider.refs.step3.getDOMNode();
-    
+
   //   Simulate.mouseDown(selectedStep);
 
   //   setTimeout( function() {