From c7a5cb52e3435617250777510fc4734cdf8f9e48 Mon Sep 17 00:00:00 2001
From: Faisal Amir <urmauur@gmail.com>
Date: Thu, 19 Dec 2024 23:12:24 +0800
Subject: [PATCH] fix: conditional render model setting based on selected model
 (#4300)

---
 web/screens/Thread/ThreadRightPanel/index.tsx | 56 ++++++++++---------
 1 file changed, 30 insertions(+), 26 deletions(-)

diff --git a/web/screens/Thread/ThreadRightPanel/index.tsx b/web/screens/Thread/ThreadRightPanel/index.tsx
index 939bb3fe7..dab50aba7 100644
--- a/web/screens/Thread/ThreadRightPanel/index.tsx
+++ b/web/screens/Thread/ThreadRightPanel/index.tsx
@@ -268,34 +268,38 @@ const ThreadRightPanel = () => {
           <div className="flex flex-col gap-4 px-2 py-4">
             <ModelDropdown />
           </div>
-          <Accordion defaultValue={[]}>
-            {settings.runtimeSettings.length !== 0 && (
-              <AccordionItem
-                title={INFERENCE_SETTINGS}
-                value={INFERENCE_SETTINGS}
-              >
-                <ModelSetting
-                  componentProps={settings.runtimeSettings}
-                  onValueChanged={onValueChanged}
-                />
-              </AccordionItem>
-            )}
+          {selectedModel && (
+            <Accordion defaultValue={[]}>
+              {settings.runtimeSettings.length !== 0 && (
+                <AccordionItem
+                  title={INFERENCE_SETTINGS}
+                  value={INFERENCE_SETTINGS}
+                >
+                  <ModelSetting
+                    componentProps={settings.runtimeSettings}
+                    onValueChanged={onValueChanged}
+                  />
+                </AccordionItem>
+              )}
 
-            {promptTemplateSettings.length !== 0 && (
-              <AccordionItem title={MODEL_SETTINGS} value={MODEL_SETTINGS}>
-                <PromptTemplateSetting componentData={promptTemplateSettings} />
-              </AccordionItem>
-            )}
+              {promptTemplateSettings.length !== 0 && (
+                <AccordionItem title={MODEL_SETTINGS} value={MODEL_SETTINGS}>
+                  <PromptTemplateSetting
+                    componentData={promptTemplateSettings}
+                  />
+                </AccordionItem>
+              )}
 
-            {settings.engineSettings.length !== 0 && (
-              <AccordionItem title={ENGINE_SETTINGS} value={ENGINE_SETTINGS}>
-                <EngineSetting
-                  componentData={settings.engineSettings}
-                  onValueChanged={onValueChanged}
-                />
-              </AccordionItem>
-            )}
-          </Accordion>
+              {settings.engineSettings.length !== 0 && (
+                <AccordionItem title={ENGINE_SETTINGS} value={ENGINE_SETTINGS}>
+                  <EngineSetting
+                    componentData={settings.engineSettings}
+                    onValueChanged={onValueChanged}
+                  />
+                </AccordionItem>
+              )}
+            </Accordion>
+          )}
         </TabsContent>
         <TabsContent value="tools">
           <Tools />