Replace Hidden with React.Fragment

whawker · Sep 27, 2019 · 9d6664c · 9d6664c
1 parent 6d8191e
commit 9d6664c
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 25 deletions.
diff --git a/packages/react-jsx-highcharts/src/components/Debug/Debug.js b/packages/react-jsx-highcharts/src/components/Debug/Debug.js
@@ -1,6 +1,5 @@
 import React, { useEffect } from 'react';
 import PropTypes from 'prop-types';
-import Hidden from '../Hidden';
 import useChart from '../UseChart';
 
 const Debug = ({ varName = 'chart' }) => {
@@ -13,9 +12,7 @@ const Debug = ({ varName = 'chart' }) => {
     return () => window[varName] = undefined;
   }, [varName]);
 
-  return (
-    <Hidden />
-  );
+  return null;
 }
 
 Debug.propTypes = {

diff --git a/packages/react-jsx-highcharts/src/components/Hidden/Hidden.js b/packages/react-jsx-highcharts/src/components/Hidden/Hidden.js
diff --git a/packages/react-jsx-highcharts/src/components/Hidden/index.js b/packages/react-jsx-highcharts/src/components/Hidden/index.js
diff --git a/packages/react-jsx-highcharts/src/components/HighchartsSparkline/HighchartsSparkline.js b/packages/react-jsx-highcharts/src/components/HighchartsSparkline/HighchartsSparkline.js
@@ -4,7 +4,6 @@ import HighchartsChart from '../HighchartsChart';
 import Chart from '../Chart';
 import XAxis from '../XAxis';
 import YAxis from '../YAxis';
-import Hidden from '../Hidden';
 
 const defaultSparklinePlotOptions = {
   series: {
@@ -81,7 +80,7 @@ const HighchartsSparkline = ({
         {Series}
       </YAxis>
 
-      {hasSeriesProp && <Hidden>{children}</Hidden>}
+      {hasSeriesProp && <>{children}</>}
     </HighchartsChart>
   );
 };

diff --git a/packages/react-jsx-highcharts/src/index.js b/packages/react-jsx-highcharts/src/index.js
@@ -95,7 +95,6 @@ export { default as withHighcharts }  from './components/WithHighcharts';
 export { default as Axis }  from './components/Axis';
 export { default as BaseChart }  from './components/BaseChart';
 export { default as Debug }  from './components/Debug';
-export { default as Hidden }  from './components/Hidden';
 export { default as Series }  from './components/Series';
 
 // Utils

diff --git a/packages/react-jsx-highmaps/src/components/MapNavigation/MapNavigation.js b/packages/react-jsx-highmaps/src/components/MapNavigation/MapNavigation.js
@@ -2,7 +2,6 @@ import React, { useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
 import { attempt } from 'lodash-es';
 import {
-  Hidden,
   useModifiedProps,
   useChart,
   useHighcharts
@@ -40,7 +39,7 @@ const MapNavigation = ({ children, enabled = true, ...restProps}) => {
 
   if (!children || !rendered) return null;
 
-  return <Hidden>{children}</Hidden>;
+  return <>{children}</>;
 };
 const getMapNavigationConfig = (props, Highcharts) => {
   return {

diff --git a/packages/react-jsx-highstock/src/components/Navigator/NavigatorAxis.js b/packages/react-jsx-highstock/src/components/Navigator/NavigatorAxis.js
@@ -1,6 +1,6 @@
 import React, { useEffect, Children, cloneElement, isValidElement } from 'react';
 import PropTypes from 'prop-types';
-import { Hidden, useAxis, useModifiedProps, getNonEventHandlerProps } from 'react-jsx-highcharts';
+import { useAxis, useModifiedProps, getNonEventHandlerProps } from 'react-jsx-highcharts';
 
 const NavigatorAxis = props => {
   const axis = useAxis();
@@ -32,9 +32,9 @@ const NavigatorAxis = props => {
   });
 
   return (
-    <Hidden>
+    <>
       {axisChildren}
-    </Hidden>
+    </>
   );
 }
 const updateNavigatorAxis = (config, axis) => {

diff --git a/packages/react-jsx-highstock/src/components/RangeSelector/RangeSelectorInner.js b/packages/react-jsx-highstock/src/components/RangeSelector/RangeSelectorInner.js
@@ -1,7 +1,7 @@
 import React, { useState, useEffect } from 'react';
 import PropTypes from 'prop-types';
 import { attempt } from 'lodash-es';
-import { Hidden, useHighcharts, useAxis, useChart, useModifiedProps } from 'react-jsx-highcharts';
+import { useHighcharts, useAxis, useChart, useModifiedProps } from 'react-jsx-highcharts';
 
 const RangeSelectorInner = ({ enabled = true, ...restProps }) => {
   const props = { enabled, ...restProps };
@@ -51,7 +51,7 @@ const RangeSelectorInner = ({ enabled = true, ...restProps }) => {
   if (!children || !rendered) return null;
 
   return (
-    <Hidden>{children}</Hidden>
+    <>{children}</>
   );
 }
 

diff --git a/packages/react-jsx-highstock/src/components/Scrollbar/Scrollbar.js b/packages/react-jsx-highstock/src/components/Scrollbar/Scrollbar.js
@@ -1,7 +1,7 @@
 import React, { useEffect } from 'react';
 import PropTypes from 'prop-types';
 import { attempt } from 'lodash-es';
-import { Hidden, useModifiedProps, useChart } from 'react-jsx-highcharts';
+import { useModifiedProps, useChart } from 'react-jsx-highcharts';
 
 const Scrollbar = ({ children, enabled = true, ...restProps }) => {
   const chart = useChart();
@@ -23,7 +23,7 @@ const Scrollbar = ({ children, enabled = true, ...restProps }) => {
   if (!children) return null;
 
   return (
-    <Hidden>{children}</Hidden>
+    <>{children}</>
   );
 }