From 4c15722ef13d97f1535e00743cea843d6f89289f Mon Sep 17 00:00:00 2001
From: Tony Guntharp <fusion94@gmail.com>
Date: Wed, 18 Mar 2026 13:10:34 -0500
Subject: [PATCH] fix: broken HTML structure in show and index views
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- show.html.erb: version partial was rendered inside the header
  <tr>, creating nested/invalid <tr> tags. Moved partial to its
  own <tbody> after <thead>.
- index.html.erb: empty state <td> was not wrapped in <tr>.
  Added <tr> wrapper and fixed rowspan→colspan.

Closes #62
---
 app/views/paper_trail_manager/changes/index.html.erb |  4 +++-
 app/views/paper_trail_manager/changes/show.html.erb  | 12 ++++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/app/views/paper_trail_manager/changes/index.html.erb b/app/views/paper_trail_manager/changes/index.html.erb
index ce7f282..e2aa4b8 100644
--- a/app/views/paper_trail_manager/changes/index.html.erb
+++ b/app/views/paper_trail_manager/changes/index.html.erb
@@ -26,7 +26,9 @@
           <%= render partial: 'version', object: version %>
         <% end %>
       <% else %>
-        <td rowspan='2'> &mdash; No changes found &mdash; </td>
+        <tr>
+          <td colspan='2'> &mdash; No changes found &mdash; </td>
+        </tr>
       <% end %>
     </tbody>
   </table>
diff --git a/app/views/paper_trail_manager/changes/show.html.erb b/app/views/paper_trail_manager/changes/show.html.erb
index 6f57bb3..756d583 100644
--- a/app/views/paper_trail_manager/changes/show.html.erb
+++ b/app/views/paper_trail_manager/changes/show.html.erb
@@ -1,9 +1,13 @@
 <h1>Change <%= @version.id %></h1>
 
 <table class='changes_table'>
-  <tr class='changes_header'>
-    <th class='change_time'>Time</th>
-    <th class='change_details'>Attribute with previous and current values</th>
+  <thead>
+    <tr class='changes_header'>
+      <th class='change_time'>Time</th>
+      <th class='change_details'>Attribute with previous and current values</th>
+    </tr>
+  </thead>
+  <tbody>
     <%= render partial: 'version', object: @version %>
-  </tr>
+  </tbody>
 </table>