Merge pull request #127 from SaraVieira/logs-improvements

UX improvements for the log page

.gitignore

@@ -3,6 +3,7 @@ node_modules
 /build
 /.svelte-kit
 /package
+/yarn.lock
 
 .env
 .env.prod

src/routes/applications/[id]/logs/build/_BuildLog.svelte

@@ -1,5 +1,6 @@
 <script lang="ts">
 	export let buildId;
+	export let followingBuild;
 
 	import { createEventDispatcher, onDestroy, onMount } from 'svelte';
 	const dispatch = createEventDispatcher();
@@ -18,13 +19,15 @@
 
 	const { id } = $page.params;
 
+	const cleanAnsiCodes = (str: string) => str.replace(/\x1B\[(\d+)m/g, '');
+
 	async function streamLogs(sequence = 0) {
 		try {
 			let { logs: responseLogs, status } = await get(
 				`/applications/${id}/logs/build/build.json?buildId=${buildId}&sequence=${sequence}`
 			);
 			currentStatus = status;
-			logs = logs.concat(responseLogs);
+			logs = logs.concat(responseLogs.map((log) => ({ ...log, line: cleanAnsiCodes(log.line) })));
 			loading = false;
 			streamInterval = setInterval(async () => {
 				if (status !== 'running') {
@@ -38,8 +41,13 @@
 					);
 					status = data.status;
 					currentStatus = status;
-					logs = logs.concat(data.logs);
+
+					logs = logs.concat(data.logs.map((log) => ({ ...log, line: cleanAnsiCodes(log.line) })));
 					dispatch('updateBuildStatus', { status });
+					if (followingBuild) {
+						const logEl = document.getElementById('logs');
+						logEl.scrollTop = logEl.scrollHeight;
+					}
 				} catch ({ error }) {
 					return errorNotification(error);
 				}
@@ -65,7 +73,8 @@
 			<LoadingLogs />
 		{/if}
 		<div
-			class="font-mono leading-6 text-left text-md tracking-tighter rounded bg-coolgray-200 py-5 px-6 whitespace-pre-wrap break-words"
+			class="font-mono leading-6 text-left text-md tracking-tighter rounded bg-coolgray-200 py-5 px-6 whitespace-pre-wrap break-words overflow-auto max-h-[80vh]"
+			id="logs"
 		>
 			{#each logs as log}
 				<div>{log.line + '\n'}</div>

src/routes/applications/[id]/logs/build/index.svelte

@@ -33,6 +33,7 @@
 	export let buildCount;
 
 	let buildId;
+	let followingBuild;
 	$: buildId;
 
 	let skip = 0;
@@ -85,6 +86,15 @@
 		buildId = build;
 		goto(`/applications/${id}/logs/build?buildId=${buildId}`);
 	}
+
+	function followBuild() {
+		followingBuild = !followingBuild;
+		if (followingBuild) {
+			const logEl = document.getElementById('logs');
+			logEl.scrollTop = logEl.scrollHeight;
+			window.scrollTo(0, document.body.scrollHeight);
+		}
+	}
 </script>
 
 <div class="flex space-x-1 p-6 font-bold">
@@ -92,8 +102,9 @@
 		Build logs of <a href={application.fqdn} target="_blank">{getDomain(application.fqdn)}</a>
 	</div>
 </div>
-<div class="flex flex-row justify-start space-x-2 px-10 pt-6 ">
-	<div class="min-w-[16rem] space-y-2">
+<div class="block flex-row justify-start space-x-2 px-5 pt-6 sm:px-10 md:flex">
+	<div class="mb-4 min-w-[16rem] space-y-2 md:mb-0 ">
+		<div class="top-4 md:sticky">
 			{#each builds as build (build.id)}
 				<div
 					data-tooltip={new Intl.DateTimeFormat('default', dateOptions).format(
@@ -126,14 +137,40 @@
 					</div>
 				</div>
 			{/each}
+		</div>
+		<button
+			on:click={followBuild}
+			data-tooltip="Follow logs"
+			class={followingBuild && 'text-green-500'}
+		>
+			<svg
+				xmlns="http://www.w3.org/2000/svg"
+				class="h-6 w-6"
+				fill="none"
+				viewBox="0 0 24 24"
+				stroke="currentColor"
+			>
+				<path
+					stroke-linecap="round"
+					stroke-linejoin="round"
+					stroke-width="2"
+					d="M15 13l-3 3m0 0l-3-3m3 3V8m0 13a9 9 0 110-18 9 9 0 010 18z"
+				/>
+			</svg>
+		</button>
 		{#if buildCount > 0 && !noMoreBuilds}
 			<button class="w-full" on:click={loadMoreBuilds}>Load More</button>
 		{/if}
 	</div>
-	<div class="w-96 flex-1">
+	<div class="flex-1 md:w-96">
 		{#if buildId}
 			{#key buildId}
-				<svelte:component this={BuildLog} {buildId} on:updateBuildStatus={updateBuildStatus} />
+				<svelte:component
+					this={BuildLog}
+					{buildId}
+					{followingBuild}
+					on:updateBuildStatus={updateBuildStatus}
+				/>
 			{/key}
 		{/if}
 	</div>

src/tailwind.css

@@ -69,7 +69,7 @@ a {
 }
 
 .nav-side {
-	@apply absolute right-0 top-0 z-50 m-5 flex items-center justify-end space-x-2 bg-coolblack/40 text-white;
+	@apply relative right-0 top-0 z-50 m-5 flex flex-wrap items-center justify-end space-x-2 bg-coolblack/40 text-white sm:absolute;
 }
 
 .add-icon {