bump 2.2.0

feat: provide value for `onAfterChange` trigger

assets/index.less

@@ -146,7 +146,7 @@
       display: none;
     }
 
-    &-placement-points-bc-tc {
+    &-placement-top {
       padding: @tooltip-arrow-width 0 @tooltip-distance 0;
     }
 
@@ -172,7 +172,7 @@
       border-style: solid;
     }
 
-    &-placement-points-bc-tc &-arrow {
+    &-placement-top &-arrow {
       bottom: @tooltip-distance - @tooltip-arrow-width;
       left: 50%;
       margin-left: -@tooltip-arrow-width;

examples/simple.js

@@ -7,31 +7,29 @@ var ReactDOM = require('react-dom');
 var Slider = require('rc-slider');
 var style = {width:400,margin:50};
 
-function onChange(v) {
+var log = console.log.bind(console);
-  console.log(v);
-}
 
 ReactDOM.render(
   <div>
     <div style={style}>
       <p>基础滑块</p>
-      <Slider onChange={onChange} tipTransitionName='rc-slider-tooltip-zoom-down' />
+      <Slider onChange={log} tipTransitionName='rc-slider-tooltip-zoom-down' />
     </div>
     <div style={style}>
       <p>基础滑块，step=20</p>
       <Slider step={20} />
     </div>
     <div style={style}>
-      <p>基础滑块，step=20 带原点</p>
+      <p>基础滑块，step=20 带圆点</p>
       <Slider dots step={20} />
     </div>
     <div style={style}>
       <p>双滑块</p>
-      <Slider range min={10} max={90} defaultValue={[0, 30]} onChange={onChange} />
+      <Slider range min={10} max={90} defaultValue={[0, 30]} onChange={log} />
     </div>
     <div style={style}>
       <p>双滑块，step=20 </p>
-      <Slider range dots step={20} defaultValue={[0, 30]} onChange={onChange} included={false} />
+      <Slider range dots step={20} defaultValue={[0, 30]} onAfterChange={log} included={false} />
     </div>
     <div style={style}>
       <p>分段式滑块（包含关系）</p>

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rc-slider",
-  "version": "2.0.0",
+  "version": "2.2.0",
   "description": "slider ui component for react",
   "keywords": [
     "react",
@@ -40,18 +40,18 @@
   "devDependencies": {
     "expect.js": "0.3.x",
     "jquery": "^1.11.2",
-    "precommit-hook": "^1.0.7",
+    "pre-commit": "1.x",
     "rc-server": "3.x",
     "rc-tools": "4.x",
     "react": "~0.14.0",
     "react-addons-test-utils": "^0.14.0",
     "react-dom": "~0.14.0"
   },
-  "precommit": [
+  "pre-commit": [
     "lint"
   ],
   "dependencies": {
-    "rc-tooltip": "~3.0.4",
+    "rc-tooltip": "3.x",
-    "rc-util": "2.x"
+    "rc-util": "3.x"
   }
 }

src/Handle.jsx

@@ -40,7 +40,7 @@ export default class Handle extends React.Component {
     const isTooltipVisible = dragging || this.state.isTooltipVisible;
     return (<Tooltip
               prefixCls={className.replace('handle', 'tooltip')}
-              placement={{points: ['bc', 'tc']}}
+              placement="top"
               visible={isTooltipVisible}
               overlay={<span>{value}</span>}
               delay={0}

src/Slider.jsx

@@ -1,11 +1,12 @@
 import React from 'react';
-import rcUtil, {Dom as DomUtils} from 'rc-util';
+import {Dom as DomUtils, classSet} from 'rc-util';
 import Track from './Track';
 import Handle from './Handle';
 import Steps from './Steps';
 import Marks from './Marks';
 
-function noop() {}
+function noop() {
+}
 
 function isNotTouchEvent(e) {
   return e.touches.length > 1 || (e.type.toLowerCase() === 'touchend' && e.touches.length > 0);
@@ -16,18 +17,12 @@ function getTouchPosition(e) {
 }
 
 function getMousePosition(e) {
-  return e.pageX || (e.clientX + document.documentElement.scrollLeft); // to compat ie8
+  return e.pageX;
 }
 
 function pauseEvent(e) {
-  e.cancelBubble = true;
+  e.stopPropagation();
-  e.returnValue = false;
+  e.preventDefault();
-  if (e.stopPropagation) {
-    e.stopPropagation();
-  }
-  if (e.preventDefault) {
-    e.preventDefault();
-  }
 }
 
 class Slider extends React.Component {
@@ -119,21 +114,9 @@ class Slider extends React.Component {
     if (value === oldValue) return;
 
     if (!('value' in props) && !('index' in props)) {
-      this.setState({[state.handle]: value});
+      this.setState({[state.handle]: value}, () => {
-    }
+        this.triggerEvents('onChange', this.getValue());
-
+      });
-    if (props.range) {
-      // `this.state` will not be updated immediately after `this.setState`.
-      // So, create a similar object.
-      // const data = Object.assign({}, state, {[state.handle]: value});
-      const data = {
-        upperBound: state.upperBound,
-        lowerBound: state.lowerBound,
-      };
-      data[state.handle] = value;
-      this.triggerEvents('onChange', [data.lowerBound, data.upperBound]);
-    } else {
-      this.triggerEvents('onChange', value);
     }
   }
 
@@ -184,19 +167,14 @@ class Slider extends React.Component {
       handle: valueNeedChanging,
       recent: valueNeedChanging,
       [valueNeedChanging]: value,
+    }, () => {
+      this.triggerEvents('onChange', this.getValue());
     });
+  }
 
-    if (this.props.range) {
+  getValue() {
-      // const data = Object.assign({}, state, {[valueNeedChanging]: value});
+    const {lowerBound, upperBound} = this.state;
-      const data = {
+    return this.props.range ? [lowerBound, upperBound] : upperBound;
-        upperBound: state.upperBound,
-        lowerBound: state.lowerBound,
-      };
-      data[valueNeedChanging] = value;
-      this.triggerEvents('onChange', [data.lowerBound, data.upperBound]);
-    } else {
-      this.triggerEvents('onChange', value);
-    }
   }
 
   getIndex(value) {
@@ -323,18 +301,18 @@ class Slider extends React.Component {
 
   end(type) {
     this.removeEventons(type);
-    this.triggerEvents('onAfterChange');
+    this.triggerEvents('onAfterChange', this.getValue());
-    this.setState({ handle: null });
+    this.setState({handle: null});
   }
 
   render() {
     const {handle, upperBound, lowerBound} = this.state;
     const props = this.props;
-    const {className, prefixCls, disabled, included, isIncluded, dots, withDots, range} = props;
+    const {className, prefixCls, disabled, included, isIncluded, dots, range} = props;
     const {marks, step, max, min, tipTransitionName, children} = props;
     const marksLen = marks.length;
 
-    const sliderClassName = rcUtil.classSet({
+    const sliderClassName = classSet({
       [prefixCls]: true,
       [prefixCls + '-disabled']: disabled,
       [className]: !!className,
@@ -346,42 +324,42 @@ class Slider extends React.Component {
     let track = null;
     if ((included && isIncluded) || range) {
       const trackClassName = prefixCls + '-track';
-      track = <Track className={trackClassName} offset={lowerOffset} length={upperOffset - lowerOffset} />;
+      track = <Track className={trackClassName} offset={lowerOffset} length={upperOffset - lowerOffset}/>;
     }
 
     const handleClassName = prefixCls + '-handle';
     const isNoTip = marksLen > 0;
     const upper = (<Handle className={handleClassName} tipTransitionName={tipTransitionName} noTip={isNoTip}
-                     offset={upperOffset} value={upperBound} dragging={handle === 'upperBound'} />);
+                           offset={upperOffset} value={upperBound} dragging={handle === 'upperBound'}/>);
 
     let lower = null;
     if (range) {
       lower = (<Handle className={handleClassName} tipTransitionName={tipTransitionName} noTip={isNoTip}
-                 offset={lowerOffset} value={lowerBound} dragging={handle === 'lowerBound'} />);
+                       offset={lowerOffset} value={lowerBound} dragging={handle === 'lowerBound'}/>);
     }
 
     const upperIndex = this.getIndex(upperBound);
 
     let steps = null;
-    if (marksLen > 0 || (step > 1 && (dots || withDots))) {
+    if (marksLen > 0 || (step > 1 && dots)) {
       const stepsClassName = prefixCls + '-step';
       const stepNum = marksLen > 0 ? marksLen : Math.floor((max - min) / step) + 1;
       steps = (<Steps className={stepsClassName} stepNum={stepNum}
-                 lowerIndex={this.getIndex(lowerBound)} upperIndex={upperIndex}
+                      lowerIndex={this.getIndex(lowerBound)} upperIndex={upperIndex}
-                 included={(included && isIncluded) || range} />);
+                      included={(included && isIncluded) || range}/>);
     }
 
     let mark = null;
     if (marksLen > 0) {
       const markClassName = prefixCls + '-mark';
       mark = (<Marks className={markClassName} marks={marks}
-                index={upperIndex} included={(included && isIncluded)} />);
+                     index={upperIndex} included={(included && isIncluded)}/>);
     }
 
     return (
       <div ref="slider" className={sliderClassName}
-        onTouchStart={disabled ? noop : this.onTouchStart.bind(this)}
+           onTouchStart={disabled ? noop : this.onTouchStart.bind(this)}
-        onMouseDown={disabled ? noop : this.onSliderMouseDown.bind(this)}>
+           onMouseDown={disabled ? noop : this.onSliderMouseDown.bind(this)}>
         {track}
         {upper}
         {lower}
@@ -418,7 +396,6 @@ Slider.propTypes = {
   onChange: React.PropTypes.func,
   onAfterChange: React.PropTypes.func,
   tipTransitionName: React.PropTypes.string,
-  withDots: React.PropTypes.bool, // @Deprecated
   dots: React.PropTypes.bool,
   range: React.PropTypes.bool,
 };
@@ -435,7 +412,6 @@ Slider.defaultProps = {
   prefixCls: 'rc-slider',
   disabled: false,
   tipTransitionName: '',
-  withDots: false, // @Deprecated
   dots: false,
   range: false,
 };