Przeglądaj źródła

migrate from other branch

Steven Fukase 4 lat temu
rodzic
commit
ea22210bc0

+ 4 - 18
packages/app/src/components/PageEditor/OptionsSelector.jsx

@@ -34,7 +34,6 @@ class OptionsSelector extends React.Component {
     this.state = {
     this.state = {
       isCddMenuOpened: false,
       isCddMenuOpened: false,
       isMathJaxEnabled,
       isMathJaxEnabled,
-      isTextlintEnabled: false,
     };
     };
 
 
     this.availableThemes = [
     this.availableThemes = [
@@ -59,14 +58,6 @@ class OptionsSelector extends React.Component {
     this.onChangeIndentSize = this.onChangeIndentSize.bind(this);
     this.onChangeIndentSize = this.onChangeIndentSize.bind(this);
   }
   }
 
 
-
-  async componentDidMount() {
-    const { editorContainer } = this.props;
-    const isTextlintEnabled = await editorContainer.retrieveEditorSettings();
-    this.setState({ isTextlintEnabled });
-  }
-
-
   onChangeTheme(newValue) {
   onChangeTheme(newValue) {
     const { editorContainer } = this.props;
     const { editorContainer } = this.props;
 
 
@@ -121,25 +112,21 @@ class OptionsSelector extends React.Component {
 
 
     // save to localStorage
     // save to localStorage
     editorContainer.saveOptsToLocalStorage();
     editorContainer.saveOptsToLocalStorage();
-
-
   }
   }
 
 
   async updateIsTextlintEnabledToDB(newVal) {
   async updateIsTextlintEnabledToDB(newVal) {
     const { appContainer } = this.props;
     const { appContainer } = this.props;
     try {
     try {
-      await appContainer.apiv3Put('/personal-setting/editor-settings', { isTextlintEnabled: newVal });
+      await appContainer.apiv3Put('/personal-setting/editor-settings', { textlintSettings: { isTextlintEnabled: newVal } });
     }
     }
     catch (err) {
     catch (err) {
       toastError(err);
       toastError(err);
     }
     }
-
   }
   }
 
 
   async switchTextlintEnabledHandler() {
   async switchTextlintEnabledHandler() {
     const { editorContainer } = this.props;
     const { editorContainer } = this.props;
-    const newVal = !this.state.isTextlintEnabled;
-    this.setState({ isTextlintEnabled: newVal });
+    const newVal = !editorContainer.state.isTextlintEnabled;
     editorContainer.setState({ isTextlintEnabled: newVal });
     editorContainer.setState({ isTextlintEnabled: newVal });
     this.updateIsTextlintEnabledToDB(newVal);
     this.updateIsTextlintEnabledToDB(newVal);
   }
   }
@@ -321,7 +308,7 @@ class OptionsSelector extends React.Component {
   }
   }
 
 
   renderIsTextlintEnabledMenuItem() {
   renderIsTextlintEnabledMenuItem() {
-    const isActive = this.state.isTextlintEnabled;
+    const isActive = this.props.editorContainer.state.isTextlintEnabled;
 
 
     const iconClasses = ['text-info'];
     const iconClasses = ['text-info'];
     if (isActive) {
     if (isActive) {
@@ -330,7 +317,6 @@ class OptionsSelector extends React.Component {
     const iconClassName = iconClasses.join(' ');
     const iconClassName = iconClasses.join(' ');
 
 
     return (
     return (
-      // eslint-disable-next-line no-console
       <DropdownItem toggle={false} onClick={this.switchTextlintEnabledHandler}>
       <DropdownItem toggle={false} onClick={this.switchTextlintEnabledHandler}>
         <div className="d-flex justify-content-between">
         <div className="d-flex justify-content-between">
           <span className="icon-container"></span>
           <span className="icon-container"></span>
@@ -396,4 +382,4 @@ OptionsSelector.propTypes = {
   editorContainer: PropTypes.instanceOf(EditorContainer).isRequired,
   editorContainer: PropTypes.instanceOf(EditorContainer).isRequired,
 };
 };
 
 
-export default withTranslation()(OptionsSelectorWrapper);
+export default withTranslation()(OptionsSelectorWrapper);