From 86249d7ce9c11f4fcd01e9354504e3aeea4af797 Mon Sep 17 00:00:00 2001
From: andrewwallacespeckle <andrew@speckle.systems>
Date: Tue, 24 Jun 2025 13:16:13 +0200
Subject: [PATCH] Add max height menu prop to FormSelectBase. Use it in move
 version dialog

---
 .../frontend-2/components/common/model/Select.vue    |  1 +
 .../src/components/form/select/Base.vue              | 12 +++++++++++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/packages/frontend-2/components/common/model/Select.vue b/packages/frontend-2/components/common/model/Select.vue
index 0a6a12a52..d85ef7cd5 100644
--- a/packages/frontend-2/components/common/model/Select.vue
+++ b/packages/frontend-2/components/common/model/Select.vue
@@ -10,6 +10,7 @@
     :allow-unset="allowUnset"
     :label-id="labelId"
     :button-id="buttonId"
+    :menu-max-height-classes="`max-h-[30vh]`"
     :help="help"
     by="id"
   >
diff --git a/packages/ui-components/src/components/form/select/Base.vue b/packages/ui-components/src/components/form/select/Base.vue
index 3583ff56a..587de2f13 100644
--- a/packages/ui-components/src/components/form/select/Base.vue
+++ b/packages/ui-components/src/components/form/select/Base.vue
@@ -134,7 +134,10 @@
                     />
                   </div>
                 </label>
-                <div class="overflow-auto simple-scrollbar max-h-60 xl:max-h-80">
+                <div
+                  class="overflow-auto simple-scrollbar"
+                  :class="props.menuMaxHeightClasses || 'max-h-[50vh] xl:max-h-80'"
+                >
                   <div v-if="isAsyncSearchMode && isAsyncLoading" class="px-1">
                     <CommonLoadingBar :loading="true" />
                   </div>
@@ -490,6 +493,13 @@ const props = defineProps({
   menuOpenDirection: {
     type: String as PropType<'left' | 'right'>,
     default: 'left'
+  },
+  /**
+   * Custom max height classes for the dropdown menu. If not provided, defaults to 'max-h-[50vh] xl:max-h-80'
+   */
+  menuMaxHeightClasses: {
+    type: String,
+    default: undefined
   }
 })