From 2a54f14195e5785df17139219d3a6f92894f21d8 Mon Sep 17 00:00:00 2001
From: pablodanswer <pablo@danswer.ai>
Date: Wed, 20 Nov 2024 09:26:22 -0800
Subject: [PATCH] ensure everythigng has a default max height in
 selectorformfield (#3174)

---
 web/src/components/admin/connectors/Field.tsx | 5 ++++-
 web/src/components/llm/LLMList.tsx            | 4 +++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/web/src/components/admin/connectors/Field.tsx b/web/src/components/admin/connectors/Field.tsx
index 1f75d30adb5..a1599fb4d44 100644
--- a/web/src/components/admin/connectors/Field.tsx
+++ b/web/src/components/admin/connectors/Field.tsx
@@ -662,7 +662,10 @@ export function SelectorFormField({
           {container && (
             <SelectContent
               side={side}
-              className={maxHeight ? `max-h-[${maxHeight}]` : undefined}
+              className={`
+               ${maxHeight ? `${maxHeight}` : "max-h-72"}
+               overflow-y-scroll
+              `}
               container={container}
             >
               {options.length === 0 ? (
diff --git a/web/src/components/llm/LLMList.tsx b/web/src/components/llm/LLMList.tsx
index ca2d3f577a1..4e1ed47d028 100644
--- a/web/src/components/llm/LLMList.tsx
+++ b/web/src/components/llm/LLMList.tsx
@@ -62,7 +62,9 @@ export const LlmList: React.FC<LlmListProps> = ({
 
   return (
     <div
-      className={`${scrollable ? "max-h-[200px] include-scrollbar" : "max-h-[300px]"} bg-background-175 flex flex-col gap-y-1 overflow-y-scroll`}
+      className={`${
+        scrollable ? "max-h-[200px] include-scrollbar" : "max-h-[300px]"
+      } bg-background-175  flex flex-col gap-y-1 overflow-y-scroll`}
     >
       {userDefault && (
         <button