From 9cc2af9cec9b2c7c699377e42eb9c24ada52dd9f Mon Sep 17 00:00:00 2001
From: Rick Barrette <rickbarrette@gmail.com>
Date: Tue, 11 Aug 2026 18:33:16 -0400
Subject: [PATCH] Refactor vehicle views for improved formatting and note
 handling

---
 .../issues/_show_issue_view_right.html.erb    | 21 ++++----
 app/views/vehicles/_details.html.erb          | 49 ++++++++-----------
 app/views/vehicles/_form.html.erb             |  9 +++-
 .../hooks/issues_show_hook_listener.rb        |  3 +-
 4 files changed, 42 insertions(+), 40 deletions(-)

diff --git a/app/views/issues/_show_issue_view_right.html.erb b/app/views/issues/_show_issue_view_right.html.erb
index 86942f4..4301b78 100644
--- a/app/views/issues/_show_issue_view_right.html.erb
+++ b/app/views/issues/_show_issue_view_right.html.erb
@@ -1,17 +1,20 @@
 <div class="vehicle attribute">
-    <div class="label"><%=t(:field_vehicle)%>:</div>
-    <div class="value"><%= vehicle %></div>
+  <div class="label"><%= t(:field_vehicle) %>:</div>
+  <div class="value"><%= vehicle %></div>
 </div>
     
 <div class="vehicle_vin attribute">
-    <div class="label"><%=t(:field_vin)%>:</div>
-    <div class="value" id="vin">
-        <div id="copyLink" onclick="handleCopy(event)"><%=split_vin[0] if split_vin%><b><%=split_vin[1] if split_vin%></b></div>
-    </div>
-
+  <div class="label"><%= t(:field_vin) %>:</div>
+  <div class="value" id="vin">
+    <div id="copyLink" onclick="handleCopy(event)"><%= split_vin[0] if split_vin %><b><%= split_vin[1] if split_vin %></b></div>
+  </div>
 </div>
     
 <div class="vehicle_notes attribute">
-    <div class="label"><%=t(:field_notes)%>:</div>
-    <pre class="value" id="note-display" style="text-align: left; white-space: pre-wrap; font-family: inherit; "><%=notes%></pre>
+  <div class="label"><%= t(:field_notes) %>:</div>
+  <% if vehicle.present? %>
+    <div class="value wiki">
+      <%= textilizable(@issue.vehicle, :notes) %>
+    </div>
+  <% end %>
 </div>
\ No newline at end of file
diff --git a/app/views/vehicles/_details.html.erb b/app/views/vehicles/_details.html.erb
index b46a232..4a0d24b 100644
--- a/app/views/vehicles/_details.html.erb
+++ b/app/views/vehicles/_details.html.erb
@@ -1,13 +1,12 @@
 <div class="issue">
   <div class="splitcontent">
     <div class="splitcontentleft">
-      <h4><%=t(:label_details)%>:</h4>
+      <h4><%= t(:label_details) %>:</h4>
       
       <table>
         <tbody>
-
           <tr>
-            <th><%= t(:field_customer)%></th>
+            <th><%= t(:field_customer) %></th>
             <td><%= vehicle.customer ? link_to(vehicle.customer.name, customer_path(vehicle.customer)) : t(:no_customer) %></td>
           </tr>
         
@@ -19,39 +18,33 @@
           <tr>
             <th><%= t(:field_vin) %></th>
             <td>
-               <div onclick="handleCopy(event)"><%= @vin[0] if @vin %><b><%=@vin[1] if @vin%></b></div>
+               <div onclick="handleCopy(event)"><%= @vin[0] if @vin %><b><%= @vin[1] if @vin %></b></div>
             </td>
           </tr>
-          
-          <th><%= t(:label_trim) %></th>
-            <td><%= vehicle.doors %> <%=t(:label_door) if vehicle.doors? %> <%= vehicle.trim %></td>
+
+          <tr>
+            <th><%= t(:label_trim) %></th>
+            <td><%= vehicle.doors %> <%= t(:label_door) if vehicle.doors? %> <%= vehicle.trim %></td>
           </tr>
         </tbody>
       </table>
-      
     </div>
 
-    <div class="splitcontentleft">
-
-      <h4><%=t(:field_notes)%>:</h4>
-        <pre id="note-display" style="text-align: left; white-space: pre-wrap; font-family: inherit; ">
-          <td><%= vehicle.notes %></td>
-        </pre>
-      </tr>
-
-      <script>
-        const preElement = document.getElementById('note-display');
-        // This takes the text, trims the edges, and puts it back
-        preElement.textContent = preElement.textContent.trim();
-      </script>
-
+    <div class="splitcontentright">
+      <h4><%= t(:field_notes) %>:</h4>
+      <div class="vehicle_notes attribute">
+        <div class="label"><%= t(:field_notes) %>:</div>
+        <% if vehicle.notes.present? %>
+          <div class="value wiki">
+            <%= textilizable(vehicle, :notes) %>
+          </div>
+        <% end %>
+      </div>
     </div>
   </div>
 </div>
 
-<% 
-  mileage = vehicle.last_service_mileage
-%>
+<% mileage = vehicle.last_service_mileage %>
 
 <% if mileage %>
   <p>
@@ -61,6 +54,6 @@
 <% end %>
 
 <div style="float: right;">
-  <%= button_to t(:label_edit), edit_vehicle_path(vehicle), method: :get%> 
-  <%= button_to t(:label_delete), vehicle, method: :delete, data: {confirm: t(:warn_ru_sure)} %> 
-</div>
+  <%= button_to t(:label_edit), edit_vehicle_path(vehicle), method: :get %> 
+  <%= button_to t(:label_delete), vehicle, method: :delete, data: { confirm: t(:warn_ru_sure) } %> 
+</div>
\ No newline at end of file
diff --git a/app/views/vehicles/_form.html.erb b/app/views/vehicles/_form.html.erb
index ce9e7e0..1cb4e54 100644
--- a/app/views/vehicles/_form.html.erb
+++ b/app/views/vehicles/_form.html.erb
@@ -42,7 +42,14 @@
         <div class="clearfix">
           <%=t(:field_notes)%>: 
           <div class="input">
-            <%= f.text_area :notes, cols: 60, rows: 10, no_label: true %>
+             <p>
+                <%= f.text_area :notes,
+                                rows: 8,
+                                class: 'wiki-edit',
+                                style: 'width: 95%;',
+                                id: 'customer_appointment_description' %>
+                <%= wikitoolbar_for 'customer_appointment_description' %>
+              </p>
           </div>
         </div>
         
diff --git a/lib/vehicles/hooks/issues_show_hook_listener.rb b/lib/vehicles/hooks/issues_show_hook_listener.rb
index 4a80fa9..1f72331 100644
--- a/lib/vehicles/hooks/issues_show_hook_listener.rb
+++ b/lib/vehicles/hooks/issues_show_hook_listener.rb
@@ -20,8 +20,7 @@ module Vehicles
           partial: 'issues/show_issue_view_right',
             locals: {
               vehicle: context[:issue].vehicle ? link_to(context[:issue].vehicle) : nil, 
-              split_vin: context[:issue].vehicle ? context[:issue].vehicle.vin.to_s.scan(/.{1,9}/) : nil,
-              notes: context[:issue].vehicle ? context[:issue].vehicle.notes : nil
+              split_vin: context[:issue].vehicle ? context[:issue].vehicle.vin.to_s.scan(/.{1,9}/) : nil
             } 
           })
       end