fix(chat): fix settings tooltip (Issue #3020) (#3021)

epam · Jan 27, 2025 · a704e8d · a704e8d
1 parent fb799dc
commit a704e8d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/apps/chat/src/components/Chat/ChatHeader/Header.tsx b/apps/chat/src/components/Chat/ChatHeader/Header.tsx
@@ -22,6 +22,7 @@ import {
   doesModelAllowAddons,
   doesModelAllowSystemPrompt,
   doesModelAllowTemperature,
+  doesModelHaveSettings,
 } from '@/src/utils/app/models';
 
 import { Conversation } from '@/src/types/chat';
@@ -346,6 +347,7 @@ export const ChatHeader = Inversify.register(
                   tooltip={
                     <HeaderSettingsTooltip
                       disallowChangeSettings={disallowChangeSettings}
+                      hasSettings={!!doesModelHaveSettings(model)}
                       subModel={
                         conversation.assistantModelId &&
                         model?.type === EntityType.Assistant

diff --git a/apps/chat/src/components/Chat/ChatHeader/HeaderSettingsTooltip.tsx b/apps/chat/src/components/Chat/ChatHeader/HeaderSettingsTooltip.tsx
@@ -15,6 +15,7 @@ interface Props {
   temperature: number | null;
   selectedAddons: DialAIEntityAddon[] | null;
   disallowChangeSettings: boolean;
+  hasSettings: boolean;
 }
 
 const SM_HEIGHT_THRESHOLDS = [
@@ -31,6 +32,7 @@ export const HeaderSettingsTooltip = ({
   temperature,
   selectedAddons,
   disallowChangeSettings,
+  hasSettings,
 }: Props) => {
   const { t } = useTranslation(Translation.Chat);
 
@@ -46,7 +48,7 @@ export const HeaderSettingsTooltip = ({
     >
       <div className="font-semibold">
         {t(
-          disallowChangeSettings
+          disallowChangeSettings || !hasSettings
             ? 'Conversation settings'
             : 'Change conversation settings',
         )}