From 4214a3a6e2c4127df4dd9ae839069ba292eb4816 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Mon, 7 Oct 2024 15:23:37 -0700 Subject: [PATCH] Inline code + effect clarity (#2715) * cleaner code blocks + form context * cleaner * nit --- web/src/app/chat/message/CodeBlock.tsx | 49 ++++++++++--------- .../components/context/EmbeddingContext.tsx | 2 +- web/src/components/context/FormContext.tsx | 2 +- 3 files changed, 27 insertions(+), 26 deletions(-) diff --git a/web/src/app/chat/message/CodeBlock.tsx b/web/src/app/chat/message/CodeBlock.tsx index ee6b6e6eab6..c1f2f99397c 100644 --- a/web/src/app/chat/message/CodeBlock.tsx +++ b/web/src/app/chat/message/CodeBlock.tsx @@ -55,32 +55,33 @@ export const CodeBlock = memo(function CodeBlock({ ); + if (typeof children === "string") { + return ( + + {children} + + ); + } + const CodeContent = () => { if (!language) { - if (typeof children === "string") { - return ( - - {children} - - ); - } return (
           
diff --git a/web/src/components/context/EmbeddingContext.tsx b/web/src/components/context/EmbeddingContext.tsx
index 33bb850a87c..3d03b99b1d5 100644
--- a/web/src/components/context/EmbeddingContext.tsx
+++ b/web/src/components/context/EmbeddingContext.tsx
@@ -74,7 +74,7 @@ export const EmbeddingFormProvider: React.FC<{
     if (stepFromUrl !== formStep) {
       setFormStep(stepFromUrl);
     }
-  }, [searchParams, formStep]);
+  }, [searchParams]);
 
   const contextValue: EmbeddingFormContextType = {
     formStep,
diff --git a/web/src/components/context/FormContext.tsx b/web/src/components/context/FormContext.tsx
index ee6e9dab786..d445782f718 100644
--- a/web/src/components/context/FormContext.tsx
+++ b/web/src/components/context/FormContext.tsx
@@ -73,7 +73,7 @@ export const FormProvider: React.FC<{
     if (stepFromUrl !== formStep) {
       setFormStep(stepFromUrl);
     }
-  }, [searchParams, formStep]);
+  }, [searchParams]);
 
   const contextValue: FormContextType = {
     formStep,