diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-09 04:53:56 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-09 04:53:56 +0200 |
| commit | 3ae22916cd5fd0f63ffacc7a9e1aa97d311b83e6 (patch) | |
| tree | 05fa25472040295c9a318e17a03d9496cf91d825 /app/views/events/show.html.erb | |
| parent | 332ffbeed90cf07342925b82ed3fda1762346be2 (diff) | |
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.
Diffstat (limited to 'app/views/events/show.html.erb')
| -rw-r--r-- | app/views/events/show.html.erb | 85 |
1 files changed, 42 insertions, 43 deletions
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 @@ | |||
| 1 | <% content_for :subnavigation do %> | 1 | <div id="page_editor" class="show_node"> |
| 2 | <% if @event.node %> | 2 | <h1><%= @event.node ? "Event for node #{@event.node.unique_name}" : "Event" %></h1> |
| 3 | <%= link_to 'node', node_path(@event.node) %> | ||
| 4 | <% end %> | ||
| 5 | <%= link_to 'edit', edit_event_path(@event, return_to: request.path) %> | ||
| 6 | <%= button_to 'destroy', event_path(@event), method: :delete, form: { data: { confirm: 'Delete this event?' }, class: 'button_to destructive' } %> | ||
| 7 | <% end %> | ||
| 8 | 3 | ||
| 9 | <h2>Event for node <%= @event.node.unique_name %></h2> | 4 | <div id="content"> |
| 5 | <div class="node_description">Actions</div> | ||
| 6 | <div class="node_content node_info_group"> | ||
| 7 | <div class="node_info_group_items"> | ||
| 8 | <% if @event.node %> | ||
| 9 | <div class="node_info_item"> | ||
| 10 | <%= link_to 'Node', node_path(@event.node) %> | ||
| 11 | </div> | ||
| 12 | <% end %> | ||
| 13 | <div class="node_info_item"> | ||
| 14 | <%= link_to 'Edit', edit_event_path(@event, return_to: request.path), class: 'action_button' %> | ||
| 15 | </div> | ||
| 16 | <div class="node_info_item"> | ||
| 17 | <%= button_to 'Destroy', event_path(@event), method: :delete, form: { data: { confirm: 'Delete this event?' }, class: 'button_to destructive' } %> | ||
| 18 | </div> | ||
| 19 | </div> | ||
| 20 | </div> | ||
| 10 | 21 | ||
| 11 | <p> | 22 | <div class="node_description">Start time</div> |
| 12 | <b>Start time:</b> | 23 | <div class="node_content"><%=h @event.start_time %></div> |
| 13 | <%=h @event.start_time %> | ||
| 14 | </p> | ||
| 15 | 24 | ||
| 16 | <p> | 25 | <div class="node_description">End time</div> |
| 17 | <b>End time:</b> | 26 | <div class="node_content"><%=h @event.end_time %></div> |
| 18 | <%=h @event.end_time %> | ||
| 19 | </p> | ||
| 20 | 27 | ||
| 21 | <p> | 28 | <div class="node_description">Rrule</div> |
| 22 | <b>Rrule:</b> | 29 | <div class="node_content"><%=h @event.rrule %></div> |
| 23 | <%=h @event.rrule %> | ||
| 24 | <% if (human = @event.humanize_rrule(I18n.locale)) %> | ||
| 25 | <br/><small><em>( <%= human %></em> )</small> | ||
| 26 | <% end %> | ||
| 27 | </p> | ||
| 28 | 30 | ||
| 29 | <p> | 31 | <% if (human = @event.humanize_rrule(I18n.locale)) %> |
| 30 | <b>Tags:</b> | 32 | <div class="node_description">Recurrence</div> |
| 31 | <%=h @event.tag_list %> | 33 | <div class="node_content"><em><%= human %></em></div> |
| 32 | </p> | 34 | <% end %> |
| 33 | 35 | ||
| 34 | <p> | 36 | <div class="node_description">Tags</div> |
| 35 | <b>Allday:</b> | 37 | <div class="node_content"><%=h @event.tag_list %></div> |
| 36 | <%=h @event.allday %> | ||
| 37 | </p> | ||
| 38 | 38 | ||
| 39 | <p> | 39 | <div class="node_description">Allday</div> |
| 40 | <b>Url:</b> | 40 | <div class="node_content"><%=h @event.allday %></div> |
| 41 | <%=h @event.url %> | ||
| 42 | </p> | ||
| 43 | 41 | ||
| 44 | <p> | 42 | <div class="node_description">Url</div> |
| 45 | <b>Latitude:</b> | 43 | <div class="node_content"><%=h @event.url %></div> |
| 46 | <%=h @event.latitude %> | ||
| 47 | </p> | ||
| 48 | 44 | ||
| 49 | <p> | 45 | <div class="node_description">Latitude</div> |
| 50 | <b>Longitude:</b> | 46 | <div class="node_content"><%=h @event.latitude %></div> |
| 51 | <%=h @event.longitude %> | 47 | |
| 52 | </p> | 48 | <div class="node_description">Longitude</div> |
| 49 | <div class="node_content"><%=h @event.longitude %></div> | ||
| 50 | </div> | ||
| 51 | </div> | ||