starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-09-06 00:02:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: cap the run form's height and scroll its fields

Browse Source
This commit is contained in:
AlexRV12
2026-09-03 16:30:09 +02:00
parent c7c473be0f
commit cb9d5ef856
1 changed files with 32 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/src/lib/components/copilot/chat/RunArgsFormDisplay.svelte
+32 -28
View File
@@ -111,7 +111,10 @@
</div>
</div>
<div class="mt-3">
<!-- Only the fields scroll. A script with many arguments would otherwise grow a card
taller than the pane, pushing the Run button and the lines naming what the form
dropped — a secret it opened empty among them — below the fold. -->
<div class="mt-3 max-h-[min(28rem,50vh)] overflow-y-auto">
{#if hasArgs}
<!-- The one thing here that runs before Run: a `dynselect-`/`dynmultiselect-`
argument makes DynamicInput execute that entrypoint on mount to fill its options —
@@ -132,35 +135,36 @@
{:else}
<p class="text-xs text-secondary">This script takes no arguments.</p>
{/if}
{#if runForm.droppedKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Not an input of this script, so it will not be sent:
<span class="font-mono">{runForm.droppedKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.unshowableKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Sent in a shape this form has no field for, so it opened empty:
<span class="font-mono">{runForm.unshowableKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.resetKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Disabled by this script, so it will run with its default:
<span class="font-mono">{runForm.resetKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.strippedKeys?.length}
<p class="mt-2 text-2xs text-secondary">
A secret or a file, so it opened empty for you to fill in:
<span class="font-mono">{runForm.strippedKeys.join(', ')}</span>
</p>
{/if}
{#if planMode}
<p class="mt-2 text-2xs text-secondary">{PLAN_MODE_MESSAGES.runFormRefused}</p>
{/if}
</div>
{#if runForm.droppedKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Not an input of this script, so it will not be sent:
<span class="font-mono">{runForm.droppedKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.unshowableKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Sent in a shape this form has no field for, so it opened empty:
<span class="font-mono">{runForm.unshowableKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.resetKeys?.length}
<p class="mt-2 text-2xs text-secondary">
Disabled by this script, so it will run with its default:
<span class="font-mono">{runForm.resetKeys.join(', ')}</span>
</p>
{/if}
{#if runForm.strippedKeys?.length}
<p class="mt-2 text-2xs text-secondary">
A secret or a file, so it opened empty for you to fill in:
<span class="font-mono">{runForm.strippedKeys.join(', ')}</span>
</p>
{/if}
{#if planMode}
<p class="mt-2 text-2xs text-secondary">{PLAN_MODE_MESSAGES.runFormRefused}</p>
{/if}
<!-- Both buttons rest while a submit is in flight: the ephemeral variables exist by
then, so cancelling would settle the call as declined on a run that is already
starting. Marked as the one part of the form Escape still stops the turn from. -->