From 1e2c26123bcfaa16fe3dfeda824f316e15ef1208 Mon Sep 17 00:00:00 2001
From: "Jade (Rose) Rowland" <jaderowlanddev@gmail.com>
Date: Wed, 3 Jan 2024 00:35:47 -0500
Subject: [PATCH] working

---
 website/src/repl/Header.jsx            |  2 +-
 website/src/repl/Repl.jsx              | 12 +++--
 website/src/repl/panel/PatternsTab.jsx | 69 ++++++++++++--------------
 3 files changed, 41 insertions(+), 42 deletions(-)

diff --git a/website/src/repl/Header.jsx b/website/src/repl/Header.jsx
index 774d0142..f806d5d3 100644
--- a/website/src/repl/Header.jsx
+++ b/website/src/repl/Header.jsx
@@ -85,7 +85,7 @@ export function Header({ context }) {
             )}
           </button>
           <button
-            onClick={() => handleUpdate()}
+            onClick={() => handleUpdate({})}
             title="update"
             className={cx(
               'flex items-center space-x-1',
diff --git a/website/src/repl/Repl.jsx b/website/src/repl/Repl.jsx
index d53e7e5b..404db1bd 100644
--- a/website/src/repl/Repl.jsx
+++ b/website/src/repl/Repl.jsx
@@ -139,17 +139,19 @@ export function Repl({ embedded = false }) {
   //
 
   const handleTogglePlay = async () => editorRef.current?.toggle();
-  const handleUpdate = async (newCode, reset = false) => {
+
+  // payload = {reset: boolean, code: string, evaluate: boolean}
+  const handleUpdate = async ({ reset = false, code = null, evaluate = true }) => {
     if (reset) {
       clearCanvas();
       resetLoadedSounds();
       editorRef.current.repl.setCps(1);
       await prebake(); // declare default samples
     }
-    if (newCode) {
-      editorRef.current.setCode(newCode);
-      editorRef.current.repl.evaluate(newCode);
-    } else if (isDirty) {
+    if (code != null) {
+      editorRef.current.setCode(code);
+    }
+    if (evaluate && isDirty) {
       editorRef.current.evaluate();
     }
   };
diff --git a/website/src/repl/panel/PatternsTab.jsx b/website/src/repl/panel/PatternsTab.jsx
index 4466b599..602c4eaa 100644
--- a/website/src/repl/panel/PatternsTab.jsx
+++ b/website/src/repl/panel/PatternsTab.jsx
@@ -19,10 +19,40 @@ function classNames(...classes) {
   return classes.filter(Boolean).join(' ');
 }
 
+function PatternButton({ isActive, onClick, label }) {
+  return (
+    <a
+      className={classNames('mr-4 hover:opacity-50 cursor-pointer inline-block', isActive ? 'outline outline-1' : '')}
+      onClick={onClick}
+    >
+      {label}
+    </a>
+  );
+}
+
+function PatternButtons({ patterns, activePattern, onClick }) {
+  return (
+    <div className="font-mono text-sm">
+      {Object.entries(patterns).map(([key, data]) => (
+        <PatternButton key={key} label={key} isActive={key === activePattern} onClick={() => onClick(key, data)} />
+      ))}
+    </div>
+  );
+}
+
 export function PatternsTab({ context }) {
   const { userPatterns } = useSettings();
   const activePattern = useActivePattern();
   const isExample = useMemo(() => activePattern && !!tunes[activePattern], [activePattern]);
+  const onPatternClick = (key, data) => {
+    const { code } = data;
+    setActivePattern(key);
+    context.handleUpdate({ code, evaluate: false });
+  };
+
+  const examplePatterns = {};
+  Object.entries(tunes).forEach(([key, code]) => (examplePatterns[key] = { code }));
+
   return (
     <div className="px-4 w-full dark:text-white text-stone-900 space-y-4 pb-4">
       <section>
@@ -47,31 +77,14 @@ export function PatternsTab({ context }) {
             </div>
           </div>
         )}
-        <div className="font-mono text-sm">
-          {Object.entries(userPatterns).map(([key, up]) => (
-            <a
-              key={key}
-              className={classNames(
-                'mr-4 hover:opacity-50 cursor-pointer inline-block',
-                key === activePattern ? 'outline outline-1' : '',
-              )}
-              onClick={() => {
-                const { code } = up;
-                setActivePattern(key);
-                context.handleUpdate(code, true);
-              }}
-            >
-              {key}
-            </a>
-          ))}
-        </div>
+        <PatternButtons onClick={onPatternClick} patterns={userPatterns} activePattern={activePattern} />
         <div className="pr-4 space-x-4 border-b border-foreground mb-2 h-8 flex overflow-auto max-w-full items-center">
           <button
             className="hover:opacity-50"
             onClick={() => {
               const name = newUserPattern();
               const { code } = getUserPattern(name);
-              context.handleUpdate(code, true);
+              context.handleUpdate({ code, evaluate: false });
             }}
           >
             new
@@ -96,23 +109,7 @@ export function PatternsTab({ context }) {
       </section>
       <section>
         <h2 className="text-xl mb-2">Examples</h2>
-        <div className="font-mono text-sm">
-          {Object.entries(tunes).map(([key, tune]) => (
-            <a
-              key={key}
-              className={classNames(
-                'mr-4 hover:opacity-50 cursor-pointer inline-block',
-                key === activePattern ? 'outline outline-1' : '',
-              )}
-              onClick={() => {
-                setActivePattern(key);
-                context.handleUpdate(tune, true);
-              }}
-            >
-              {key}
-            </a>
-          ))}
-        </div>
+        <PatternButtons onClick={onPatternClick} patterns={examplePatterns} activePattern={activePattern} />
       </section>
     </div>
   );