From 3d034f43c01615d5dfa660957bf7bd41becd713d Mon Sep 17 00:00:00 2001 From: Drew Bollinger Date: Tue, 24 Nov 2015 15:41:33 -0500 Subject: [PATCH] Remove ReactDOM.findDOMNode calls from src Per the react docs, this use case is discouraged and can be replaced with the ref itself. It also allows the removal of ReactDOM from the src require statements https://facebook.github.io/react/docs/top-level-api.html#reactdom.finddomnode --- src/map-interactions.react.js | 3 +-- src/map.react.js | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/map-interactions.react.js b/src/map-interactions.react.js index b688cdcd..ee28942e 100644 --- a/src/map-interactions.react.js +++ b/src/map-interactions.react.js @@ -20,7 +20,6 @@ 'use strict'; var React = require('react'); -var ReactDOM = require('react-dom'); var MapboxGL = require('mapbox-gl'); var Point = MapboxGL.Point; var document = require('global/document'); @@ -79,7 +78,7 @@ var MapInteractions = React.createClass({ }, _getMousePos: function _getMousePos(event) { - var el = ReactDOM.findDOMNode(this.refs.container); + var el = this.refs.container; return mousePos(el, event); }, diff --git a/src/map.react.js b/src/map.react.js index 9082871e..bddedcce 100644 --- a/src/map.react.js +++ b/src/map.react.js @@ -21,7 +21,6 @@ var assert = require('assert'); var React = require('react'); -var ReactDOM = require('react-dom'); var debounce = require('debounce'); var r = require('r-dom'); var d3 = require('d3'); @@ -253,7 +252,7 @@ var MapGL = React.createClass({ mapStyle = this.props.mapStyle; } var map = new MapboxGL.Map({ - container: ReactDOM.findDOMNode(this.refs.mapboxMap), + container: this.refs.mapboxMap, center: [this.state.longitude, this.state.latitude], zoom: this.state.zoom, style: mapStyle,