From f37a220a0560c7567548bfeef0fa44148574fed9 Mon Sep 17 00:00:00 2001
From: Roipoussiere <roipoussiere@protonmail.com>
Date: Thu, 15 Jun 2023 11:46:39 +0200
Subject: [PATCH] add checkbox in settings to enable line wrapping

---
 packages/react/src/components/CodeMirror6.jsx | 15 +++++++--------
 website/src/repl/Footer.jsx                   | 17 +++++++++++++++--
 website/src/repl/Repl.jsx                     | 11 ++++++++++-
 website/src/settings.mjs                      |  2 ++
 4 files changed, 34 insertions(+), 11 deletions(-)

diff --git a/packages/react/src/components/CodeMirror6.jsx b/packages/react/src/components/CodeMirror6.jsx
index 18306d31..36c7601b 100644
--- a/packages/react/src/components/CodeMirror6.jsx
+++ b/packages/react/src/components/CodeMirror6.jsx
@@ -88,13 +88,7 @@ const highlightField = StateField.define({
   provide: (f) => EditorView.decorations.from(f),
 });
 
-const staticExtensions = [
-  javascript(),
-  highlightField,
-  flashField,
-  javascriptLanguage.data.of({ autocomplete: strudelAutocomplete }),
-  EditorView.lineWrapping,
-];
+const staticExtensions = [javascript(), highlightField, flashField];
 
 export default function CodeMirror({
   value,
@@ -105,6 +99,7 @@ export default function CodeMirror({
   keybindings,
   isLineNumbersDisplayed,
   isAutoCompletionEnabled,
+  isLineWrappingEnabled,
   fontSize = 18,
   fontFamily = 'monospace',
   options,
@@ -150,8 +145,12 @@ export default function CodeMirror({
       _extensions.push(autocompletion({ override: [] }));
     }
 
+    if (isLineWrappingEnabled) {
+      _extensions.push(EditorView.lineWrapping);
+    }
+
     return _extensions;
-  }, [keybindings, isAutoCompletionEnabled]);
+  }, [keybindings, isAutoCompletionEnabled, isLineWrappingEnabled]);
 
   return (
     <div style={{ fontSize, fontFamily }} className="w-full">
diff --git a/website/src/repl/Footer.jsx b/website/src/repl/Footer.jsx
index 30e2f3a1..acd5a682 100644
--- a/website/src/repl/Footer.jsx
+++ b/website/src/repl/Footer.jsx
@@ -364,7 +364,15 @@ const fontFamilyOptions = {
 };
 
 function SettingsTab({ scheduler }) {
-  const { theme, keybindings, isLineNumbersDisplayed, isAutoCompletionEnabled, fontSize, fontFamily } = useSettings();
+  const {
+    theme,
+    keybindings,
+    isLineNumbersDisplayed,
+    isAutoCompletionEnabled,
+    isLineWrappingEnabled,
+    fontSize,
+    fontFamily,
+  } = useSettings();
 
   return (
     <div className="text-foreground p-4 space-y-4">
@@ -407,7 +415,7 @@ function SettingsTab({ scheduler }) {
           />
         </FormItem>
       </div>
-      <div className="grid grid-cols-1 md:grid-cols-3 gap-4">
+      <div className="grid grid-cols-1 md:grid-cols-4 gap-4">
         <FormItem label="Keybindings">
           <ButtonGroup
             value={keybindings}
@@ -425,6 +433,11 @@ function SettingsTab({ scheduler }) {
           onChange={(cbEvent) => settingsMap.setKey('isAutoCompletionEnabled', cbEvent.target.checked)}
           value={isAutoCompletionEnabled}
         />
+        <Checkbox
+          label="Enable line wrapping"
+          onChange={(cbEvent) => settingsMap.setKey('isLineWrappingEnabled', cbEvent.target.checked)}
+          value={isLineWrappingEnabled}
+        />
       </div>
       <FormItem label="Reset Settings">
         <button
diff --git a/website/src/repl/Repl.jsx b/website/src/repl/Repl.jsx
index 74102c91..fb05b492 100644
--- a/website/src/repl/Repl.jsx
+++ b/website/src/repl/Repl.jsx
@@ -105,7 +105,15 @@ export function Repl({ embedded = false }) {
   const [lastShared, setLastShared] = useState();
   const [pending, setPending] = useState(true);
 
-  const { theme, keybindings, fontSize, fontFamily, isLineNumbersDisplayed, isAutoCompletionEnabled } = useSettings();
+  const {
+    theme,
+    keybindings,
+    fontSize,
+    fontFamily,
+    isLineNumbersDisplayed,
+    isAutoCompletionEnabled,
+    isLineWrappingEnabled,
+  } = useSettings();
 
   const { code, setCode, scheduler, evaluate, activateCode, isDirty, activeCode, pattern, started, stop, error } =
     useStrudel({
@@ -273,6 +281,7 @@ export function Repl({ embedded = false }) {
             keybindings={keybindings}
             isLineNumbersDisplayed={isLineNumbersDisplayed}
             isAutoCompletionEnabled={isAutoCompletionEnabled}
+            isLineWrappingEnabled={isLineWrappingEnabled}
             fontSize={fontSize}
             fontFamily={fontFamily}
             onChange={handleChangeCode}
diff --git a/website/src/settings.mjs b/website/src/settings.mjs
index 2e42f746..0e485f29 100644
--- a/website/src/settings.mjs
+++ b/website/src/settings.mjs
@@ -6,6 +6,7 @@ export const defaultSettings = {
   keybindings: 'codemirror',
   isLineNumbersDisplayed: true,
   isAutoCompletionEnabled: false,
+  isLineWrappingEnabled: false,
   theme: 'strudelTheme',
   fontFamily: 'monospace',
   fontSize: 18,
@@ -23,6 +24,7 @@ export function useSettings() {
     isZen: [true, 'true'].includes(state.isZen) ? true : false,
     isLineNumbersDisplayed: [true, 'true'].includes(state.isLineNumbersDisplayed) ? true : false,
     isAutoCompletionEnabled: [true, 'true'].includes(state.isAutoCompletionEnabled) ? true : false,
+    isLineWrappingEnabled: [true, 'true'].includes(state.isLineWrappingEnabled) ? true : false,
     fontSize: Number(state.fontSize),
   };
 }