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

Check if Thumbnail is still mounted before confirming its image loaded

Browse source
This commit is contained in:
Alex Liebowitz 2017-01-04 15:46:20 -05:00
parent a6b7b3de5a
commit c3e08809c2
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
js/component/common.js
View file

@ -120,6 +120,7 @@ export let Address = React.createClass({
export let Thumbnail = React.createClass({ export let Thumbnail = React.createClass({
_defaultImageUri: '/img/default-thumb.svg', _defaultImageUri: '/img/default-thumb.svg',
_maxLoadTime: 10000, _maxLoadTime: 10000,
_isMounted: false,
propTypes: { propTypes: {
src: React.PropTypes.string.isRequired, src: React.PropTypes.string.isRequired,
@ -137,14 +138,18 @@ export let Thumbnail = React.createClass({
}; };
}, },
componentDidMount: function() { componentDidMount: function() {
this._isMounted = true;
setTimeout(() => { setTimeout(() => {
if (!this.refs.img.complete) { if (this._isMounted && !this.refs.img.complete) {
this.setState({ this.setState({
imageUri: this._defaultImageUri, imageUri: this._defaultImageUri,
}); });
} }
}, this._maxLoadTime); }, this._maxLoadTime);
}, },
componentWillUnmount: function() {
this._isMounted = false;
},
render: function() { render: function() {
return <img ref="img" onError={this.handleError} {... this.props} src={this.state.imageUri} /> return <img ref="img" onError={this.handleError} {... this.props} src={this.state.imageUri} />
}, },