Updated dependencies and support for react-dnd@6

Author: dolezel

.babelrc

@@ -0,0 +1,9 @@
+module.exports = {
+    presets: [
+        '@babel/preset-env',
+        '@babel/preset-react',
+    ],
+    plugins: [
+        '@babel/plugin-proposal-class-properties',
+    ],
+}

.babelrc.js

@@ -0,0 +1,15 @@
+module.exports = {
+    parser: 'babel-eslint',
+    extends: 'airbnb',
+
+    rules: {
+        'react/jsx-filename-extension': 0,
+        'no-unused-vars': 0,
+        'react/prop-types': 0,
+    },
+
+    env: {
+        browser: true,
+        mocha: true
+    },
+}

.eslintrc

@@ -38,30 +38,35 @@
   },
   "license": "MIT",
   "dependencies": {
-    "hoist-non-react-statics": "^1.2.0",
+    "hoist-non-react-statics": "^3.1.0",
     "lodash.throttle": "^4.0.1",
     "prop-types": "^15.5.9",
     "raf": "^3.2.0",
-    "react-display-name": "^0.2.0"
+    "react": "^16.3.0",
+    "react-display-name": "^0.2.0",
+    "react-dom": "^16.3.0"
   },
   "devDependencies": {
-    "babel-cli": "^6.4.5",
-    "babel-eslint": "^6.0.4",
-    "babel-preset-es2015": "^6.3.13",
-    "babel-preset-react": "^6.5.0",
-    "babel-preset-stage-1": "^6.3.13",
-    "babel-register": "^6.4.3",
-    "chai": "^3.4.1",
-    "eslint": "^2.12.0",
-    "eslint-config-airbnb": "^9.0.1",
-    "eslint-plugin-import": "^1.8.1",
-    "eslint-plugin-jsx-a11y": "^1.4.2",
-    "eslint-plugin-react": "^5.1.1",
+    "@babel/cli": "^7.0.0",
+    "@babel/core": "^7.0.0",
+    "@babel/plugin-proposal-class-properties": "^7.0.0",
+    "@babel/preset-env": "^7.0.0",
+    "@babel/preset-react": "^7.0.0",
+    "@babel/register": "^7.0.0",
+    "babel-eslint": "^10.0.1",
+    "chai": "^4.2.0",
+    "eslint": "^5.9.0",
+    "eslint-config-airbnb": "^17.1.0",
+    "eslint-plugin-import": "^2.14.0",
+    "eslint-plugin-jsx-a11y": "^6.1.2",
+    "eslint-plugin-react": "^7.11.1",
     "in-publish": "^2.0.0",
-    "mocha": "^2.3.4",
-    "react": "^15.1.0",
-    "react-dom": "^15.1.0",
-    "sinon": "^1.17.2",
-    "sinon-chai": "^2.8.0"
+    "mocha": "^5.2.0",
+    "react-dnd": "^6.0.0",
+    "sinon": "^7.1.1",
+    "sinon-chai": "^3.2.0"
+  },
+  "peerDependencies": {
+    "react-dnd": "^6.0.0"
   }
 }

.eslintrc.js

@@ -4,6 +4,7 @@ import { findDOMNode } from 'react-dom';
 import throttle from 'lodash.throttle';
 import raf from 'raf';
 import getDisplayName from 'react-display-name';
+import { Consumer as DragDropContextConsumer } from 'react-dnd/lib/DragDropContext';
 import hoist from 'hoist-non-react-statics';
 import { noop, intBetween, getCoords } from './util';
 
@@ -18,7 +19,8 @@ export function createHorizontalStrength(_buffer) {
     if (inBox) {
       if (point.x < x + buffer) {
         return (point.x - x - buffer) / buffer;
-      } else if (point.x > (x + w - buffer)) {
+      }
+      if (point.x > (x + w - buffer)) {
         return -(x + w - point.x - buffer) / buffer;
       }
     }
@@ -36,7 +38,8 @@ export function createVerticalStrength(_buffer) {
     if (inBox) {
       if (point.y < y + buffer) {
         return (point.y - y - buffer) / buffer;
-      } else if (point.y > (y + h - buffer)) {
+      }
+      if (point.y > (y + h - buffer)) {
         return -(y + h - point.y - buffer) / buffer;
       }
     }
@@ -50,12 +53,13 @@ export const defaultHorizontalStrength = createHorizontalStrength(DEFAULT_BUFFER
 export const defaultVerticalStrength = createVerticalStrength(DEFAULT_BUFFER);
 
 
-export default function createScrollingComponent(WrappedComponent) {
+export function createScrollingComponent(WrappedComponent) {
   class ScrollingComponent extends Component {
-
     static displayName = `Scrolling(${getDisplayName(WrappedComponent)})`;
 
     static propTypes = {
+      // eslint-disable-next-line react/forbid-prop-types
+      dragDropManager: PropTypes.object.isRequired,
       onScrollChange: PropTypes.func,
       verticalStrength: PropTypes.func,
       horizontalStrength: PropTypes.func,
@@ -69,13 +73,33 @@ export default function createScrollingComponent(WrappedComponent) {
       strengthMultiplier: 30,
     };
 
-    static contextTypes = {
-      dragDropManager: PropTypes.object,
-    };
+    // Update scaleX and scaleY every 100ms or so
+    // and start scrolling if necessary
+    updateScrolling = throttle((evt) => {
+      const {
+        left: x, top: y, width: w, height: h,
+      } = this.container.getBoundingClientRect();
+      const box = {
+        x, y, w, h,
+      };
+      const coords = getCoords(evt);
+
+      // calculate strength
+      const { horizontalStrength, verticalStrength } = this.props;
+      this.scaleX = horizontalStrength(box, coords);
+      this.scaleY = verticalStrength(box, coords);
+
+      // start scrolling if we need to
+      if (!this.frame && (this.scaleX || this.scaleY)) {
+        this.startScrolling();
+      }
+    }, 100, { trailing: false })
 
     constructor(props, ctx) {
       super(props, ctx);
 
+      this.wrappedInstance = React.createRef();
+
       this.scaleX = 0;
       this.scaleY = 0;
       this.frame = null;
@@ -85,16 +109,17 @@ export default function createScrollingComponent(WrappedComponent) {
     }
 
     componentDidMount() {
-      this.container = findDOMNode(this.wrappedInstance);
+      // eslint-disable-next-line react/no-find-dom-node
+      this.container = findDOMNode(this.wrappedInstance.current);
       this.container.addEventListener('dragover', this.handleEvent);
       // touchmove events don't seem to work across siblings, so we unfortunately
       // have to attach the listeners to the body
       window.document.body.addEventListener('touchmove', this.handleEvent);
 
-      this.clearMonitorSubscription = this.context
-          .dragDropManager
-          .getMonitor()
-          .subscribeToStateChange(() => this.handleMonitorChange());
+      const { dragDropManager } = this.props;
+      this.clearMonitorSubscription = dragDropManager
+        .getMonitor()
+        .subscribeToStateChange(() => this.handleMonitorChange());
     }
 
     componentWillUnmount() {
@@ -112,7 +137,8 @@ export default function createScrollingComponent(WrappedComponent) {
     }
 
     handleMonitorChange() {
-      const isDragging = this.context.dragDropManager.getMonitor().isDragging();
+      const { dragDropManager } = this.props;
+      const isDragging = dragDropManager.getMonitor().isDragging();
 
       if (!this.dragging && isDragging) {
         this.dragging = true;
@@ -134,23 +160,6 @@ export default function createScrollingComponent(WrappedComponent) {
       window.document.body.removeEventListener('touchmove', this.updateScrolling);
     }
 
-    // Update scaleX and scaleY every 100ms or so
-    // and start scrolling if necessary
-    updateScrolling = throttle(evt => {
-      const { left: x, top: y, width: w, height: h } = this.container.getBoundingClientRect();
-      const box = { x, y, w, h };
-      const coords = getCoords(evt);
-
-      // calculate strength
-      this.scaleX = this.props.horizontalStrength(box, coords);
-      this.scaleY = this.props.verticalStrength(box, coords);
-
-      // start scrolling if we need to
-      if (!this.frame && (this.scaleX || this.scaleY)) {
-        this.startScrolling();
-      }
-    }, 100, { trailing: false })
-
     startScrolling() {
       let i = 0;
       const tick = () => {
@@ -167,7 +176,8 @@ export default function createScrollingComponent(WrappedComponent) {
         // mousemove events from a container that also emits a scroll
         // event that same frame. So we double the strengthMultiplier and only adjust
         // the scroll position at 30fps
-        if (i++ % 2) {
+        i += 1;
+        if (i % 2) {
           const {
             scrollLeft,
             scrollTop,
@@ -181,15 +191,15 @@ export default function createScrollingComponent(WrappedComponent) {
             ? container.scrollLeft = intBetween(
               0,
               scrollWidth - clientWidth,
-              scrollLeft + scaleX * strengthMultiplier
+              scrollLeft + scaleX * strengthMultiplier,
             )
             : scrollLeft;
 
           const newTop = scaleY
             ? container.scrollTop = intBetween(
               0,
               scrollHeight - clientHeight,
-              scrollTop + scaleY * strengthMultiplier
+              scrollTop + scaleY * strengthMultiplier,
             )
             : scrollTop;
 
@@ -220,12 +230,12 @@ export default function createScrollingComponent(WrappedComponent) {
         horizontalStrength,
         onScrollChange,
 
-        ...props,
+        ...props
       } = this.props;
 
       return (
         <WrappedComponent
-          ref={(ref) => { this.wrappedInstance = ref; }}
+          ref={this.wrappedInstance}
           {...props}
         />
       );
@@ -234,3 +244,16 @@ export default function createScrollingComponent(WrappedComponent) {
 
   return hoist(ScrollingComponent, WrappedComponent);
 }
+
+export default function createScrollingComponentWithConsumer(WrappedComponent) {
+  const ScrollingComponent = createScrollingComponent(WrappedComponent);
+  return props => (
+    <DragDropContextConsumer>
+      {({ dragDropManager }) => (
+        dragDropManager === undefined
+          ? null
+          : <ScrollingComponent {...props} dragDropManager={dragDropManager} />
+      )}
+    </DragDropContextConsumer>
+  );
+}

package.json

@@ -5,7 +5,7 @@ export function noop() {
 
 export function intBetween(min, max, val) {
   return Math.floor(
-    Math.min(max, Math.max(min, val))
+    Math.min(max, Math.max(min, val)),
   );
 }
 

src/index.js

@@ -1,4 +1,4 @@
---require babel-register
+--require @babel/register
 --require test/setup
 --check-leaks
 --throw-deprecation