Merge pull request #328 from anajavi/react16

Add React 16 peer dependency and fix warning about defaultProps

Author: kirjs

package.json

@@ -24,8 +24,8 @@
     "url": "https://github.com/kirjs/react-highcharts"
   },
   "peerDependencies": {
-    "react": "~0.14 || ^15.0.0",
-    "react-dom": "~0.14 || ^15.0.0"
+    "react": "~0.14 || ^15.0.0 || ^16.0.0",
+    "react-dom": "~0.14 || ^15.0.0 || ^16.0.0"
   },
   "bugs": "https://github.com/kirjs/react-highcharts/issues",
   "keywords": [

src/chartsFactory.jsx

@@ -15,12 +15,15 @@ module.exports = function (chartType, Highcharts){
       callback: PropTypes.func,
       domProps: PropTypes.object
     },
-
-    defaultProps: {
-      callback: () =>{},
-      domProps: {}
+    getDefaultProps: function() {
+      return {
+        callback: () =>{},
+        domProps: {}
+      };
+    },
+    setChartRef: function(chartRef) {
+      this.chartRef = chartRef;
     },
-
     renderChart: function (config){
       if (!config) {
         throw new Error('Config must be specified for the ' + displayName + ' component');
@@ -30,7 +33,7 @@ module.exports = function (chartType, Highcharts){
         ...config,
         chart: {
           ...chartConfig,
-          renderTo: this.refs.chart
+          renderTo: this.chartRef
         }
       }, this.props.callback);
 
@@ -65,7 +68,7 @@ module.exports = function (chartType, Highcharts){
     },
 
     render: function (){
-      return <div ref="chart" {...this.props.domProps} />;
+      return <div ref={this.setChartRef} {...this.props.domProps} />;
     }
   });