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

Merge pull request #1432 from lbryio/revert-1431-remove-price-placeholder

Browse source 
Revert "Remove price placeholder during thumbnail loading"
This commit is contained in:
Sean Yesmunt 2018-05-03 01:20:46 -04:00 committed by GitHub
parent 5519428c68 1d347d2c6b
commit 54c693767c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 15 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -16,7 +16,6 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/).
### Changed
* Add flair to snackbar ([#1313](https://github.com/lbryio/lbry-app/pull/1313))
* Remove price placeholder during thumbnail loading ([#1431](https://github.com/lbryio/lbry-app/pull/1431))
### Fixed
* Black screen on macOS after maximizing LBRY and then closing ([#1235](https://github.com/lbryio/lbry-app/pull/1235))

18
src/renderer/component/filePrice/view.jsx
View file

@ -16,17 +16,29 @@ class FilePrice extends React.PureComponent<Props> {
showFullPrice: false,
};
render() {
const { claim, costInfo, fetchCostInfo, fetching, showFullPrice, uri } = this.props;
componentWillMount() {
this.fetchCost(this.props);
}
componentWillReceiveProps(nextProps: Props) {
this.fetchCost(nextProps);
}
fetchCost = (props: Props) => {
const { costInfo, fetchCostInfo, uri, fetching, claim } = props;
if (costInfo === undefined && !fetching && claim) {
fetchCostInfo(uri);
}
};
render() {
const { costInfo, showFullPrice } = this.props;
const isEstimate = costInfo ? !costInfo.includesData : false;
if (!costInfo) {
return null;
return <span className="credit-amount">PRICE</span>;
}
return (