update. Add tipFormat property. See pull #33.

feat: provide value for `onAfterChange` trigger

README.md

@@ -148,6 +148,12 @@ ReactDOM.render(<Rcslider />, container);
           <td>''</td>
           <td>Set the animation for tooltip if it shows.</td>
         </tr>
+        <tr>
+          <td>tipFormatter</td>
+          <td>func</td>
+          <td></td>
+          <td>Format the value of the tooltip if it shows.</td>
+        </tr>
         <tr>
           <td>dots</td>
           <td>bool</td>

assets/index.less

@@ -151,8 +151,8 @@
     }
 
     &-inner {
-      padding: 6px 0;
-      width: 24px;
+      padding: 6px 2px;
+      min-width: 24px;
       height: 24px;
       font-size: 12px;
       line-height: 1;

examples/simple.js

@@ -7,31 +7,33 @@ var ReactDOM = require('react-dom');
 var Slider = require('rc-slider');
 var style = {width:400,margin:50};
 
-function onChange(v) {
-  console.log(v);
+var log = console.log.bind(console);
+
+function percentFormatter(v) {
+  return 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>
@@ -41,5 +43,9 @@ ReactDOM.render(
       <p>分段式滑块（并列关系）</p>
       <Slider marks={["状态1","状态2","状态3","状态4"]} included={false} defaultIndex={1} />
     </div>
+    <div style={style}>
+      <p>基础滑块</p>
+      <Slider onChange={log} tipFormatter={percentFormatter} tipTransitionName='rc-slider-tooltip-zoom-down' />
+    </div>
   </div>
   , document.getElementById('__react-content'));

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rc-slider",
-  "version": "2.1.0",
+  "version": "2.2.1",
   "description": "slider ui component for react",
   "keywords": [
     "react",

src/Handle.jsx

@@ -24,7 +24,7 @@ export default class Handle extends React.Component {
 
   render() {
     const props = this.props;
-    const {className, tipTransitionName, offset, value} = props;
+    const {className, tipTransitionName, tipFormatter, offset, value} = props;
     const {dragging, noTip} = props;
 
     const style = { left: offset + '%' };
@@ -42,7 +42,7 @@ export default class Handle extends React.Component {
               prefixCls={className.replace('handle', 'tooltip')}
               placement="top"
               visible={isTooltipVisible}
-              overlay={<span>{value}</span>}
+              overlay={<span>{tipFormatter ? tipFormatter(value) : value}</span>}
               delay={0}
               transitionName={tipTransitionName}>
               {handle}
@@ -54,6 +54,7 @@ Handle.propTypes = {
   className: React.PropTypes.string,
   offset: React.PropTypes.number,
   tipTransitionName: React.PropTypes.string,
+  tipFormatter: React.PropTypes.func,
   value: React.PropTypes.number,
   dragging: React.PropTypes.bool,
   noTip: React.PropTypes.bool,

src/Slider.jsx

@@ -114,21 +114,9 @@ class Slider extends React.Component {
     if (value === oldValue) return;
 
     if (!('value' in props) && !('index' in props)) {
-      this.setState({[state.handle]: value});
-    }
-
-    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);
+      this.setState({[state.handle]: value}, () => {
+        this.triggerEvents('onChange', this.getValue());
+      });
     }
   }
 
@@ -179,19 +167,14 @@ class Slider extends React.Component {
       handle: valueNeedChanging,
       recent: valueNeedChanging,
       [valueNeedChanging]: value,
+    }, () => {
+      this.triggerEvents('onChange', this.getValue());
     });
+  }
 
-    if (this.props.range) {
-      // const data = Object.assign({}, state, {[valueNeedChanging]: value});
-      const data = {
-        upperBound: state.upperBound,
-        lowerBound: state.lowerBound,
-      };
-      data[valueNeedChanging] = value;
-      this.triggerEvents('onChange', [data.lowerBound, data.upperBound]);
-    } else {
-      this.triggerEvents('onChange', value);
-    }
+  getValue() {
+    const {lowerBound, upperBound} = this.state;
+    return this.props.range ? [lowerBound, upperBound] : upperBound;
   }
 
   getIndex(value) {
@@ -318,15 +301,15 @@ class Slider extends React.Component {
 
   end(type) {
     this.removeEventons(type);
-    this.triggerEvents('onAfterChange');
+    this.triggerEvents('onAfterChange', this.getValue());
     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 {marks, step, max, min, tipTransitionName, children} = props;
+    const {className, prefixCls, disabled, included, isIncluded, dots, range} = props;
+    const {marks, step, max, min, tipTransitionName, tipFormatter, children} = props;
     const marksLen = marks.length;
 
     const sliderClassName = classSet({
@@ -345,20 +328,20 @@ class Slider extends React.Component {
     }
 
     const handleClassName = prefixCls + '-handle';
-    const isNoTip = marksLen > 0;
-    const upper = (<Handle className={handleClassName} tipTransitionName={tipTransitionName} noTip={isNoTip}
-                           offset={upperOffset} value={upperBound} dragging={handle === 'upperBound'}/>);
+    const isNoTip = (marksLen > 0) && !tipFormatter;
+    const upper = (<Handle className={handleClassName} tipTransitionName={tipTransitionName} noTip={isNoTip} tipFormatter={tipFormatter}
+                     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'}/>);
+      lower = (<Handle className={handleClassName} tipTransitionName={tipTransitionName} noTip={isNoTip} tipFormatter={tipFormatter}
+                 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}
@@ -413,7 +396,7 @@ Slider.propTypes = {
   onChange: React.PropTypes.func,
   onAfterChange: React.PropTypes.func,
   tipTransitionName: React.PropTypes.string,
-  withDots: React.PropTypes.bool, // @Deprecated
+  tipFormatter: React.PropTypes.func,
   dots: React.PropTypes.bool,
   range: React.PropTypes.bool,
 };
@@ -430,7 +413,6 @@ Slider.defaultProps = {
   prefixCls: 'rc-slider',
   disabled: false,
   tipTransitionName: '',
-  withDots: false, // @Deprecated
   dots: false,
   range: false,
 };