fix experiemnts

Author: SaraVieira

packages/app/src/app/overmind/namespaces/preferences/actions.ts

@@ -131,30 +131,3 @@ export const zenModeToggled: Action = ({ state }) => {
 export const codeMirrorForced: Action = ({ state }) => {
   state.preferences.settings.codeMirror = true;
 };
-
-export const toggleContainerLspExperiment: AsyncAction = async ({
-  state,
-  effects,
-}) => {
-  if (!state.user) {
-    return;
-  }
-  try {
-    await effects.api.updateExperiments({
-      container_lsp: !state.user.experiments.containerLsp,
-    });
-    state.user.experiments.containerLsp = !state.user.experiments.containerLsp;
-    // Allow the flush to go through and flip button
-    requestAnimationFrame(() => {
-      if (
-        effects.browser.confirm(
-          'We need to refresh for this to take effect, or you can refresh later'
-        )
-      ) {
-        effects.browser.reload();
-      }
-    });
-  } catch (error) {
-    effects.notificationToast.error('Unable to toggl LSP experiment');
-  }
-};

packages/app/src/app/pages/common/Modals/PreferencesModal/Experiments/ContainerLSP.tsx

@@ -0,0 +1,38 @@
+import React, { useState, useEffect } from 'react';
+import { useOvermind } from 'app/overmind';
+import { SubDescription, PaddedPreference } from '../elements';
+
+export const ContainerLSP: React.FunctionComponent = () => {
+  const { state } = useOvermind();
+  const [containerLSP, setContainerLSP] = useState(false);
+  useEffect(() => {
+    const value = window.localStorage.getItem('CONTAINER_LSP');
+
+    if (value === 'true') {
+      return setContainerLSP(true);
+    }
+    return setContainerLSP(false);
+  }, []);
+
+  const setValue = (val: boolean) => {
+    setContainerLSP(val);
+    window.localStorage.setItem('CONTAINER_LSP', val.toString());
+    location.reload();
+  };
+
+  return state.user ? (
+    <>
+      <PaddedPreference
+        title="Use container language server"
+        type="boolean"
+        value={containerLSP}
+        setValue={val => setValue(val)}
+        tooltip="Language server"
+      />
+      <SubDescription>
+        As part of making containers more powerful we now allow the language
+        server to run there. Please help us test it :-)
+      </SubDescription>
+    </>
+  ) : null;
+};

packages/app/src/app/pages/common/Modals/PreferencesModal/Experiments/index.tsx

@@ -1,41 +1,18 @@
-import { useOvermind } from 'app/overmind';
 import React from 'react';
 
-import {
-  PaddedPreference,
-  PreferenceContainer,
-  Rule,
-  SubContainer,
-  SubDescription,
-  Title,
-} from '../elements';
+import { PreferenceContainer, Rule, SubContainer, Title } from '../elements';
 import { NewSidebar } from './NewSidebar';
+import { ContainerLSP } from './ContainerLSP';
 
-export const Experiments: React.FunctionComponent = () => {
-  const { state, actions } = useOvermind();
-
-  return (
-    <div>
-      <Title>Experiments</Title>
-      <SubContainer>
-        <PreferenceContainer>
-          <NewSidebar />
-          <Rule />
-          <PaddedPreference
-            title="Use container language server"
-            type="boolean"
-            value={state.user.experiments.containerLsp}
-            setValue={() => {
-              actions.preferences.toggleContainerLspExperiment();
-            }}
-            tooltip="Language server"
-          />
-          <SubDescription>
-            As part of making containers more powerful we now allow the language
-            server to run there. Please help us test it :-)
-          </SubDescription>
-        </PreferenceContainer>
-      </SubContainer>
-    </div>
-  );
-};
+export const Experiments: React.FunctionComponent = () => (
+  <div>
+    <Title>Experiments</Title>
+    <SubContainer>
+      <PreferenceContainer>
+        <NewSidebar />
+        <Rule />
+        <ContainerLSP />
+      </PreferenceContainer>
+    </SubContainer>
+  </div>
+);