Moved theme reset code to componentWillUnmount

Этот коммит содержится в:
Harrison Healey
2016-03-24 15:13:45 -04:00
родитель e5c2915e87
Коммит e02921f344
2 изменённых файлов: 6 добавлений и 21 удалений

Просмотреть файл

@@ -40,7 +40,6 @@ export default class ThemeSetting extends React.Component {
this.onChange = this.onChange.bind(this);
this.submitTheme = this.submitTheme.bind(this);
this.updateTheme = this.updateTheme.bind(this);
this.deactivate = this.deactivate.bind(this);
this.resetFields = this.resetFields.bind(this);
this.handleImportModal = this.handleImportModal.bind(this);
@@ -62,12 +61,17 @@ export default class ThemeSetting extends React.Component {
}
}
componentWillReceiveProps(nextProps) {
if (!this.props.selected && nextProps.selected) {
if (this.props.selected && !nextProps.selected) {
this.resetFields();
}
}
componentWillUnmount() {
UserStore.removeChangeListener(this.onChange);
if (this.props.selected) {
const state = this.getStateFromStores();
Utils.applyTheme(state.theme);
}
}
getStateFromStores() {
const user = UserStore.getCurrentUser();
@@ -147,11 +151,6 @@ export default class ThemeSetting extends React.Component {
updateType(type) {
this.setState({type});
}
deactivate() {
const state = this.getStateFromStores();
Utils.applyTheme(state.theme);
}
resetFields() {
const state = this.getStateFromStores();
state.serverError = null;