mirror of
https://github.com/davidkaya/aryx.git
synced 2026-08-27 21:33:58 +02:00
fix: prevent settings panel from collapsing graph canvas
The center column was a flex column with the canvas as flex-1 min-h-0 and the settings panel as shrink-0. When settings content grew tall (especially with the orchestration mode panel expanded), the canvas collapsed to zero height. Switch the center column to a scrollable container with the canvas having a guaranteed minimum height via clamp(360px, 50vh, 100%). Settings flow naturally below and the whole column scrolls. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
@@ -490,7 +490,7 @@ export function WorkflowEditor({
|
|||||||
</div>
|
</div>
|
||||||
|
|
||||||
{/* Center column: validation + canvas + settings */}
|
{/* Center column: validation + canvas + settings */}
|
||||||
<div className="flex min-w-0 flex-1 flex-col">
|
<div className="min-w-0 flex-1 overflow-y-auto">
|
||||||
{/* Validation banner */}
|
{/* Validation banner */}
|
||||||
<div className="px-5 pt-4">
|
<div className="px-5 pt-4">
|
||||||
{issues.length > 0 ? (
|
{issues.length > 0 ? (
|
||||||
@@ -517,8 +517,8 @@ export function WorkflowEditor({
|
|||||||
)}
|
)}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{/* Graph canvas */}
|
{/* Graph canvas — guaranteed minimum height so settings never collapse it */}
|
||||||
<div className="min-h-0 flex-1 px-5 pb-2 pt-4">
|
<div className="min-h-[360px] px-5 pb-2 pt-4" style={{ height: 'clamp(360px, 50vh, 100%)' }}>
|
||||||
<WorkflowGraphCanvas
|
<WorkflowGraphCanvas
|
||||||
availableModels={availableModels}
|
availableModels={availableModels}
|
||||||
onEdgeSelect={setSelectedEdgeId}
|
onEdgeSelect={setSelectedEdgeId}
|
||||||
@@ -652,7 +652,7 @@ function WorkflowSettingsPanel({
|
|||||||
onChange: (workflow: WorkflowDefinition) => void;
|
onChange: (workflow: WorkflowDefinition) => void;
|
||||||
}) {
|
}) {
|
||||||
return (
|
return (
|
||||||
<div className="shrink-0 border-t border-[var(--color-border)] px-5 py-4">
|
<div className="border-t border-[var(--color-border)] px-5 py-4">
|
||||||
<h4 className="mb-3 text-[12px] font-semibold uppercase tracking-wider text-[var(--color-text-muted)]">
|
<h4 className="mb-3 text-[12px] font-semibold uppercase tracking-wider text-[var(--color-text-muted)]">
|
||||||
Settings
|
Settings
|
||||||
</h4>
|
</h4>
|
||||||
|
|||||||
Reference in New Issue
Block a user