From 4f6c2a183ee6aa8a2ee5a57e3f4ece94a3216c74 Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Sat, 23 Mar 2024 16:51:49 +0100
Subject: [PATCH] improve flow rendering

---
 frontend/src/lib/components/FlowJobResult.svelte       |  2 +-
 .../src/lib/components/FlowStatusViewerInner.svelte    |  2 +-
 frontend/src/lib/components/LogViewer.svelte           | 10 +++++-----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/frontend/src/lib/components/FlowJobResult.svelte b/frontend/src/lib/components/FlowJobResult.svelte
index 787ef86c0a..6a6790f15e 100644
--- a/frontend/src/lib/components/FlowJobResult.svelte
+++ b/frontend/src/lib/components/FlowJobResult.svelte
@@ -43,7 +43,7 @@
 	class:border={!noBorder}
 	class="grid {!col
 		? 'grid-cols-2'
-		: 'grid-rows-2'} shadow border border-tertiary-inverse h-full max-h-screen"
+		: 'grid-rows-2'} shadow border border-tertiary-inverse h-full max-h-[70vh]"
 >
 	<div class="bg-surface {col ? '' : 'max-h-80'} h-full p-1 overflow-auto relative">
 		<span class="text-tertiary">Result</span>
diff --git a/frontend/src/lib/components/FlowStatusViewerInner.svelte b/frontend/src/lib/components/FlowStatusViewerInner.svelte
index d5d268933b..21a9378888 100644
--- a/frontend/src/lib/components/FlowStatusViewerInner.svelte
+++ b/frontend/src/lib/components/FlowStatusViewerInner.svelte
@@ -681,7 +681,7 @@
 						/>
 					</div>
 					<div
-						class="border-l border-tertiary-inverse pt-1 overflow-auto min-h-[800px] flex flex-col"
+						class="border-l border-tertiary-inverse pt-1 overflow-auto min-h-[700px] flex flex-col"
 					>
 						<Tabs bind:selected={rightColumnSelect}>
 							<Tab value="timeline"><span class="font-semibold text-md">Timeline</span></Tab>
diff --git a/frontend/src/lib/components/LogViewer.svelte b/frontend/src/lib/components/LogViewer.svelte
index f93a883345..6bfa201719 100644
--- a/frontend/src/lib/components/LogViewer.svelte
+++ b/frontend/src/lib/components/LogViewer.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	import { ClipboardCopy, Download, Loader2 } from 'lucide-svelte'
+	import { ClipboardCopy, Download, Expand, Loader2 } from 'lucide-svelte'
 	import { Button, Drawer, DrawerContent } from './common'
 	import { copyToClipboard } from '$lib/utils'
 	import { workspaceStore } from '$lib/stores'
@@ -94,19 +94,19 @@
 	<div bind:this={div} class="w-full h-full overflow-auto relative bg-surface-secondary">
 		<div class="sticky z-10 top-0 right-0 w-full flex flex-row-reverse justify-between text-sm">
 			<div class="flex gap-2 pl-0.5 bg-surface-secondary">
-				<div class="pt-2">
+				<div class="flex items-center">
 					<a
-						class="text-primary"
+						class="text-primary pb-0.5"
 						target="_blank"
 						href="/api/w/{$workspaceStore}/jobs_u/get_logs/{jobId}"
 						download="windmill-logs.json"><Download size="14" /></a
 					>
 				</div>
-				<button on:click={logViewer.openDrawer}>Expand</button>
+				<button on:click={logViewer.openDrawer}><Expand size="12" /></button>
 				<div
 					class="{small ? '' : 'py-2'} pr-2 {small
 						? '!text-2xs'
-						: '!text-xs'} flex gap-2 items-center"
+						: '!text-xs'} flex gap-2 text-tertiary items-center"
 				>
 					Auto scroll
 					<input class="windmillapp" type="checkbox" bind:checked={scroll} />