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

fix react warning

Browse source
This commit is contained in:
btzr-io 2017-08-06 21:01:10 -06:00
parent 8d1c3531b7
commit 0873cb76b0
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
ui/js/page/settings/view.jsx
View file

@ -20,6 +20,7 @@ class SettingsPage extends React.PureComponent {
language: lbry.getClientSetting("language"), language: lbry.getClientSetting("language"),
clearingCache: false, clearingCache: false,
theme: lbry.getClientSetting("theme"), theme: lbry.getClientSetting("theme"),
themes: [],
}; };
} }
@ -37,7 +38,8 @@ class SettingsPage extends React.PureComponent {
} }
getThemes() { getThemes() {
return this.props.getThemes().data.themes; const themes = this.props.getThemes().data.themes;
this.setState({ themes });
} }
setDaemonSetting(name, value) { setDaemonSetting(name, value) {
@ -115,6 +117,11 @@ class SettingsPage extends React.PureComponent {
onShowUnavailableChange(event) {} onShowUnavailableChange(event) {}
componentDidMount() {
const { themes } = this.state;
this.getThemes();
}
render() { render() {
const { daemonSettings } = this.props; const { daemonSettings } = this.props;
@ -242,7 +249,7 @@ class SettingsPage extends React.PureComponent {
defaultValue={lbry.getClientSetting("theme")} defaultValue={lbry.getClientSetting("theme")}
className="form-field__input--inline" className="form-field__input--inline"
> >
{this.getThemes().map((name, index) => {this.state.themes.map((name, index) =>
<option key={index} value={name}>{__(`${name} theme`)}</option> <option key={index} value={name}>{__(`${name} theme`)}</option>
)} )}
</FormField> </FormField>