From dc9653307b3e87bf6af1a613ff711e19042b6c72 Mon Sep 17 00:00:00 2001
From: James Read <contact@jread.com>
Date: Fri, 30 Aug 2024 21:28:12 +0100
Subject: [PATCH] feature: Logs, column for tag, and left align status. Stop
 nav flash (#393)

---
 webui.dev/index.html                | 7 ++++---
 webui.dev/js/ActionStatusDisplay.js | 3 ++-
 webui.dev/js/marshaller.js          | 2 ++
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/webui.dev/index.html b/webui.dev/index.html
index f7d0907..1f89922 100644
--- a/webui.dev/index.html
+++ b/webui.dev/index.html
@@ -27,7 +27,7 @@
 
 			<h1 id = "page-title">OliveTin</h1>
 
-			<nav>
+			<nav hidden>
 				<ul id = "navigation-links">
 					<li title = "Actions">
 						<a id = "showActions">Actions</a>
@@ -57,7 +57,8 @@
 					<thead>
 						<tr title = "untitled">
 							<th>Timestamp</th>
-							<th>Log</th>
+							<th>Action</th>
+							<th>Tags</th>
 							<th>Status</th>
 						</tr>
 					</thead>
@@ -184,8 +185,8 @@
 					<span role = "img" class = "icon"></span>
 					<a href = "javascript:void(0)" class = "content">?</a>
 
-					<div class = "tags"></div>
 				</td>
+				<td class = "tags"></td>
 				<td class = "exit-code">?</td>
 			</tr>
 		</template>
diff --git a/webui.dev/js/ActionStatusDisplay.js b/webui.dev/js/ActionStatusDisplay.js
index 757a77a..ac11739 100644
--- a/webui.dev/js/ActionStatusDisplay.js
+++ b/webui.dev/js/ActionStatusDisplay.js
@@ -15,10 +15,11 @@ export class ActionStatusDisplay {
 
   update (logEntry) {
     this.statusElement.classList.remove(...this.statusElement.classList)
+    this.statusElement.classList.add('action-status')
 
     if (logEntry.executionFinished) {
       this.statusElement.innerText = 'Completed'
-      this.exitCodeElement.innerText = ', Exit code: ' + logEntry.exitCode
+      this.exitCodeElement.innerText = ' Exit code: ' + logEntry.exitCode
 
       if (logEntry.exitCode === 0) {
         this.statusElement.classList.add('action-success')
diff --git a/webui.dev/js/marshaller.js b/webui.dev/js/marshaller.js
index 17dc7ae..b695650 100644
--- a/webui.dev/js/marshaller.js
+++ b/webui.dev/js/marshaller.js
@@ -163,6 +163,8 @@ function setSectionNavigationVisible (visible) {
   const nav = document.querySelector('nav')
   const btn = document.getElementById('sidebar-toggler-button')
 
+  nav.removeAttribute('hidden')
+
   if (document.body.classList.contains('has-sidebar')) {
     if (visible) {
       btn.setAttribute('aria-pressed', false)