From a87dbc6b81eb3bd2f0878cfe657e4ac40fa7393e Mon Sep 17 00:00:00 2001
From: Yaro Kasear <git@kasear.net>
Date: Mon, 14 Jul 2025 12:35:04 -0500
Subject: [PATCH] More layout improvements to inventory.

---
 inventory/routes/inventory.py                 |  4 +++-
 .../templates/fragments/_editor_fragment.html | 14 +++++++++-----
 inventory/templates/inventory.html            | 19 +++++++++++++++----
 3 files changed, 27 insertions(+), 10 deletions(-)

diff --git a/inventory/routes/inventory.py b/inventory/routes/inventory.py
index bdfe325..b950252 100644
--- a/inventory/routes/inventory.py
+++ b/inventory/routes/inventory.py
@@ -95,6 +95,7 @@ def inventory_item(id):
     rooms = eager_load_room_relationships(db.session.query(Room).order_by(Room.name)).all()
     worklog_query = db.session.query(WorkLog).filter(WorkLog.work_item_id == id)
     worklog = eager_load_worklog_relationships(worklog_query).all()
+    notes = [note for log in worklog for note in log.updates]
     types = db.session.query(Item).order_by(Item.description).all()
     filtered_worklog_headers = {k: v for k, v in worklog_headers.items() if k not in ['Work Item', 'Contact', 'Follow Up?', 'Quick Analysis?']}
 
@@ -113,7 +114,8 @@ def inventory_item(id):
                            worklog=worklog,
                            worklog_headers=filtered_worklog_headers,
                            worklog_rows=[{"id": log.id, "cells": [fn(log) for fn in filtered_worklog_headers.values()]} for log in worklog],
-                           types=types
+                           types=types,
+                           notes=notes
                         )
 
 @main.route("/inventory_item/new", methods=["GET"])
diff --git a/inventory/templates/fragments/_editor_fragment.html b/inventory/templates/fragments/_editor_fragment.html
index 77b0a7f..37cd350 100644
--- a/inventory/templates/fragments/_editor_fragment.html
+++ b/inventory/templates/fragments/_editor_fragment.html
@@ -6,7 +6,7 @@
         <div class="col">
             <ul class="nav nav-tabs">
                 <li class="nav-item">
-                    <a class="nav-link text-black">{{ title }}</a>
+                    <span class="nav-link text-black">{{ title }}</span>
                 </li>
                 {% if enabled %}
                 <li class="nav-item">
@@ -29,7 +29,9 @@
     <script>
         document.addEventListener("DOMContentLoaded", () => {
             textEditor{{ id }} = document.getElementById("textEditor{{ id }}");
-            editTab{{ id }} = document.getElementById("editTab{{ id }}");
+            {% if enabled %}
+                editTab{{ id }} = document.getElementById("editTab{{ id }}");
+            {% endif %}
             viewer{{ id }} = document.getElementById("viewer{{ id }}")
 
             textEditor{{ id }}.addEventListener("input", () => EditorWidget.autoResizeTextarea(textEditor{{ id }}));
@@ -37,9 +39,11 @@
 
             viewer{{ id }}.innerHTML = marked.parse(textEditor{{ id }}.value);
 
-            editTab{{ id }}.addEventListener("click", () => {
-                EditorWidget.autoResizeTextarea(textEditor{{ id }});
-            });
+            {% if enabled %}
+                editTab{{ id }}.addEventListener("click", () => {
+                    EditorWidget.autoResizeTextarea(textEditor{{ id }});
+                });
+            {% endif %}
 
             textEditor{{ id }}.addEventListener("input", () => {
                 viewer{{ id }}.innerHTML = marked.parse(textEditor{{ id }}.value);
diff --git a/inventory/templates/inventory.html b/inventory/templates/inventory.html
index 1617301..cf1a177 100644
--- a/inventory/templates/inventory.html
+++ b/inventory/templates/inventory.html
@@ -151,10 +151,21 @@
                 ) }}
             </div>
             {% if worklog %}
-            <div class="col">
-                {{ tables.render_table(headers=worklog_headers, rows=worklog_rows, id='worklog',
-                entry_route='worklog_entry', title='Work Log') }}
-            </div>
+                <div class="col border overflow-auto" style="max-height: 300px">
+                    {% for note in notes %}
+                        {% set title %}
+                            {{ note.timestamp.strftime('%Y-%m-%d %H:%M:%S') }}{{ links.entry_link('worklog_entry', note.work_log_id) }}
+                        {% endset %}
+                        <div class="row">
+                            {{ editor.render_editor(
+                                id = 'updates' + (note.id | string),
+                                title = title,
+                                content = note.content,
+                                enabled = false
+                            ) }}
+                        </div>
+                    {% endfor %}
+                </div>
             {% endif %}
         </div>
     </div>