From 835758603cb9cb9d8c63fafb11cbb6598cad92be Mon Sep 17 00:00:00 2001 From: Jaseem Jas Date: Fri, 26 Jun 2026 15:35:21 +0530 Subject: [PATCH 1/2] UN-3185: Pin LLM profile form submit button as a sticky footer MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In the prompt-studio Settings modal, the LLM-profile Add/Edit form put its submit button (Update/Add) as the last element in normal flow inside a fixed 800px scroll column. When the form is tall — e.g. Advanced Settings expanded — the button flowed past the modal's visible area and appeared to overflow. - Make the button a sticky footer (position: sticky; bottom: 0) with a solid themed background so it stays pinned at the bottom of the scroll column and fields scroll beneath it. - The form root's .settings-body-pad-top declared overflow-y: auto with no bounded height — a dead nested scroll context that would stop the sticky footer from pinning to the real scroller (.conn-modal-col). Override it back to visible for this form (scoped via .add-llm-profile-scroll-root). Verified the sticky pinning against the real CSS rules; build green, biome clean. --- .../add-llm-profile/AddLlmProfile.css | 22 +++++++++++++++++++ .../add-llm-profile/AddLlmProfile.jsx | 7 ++++-- 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css index b4a330b154..a7c5f60ad6 100644 --- a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css +++ b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css @@ -1,5 +1,27 @@ /* Styles for AddLlmProfile */ +/* The settings modal's content column (.conn-modal-col) is the real scroll + container. .settings-body-pad-top declares its own overflow-y: auto with no + bounded height, which does nothing except register a nested scroll context + that would stop the sticky footer below from pinning to .conn-modal-col. + Drop it back to visible for this form. */ +.settings-body-pad-top.add-llm-profile-scroll-root { + overflow: visible; +} + +/* Pin the submit button to the bottom of the scrollable panel so it stays + visible as the form grows (e.g. Advanced Settings expanded) instead of + flowing past the modal. Background is set inline from the antd theme token + so scrolling fields don't show through. */ +.add-llm-profile-footer { + position: sticky; + bottom: 0; + z-index: 1; + margin-bottom: 0; + padding-top: 12px; + border-top: 1px solid #f0f0f0; +} + .add-llm-profile-row { width: 100%; } diff --git a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.jsx b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.jsx index bac0e5325f..6740cf0cda 100644 --- a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.jsx +++ b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.jsx @@ -453,7 +453,7 @@ function AddLlmProfile({ }; return ( -
+
- + {editLlmProfileId ? "Update" : "Add"} From ea946a1930f02fa7b8935dde86cd6877e24d93e7 Mon Sep 17 00:00:00 2001 From: Jaseem Jas <89440144+jaseemjaskp@users.noreply.github.com> Date: Fri, 26 Jun 2026 18:18:21 +0530 Subject: [PATCH 2/2] Apply suggestion from @greptile-apps[bot] Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> Signed-off-by: Jaseem Jas <89440144+jaseemjaskp@users.noreply.github.com> --- .../components/custom-tools/add-llm-profile/AddLlmProfile.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css index a7c5f60ad6..69be0e646f 100644 --- a/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css +++ b/frontend/src/components/custom-tools/add-llm-profile/AddLlmProfile.css @@ -19,7 +19,7 @@ z-index: 1; margin-bottom: 0; padding-top: 12px; - border-top: 1px solid #f0f0f0; + border-top: 1px solid var(--ant-color-border-secondary, #f0f0f0); } .add-llm-profile-row {