mastofe

commit: 5521e94e24bc0ce4258d612ea074334b2af85e54
parent: 763a2f8511dcdb14b4a665bba2c6e2880b6d6b68
Author: Sorin Davidoi <sorin.davidoi@gmail.com>
Date:   Wed,  1 Nov 2017 12:17:53 +0100

refactor(features/ui): Avoid useless renders in WrappedSwitch (#5580)


Diffstat:
M
app/javascript/mastodon/features/ui/util/react_router_helpers.js
18
+++++++++++++-----

1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/app/javascript/mastodon/features/ui/util/react_router_helpers.js b/app/javascript/mastodon/features/ui/util/react_router_helpers.js
@@ -7,11 +7,19 @@ import BundleColumnError from '../components/bundle_column_error';
 import BundleContainer from '../containers/bundle_container';
 
 // Small wrapper to pass multiColumn to the route components
-export const WrappedSwitch = ({ multiColumn, children }) => (
-  <Switch>
-    {React.Children.map(children, child => React.cloneElement(child, { multiColumn }))}
-  </Switch>
-);
+export class WrappedSwitch extends React.PureComponent {
+
+  render () {
+    const { multiColumn, children } = this.props;
+
+    return (
+      <Switch>
+        {React.Children.map(children, child => React.cloneElement(child, { multiColumn }))}
+      </Switch>
+    );
+  }
+
+}
 
 WrappedSwitch.propTypes = {
   multiColumn: PropTypes.bool,