LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Packages Projects Releases 238 Wiki Activity

Merge pull request #1804 from lbryio/subscriptions-loader-fix

Browse source 
Fix loading indicator on Page component
This commit is contained in:
Sean Yesmunt 2018-07-23 18:46:42 -04:00 committed by GitHub
parent 8ee815a84b bf8ada27d7
commit ee5922a8ba
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/renderer/component/page/view.jsx
View file

@ -39,10 +39,16 @@ class Page extends React.PureComponent<Props, State> {
componentDidUpdate(prevProps: Props) {
const { loading } = this.props;
const { showLoader } = this.state;
if (!this.loaderTimeout && !prevProps.loading && loading) {
this.beginLoadingTimeout();
} else if (!loading && this.loaderTimeout) {
clearTimeout(this.loaderTimeout);
if (showLoader) {
this.removeLoader();
}
}
}
@ -58,6 +64,10 @@ class Page extends React.PureComponent<Props, State> {
}, LOADER_TIMEOUT);
}
removeLoader() {
this.setState({ showLoader: false });
}
loaderTimeout: ?TimeoutID;
render() {