Wire up & test disabled prop.

Author: STRML

lib/Draggable.es6

@@ -64,25 +64,6 @@ export default class Draggable extends DraggableCore {
       PropTypes.oneOf(['parent', false])
     ]),
 
-    /**
-     * `disabled`, if true, stops the <Draggable> from dragging. It will stay where it is.
-     *
-     * Example:
-     *
-     * ```jsx
-     *   let App = React.createClass({
-     *       render: function () {
-     *           return (
-     *               <Draggable disabled={true}>
-     *                   <div>I can't be dragged</div>
-     *               </Draggable>
-     *           );
-     *       }
-     *   });
-     * ```
-     */
-    disabled: PropTypes.bool,
-
     /**
      * `grid` specifies the x and y that dragging should snap to.
      *
@@ -154,7 +135,6 @@ export default class Draggable extends DraggableCore {
   static defaultProps = assign({}, DraggableCore.defaultProps, {
     axis: 'both',
     bounds: false,
-    disabled: false,
     grid: null,
     start: {x: 0, y: 0},
     zIndex: NaN

lib/DraggableCore.es6

@@ -34,6 +34,26 @@ export default class DraggableCore extends React.Component {
   static displayName = 'DraggableCore';
 
   static propTypes = {
+    /**
+     * `disabled`, if true, stops the <Draggable> from dragging. All handlers,
+     * with the exception of `onMouseDown`, will not fire.
+     *
+     * Example:
+     *
+     * ```jsx
+     *   let App = React.createClass({
+     *       render: function () {
+     *           return (
+     *               <Draggable disabled={true}>
+     *                   <div>I can't be dragged</div>
+     *               </Draggable>
+     *           );
+     *       }
+     *   });
+     * ```
+     */
+    disabled: PropTypes.bool,
+
     /**
      * By default, we add 'user-select:none' attributes to the document body
      * to prevent ugly text selection during drag. If this is causing problems
@@ -162,9 +182,10 @@ export default class DraggableCore extends React.Component {
   };
 
   static defaultProps = {
-    handle: null,
     cancel: null,
+    disabled: false,
     enableUserSelectHack: true,
+    handle: null,
     transform: null,
     onStart: function(){},
     onDrag: function(){},
@@ -193,7 +214,8 @@ export default class DraggableCore extends React.Component {
     this.props.onMouseDown(e);
 
     // Short circuit if handle or cancel prop was provided and selector doesn't match.
-    if ((this.props.handle && !matchesSelector(e.target, this.props.handle)) ||
+    if (this.props.disabled ||
+      (this.props.handle && !matchesSelector(e.target, this.props.handle)) ||
       (this.props.cancel && matchesSelector(e.target, this.props.cancel))) {
       return;
     }

specs/draggable.spec.jsx

@@ -151,6 +151,18 @@ describe('react-draggable', function () {
       expect(called).toEqual(true);
     });
 
+    it('should not call onStart when dragging begins and disabled', function () {
+      var called = false;
+      drag = TestUtils.renderIntoDocument(
+        <Draggable onStart={function () { called = true; }} disabled={true}>
+          <div/>
+        </Draggable>
+      );
+
+      TestUtils.Simulate.mouseDown(ReactDOM.findDOMNode(drag));
+      expect(called).toEqual(false);
+    });
+
     it('should render with style translate() for DOM nodes', function () {
       var dragged = false;
       drag = TestUtils.renderIntoDocument(