From 3ae22916cd5fd0f63ffacc7a9e1aa97d311b83e6 Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Thu, 9 Jul 2026 04:53:56 +0200
Subject: Fix button height mismatch between bordered and pill button styles,
 extend table styling to assets/users/events

Fix button height mismatch, extend table styling to assets/users/events
a.action_button (bordered) and the state_changing/destructive/computation
pills (borderless) rendered at different heights, since each pill variant
set its own padding at higher specificity than the shared rule meant to
equalize them. Padding and border-radius now live only in one shared
rule; also restores the #page_editor prefix on a.action_button, needed
to outrank the #page_editor a wavy-underline rule.

table.assets_table, table.user_table, and table.events_table now share
table.node_table's border-collapse/header/hover treatment. tr
min-height is dropped -- browsers don't honor height on <tr> -- in
favor of cell padding, kept at zero for node_table (spacing already
comes from its h4/p content) and set to 8px for the plain-text tables.
---
 app/views/events/show.html.erb | 103 ++++++++++++++++++++---------------------
 1 file changed, 51 insertions(+), 52 deletions(-)

(limited to 'app/views/events/show.html.erb')

diff --git a/app/views/events/show.html.erb b/app/views/events/show.html.erb
index e206bc4c..58122280 100644
--- a/app/views/events/show.html.erb
+++ b/app/views/events/show.html.erb
@@ -1,52 +1,51 @@
-<% content_for :subnavigation do %>
-  <% if @event.node %>
-    <%= link_to 'node', node_path(@event.node) %>
-  <% end %>
-  <%= link_to 'edit', edit_event_path(@event, return_to: request.path) %>
-  <%= button_to 'destroy', event_path(@event), method: :delete, form: { data: { confirm: 'Delete this event?' }, class: 'button_to destructive' } %>
-<% end %>
-
-<h2>Event for node <%= @event.node.unique_name %></h2>
-
-<p>
-  <b>Start time:</b>
-  <%=h @event.start_time %>
-</p>
-
-<p>
-  <b>End time:</b>
-  <%=h @event.end_time %>
-</p>
-
-<p>
-  <b>Rrule:</b>
-  <%=h @event.rrule %>
-  <% if (human = @event.humanize_rrule(I18n.locale)) %>
-    <br/><small><em>( <%= human %></em> )</small>
-  <% end %>
-</p>
-
-<p>
-  <b>Tags:</b>
-  <%=h @event.tag_list %>
-</p>
-
-<p>
-  <b>Allday:</b>
-  <%=h @event.allday %>
-</p>
-
-<p>
-  <b>Url:</b>
-  <%=h @event.url %>
-</p>
-
-<p>
-  <b>Latitude:</b>
-  <%=h @event.latitude %>
-</p>
-
-<p>
-  <b>Longitude:</b>
-  <%=h @event.longitude %>
-</p>
+<div id="page_editor" class="show_node">
+  <h1><%= @event.node ? "Event for node #{@event.node.unique_name}" : "Event" %></h1>
+
+  <div id="content">
+    <div class="node_description">Actions</div>
+    <div class="node_content node_info_group">
+      <div class="node_info_group_items">
+        <% if @event.node %>
+          <div class="node_info_item">
+            <%= link_to 'Node', node_path(@event.node) %>
+          </div>
+        <% end %>
+        <div class="node_info_item">
+          <%= link_to 'Edit', edit_event_path(@event, return_to: request.path), class: 'action_button' %>
+        </div>
+        <div class="node_info_item">
+          <%= button_to 'Destroy', event_path(@event), method: :delete, form: { data: { confirm: 'Delete this event?' }, class: 'button_to destructive' } %>
+        </div>
+      </div>
+    </div>
+
+    <div class="node_description">Start time</div>
+    <div class="node_content"><%=h @event.start_time %></div>
+
+    <div class="node_description">End time</div>
+    <div class="node_content"><%=h @event.end_time %></div>
+
+    <div class="node_description">Rrule</div>
+    <div class="node_content"><%=h @event.rrule %></div>
+
+    <% if (human = @event.humanize_rrule(I18n.locale)) %>
+      <div class="node_description">Recurrence</div>
+      <div class="node_content"><em><%= human %></em></div>
+    <% end %>
+
+    <div class="node_description">Tags</div>
+    <div class="node_content"><%=h @event.tag_list %></div>
+
+    <div class="node_description">Allday</div>
+    <div class="node_content"><%=h @event.allday %></div>
+
+    <div class="node_description">Url</div>
+    <div class="node_content"><%=h @event.url %></div>
+
+    <div class="node_description">Latitude</div>
+    <div class="node_content"><%=h @event.latitude %></div>
+
+    <div class="node_description">Longitude</div>
+    <div class="node_content"><%=h @event.longitude %></div>
+  </div>
+</div>
-- 
cgit v1.3