React DnD: Avoid using findDOMNode React DnD: Avoid using findDOMNode reactjs reactjs

React DnD: Avoid using findDOMNode


Assuming you're using es6 class syntax and the most recent version of React (15, at time of writing), you can attach a callback ref like Dan did in his example on the link you shared. From the docs:

When the ref attribute is used on an HTML element, the ref callback receives the underlying DOM element as its argument. For example, this code uses the ref callback to store a reference to a DOM node:

<h3    className="widget"    onMouseOver={ this.handleHover.bind( this ) }    ref={node => this.node = node}>

Then you can access the node just like we used to do with our old friends findDOMNode() or getDOMNode():

handleHover() {  const rect = this.node.getBoundingClientRect(); // Your DOM node  this.setState({ rect });}

In action:https://jsfiddle.net/ftub8ro6/

Edit:

Because React DND does a bit of magic behind the scenes, we have to use their API to get at the decorated component. They provide getDecoratedComponentInstance() so you can get at the underlying component. Once you use that, you can get the component.node as expected:

hover(props, monitor, component) {    const dragIndex = monitor.getItem().index;    const hoverIndex = props.index;    const rawComponent = component.getDecoratedComponentInstance();    console.log( rawComponent.node.getBoundingClientRect() );    ...

Here it is in action:

https://jsfiddle.net/h4w4btz9/2/


Better Solution

A better solution is to just wrap your draggable component with a div, define a ref on that and pass it to the draggable component, i.e.

<div key={key} ref={node => { this.node = node; }}>  <MyComponent    node={this.node}  /></div>

and MyComponent is wrapped in DragSource. Now you can just use

hover(props, monitor, component) {  ...  props.node && props.node.getBoundingClientRect();  ...}

(props.node && is just added to avoid to call getBoundingClientRect on an undefined object)

Alternative for findDOMNode

If you don't want to add a wrapping div, you could do the following.The reply of @imjared and the suggested solution here don't work (at least in react-dnd@2.3.0 and react@15.3.1).

The only working alternative for findDOMNode(component).getBoundingClientRect(); which does not use findDOMNode is:

hover(props, monitor, component) {  ...  component.decoratedComponentInstance._reactInternalInstance._renderedComponent._hostNode.getBoundingClientRect();  ...}

which is not very beautiful and dangerous because react could change this internal path in future versions!

Other (weaker) Alternative

Use monitor.getDifferenceFromInitialOffset(); which will not give you precise values, but is perhaps good enough in case you have a small dragSource. Then the returned value is pretty predictable with a small error margin depending on the size of your dragSource.


React-DnD's API is super flexible—we can (ab)use this.

For example, React-DnD lets us determine what connectors are passed to the underlying component. Which means we can wrap them, too. :)

For example, let's override the target connector to store the node on the monitor. We will use a Symbol so we do not leak this little hack to the outside world.

const NODE = Symbol('Node')function targetCollector(connect, monitor) {  const connectDropTarget = connect.dropTarget()  return {    // Consumer does not have to know what we're doing ;)    connectDropTarget: node => {      monitor[NODE] = node      connectDropTarget(node)    }  }}

Now in your hover method, you can use

const node = monitor[NODE]const hoverBoundingRect = node.getBoundingClientRect()

This approach piggybacks on React-DnD's flow and shields the outside world by using a Symbol.

Whether you're using this approach or the class-based this.node = node ref approach, you're relying on the underlying React node. I prefer this one because the consumer does not have to remember to manually use a ref other than the ones already required by React-DnD, and the consumer does not have to be a class component either.