Created
April 6, 2017 20:08
-
-
Save sieverk/b8815bf0b64e1fc0ef0aac298b7298f4 to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
import hoistStatics from 'hoist-non-react-statics'; | |
import React from 'react'; | |
/** | |
* Allows two animation frames to complete to allow other components to update | |
* and re-render before mounting and rendering an expensive `WrappedComponent`. | |
*/ | |
export default function deferComponentRender(WrappedComponent) { | |
class DeferredRenderWrapper extends React.Component { | |
constructor(props, context) { | |
super(props, context); | |
this.state = { shouldRender: true }; | |
} | |
componentDidMount() { | |
window.requestAnimationFrame(() => { | |
window.requestAnimationFrame(() => this.setState({ shouldRender: true })); | |
}); | |
} | |
render() { | |
return this.state.shouldRender ? <WrappedComponent {...this.props} /> : null; | |
} | |
} | |
return hoistStatics(DeferredRenderWrapper, WrappedComponent); | |
} |
Cool! I actually had the same question as @thomasdavis! Also, do you mind sharing the reasoning behind deferring using 2 calls to requestAninationFrame? Thanks!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@sieverk should #12 be
shouldRender:false
?