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

Merge pull request #449 from hackrush01/price_form2

Browse source 
Using setState in formFieldPrice
This commit is contained in:
Jeremy Kauffman 2017-08-07 18:46:30 -04:00 committed by GitHub
commit 5aa28433bf
2 changed files with 10 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -20,6 +20,7 @@ Web UI version numbers should always match the corresponding version of LBRY App
* Tiles will no longer be blurry on hover (Windows only bug) * Tiles will no longer be blurry on hover (Windows only bug)
* Removed placeholder values from price selection form fields, which was causing confusion that these were real values (#426) * Removed placeholder values from price selection form fields, which was causing confusion that these were real values (#426)
* Fixed showing "other currency" help tip in publish form, which was caused due to not "setting" state for price * Fixed showing "other currency" help tip in publish form, which was caused due to not "setting" state for price
* Now using setState in formFieldPrice
* *
### Deprecated ### Deprecated

16
ui/js/component/formFieldPrice/view.jsx
View file

@ -14,21 +14,23 @@ class FormFieldPrice extends React.PureComponent {
}; };
} }
dispatchChange() { handleChange(newValues) {
const newState = Object.assign({}, this.state, newValues);
this.setState(newState);
this.props.onChange({ this.props.onChange({
amount: this.state.amount, amount: newState.amount,
currency: this.state.currency, currency: newState.currency,
}); });
} }
handleFeeAmountChange(event) { handleFeeAmountChange(event) {
this.state.amount = event.target.value ? Number(event.target.value) : null; this.handleChange({
this.dispatchChange(); amount: event.target.value ? Number(event.target.value) : null,
});
} }
handleFeeCurrencyChange(event) { handleFeeCurrencyChange(event) {
this.state.currency = event.target.value; this.handleChange({ currency: event.target.value });
this.dispatchChange();
} }
render() { render() {