From 8310751d4db2854597d6379b24e346c65622972d Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Thu, 9 Jul 2026 14:19:36 +0200
Subject: Major resource view cleanup

Remove dead pages#index and occurrences resource; fix menu_items#new
menu_items#new called menu_item_params, which requires a submitted
menu_item key that can never be present on a fresh GET -- new only
ever needs a blank record to render against.

pages#index and the entire occurrences resource were unmodified Rails
scaffold generator output, unlinked from any nav and unreachable except
by direct URL. pages#index had no controller action at all (@pages was
never assigned); occurrences#index queried every row with no scoping or
pagination and its layout referenced a scaffold.css asset that no
longer exists in the pipeline. Neither is a real editorial surface --
occurrences are auto-generated from an event's RRULE and were never
meant to be browsed as a flat list. Removed rather than repaired.
pages#preview and pages#sort_images remain, now as explicit routes
rather than under a full resources :pages block.
---
 app/views/menu_items/edit.html.erb  | 51 ++++++++++++++++---------------------
 app/views/menu_items/index.html.erb |  9 +++----
 2 files changed, 25 insertions(+), 35 deletions(-)

(limited to 'app/views/menu_items')

diff --git a/app/views/menu_items/edit.html.erb b/app/views/menu_items/edit.html.erb
index 9891708..dc5e8f9 100644
--- a/app/views/menu_items/edit.html.erb
+++ b/app/views/menu_items/edit.html.erb
@@ -1,32 +1,25 @@
 <h1>Edit Menu Item</h1>
 
-<%= form_for @menu_item do |f| %>
-  <table>
-    
-    <tr>
-      <td class="description">Search</td>
-      <td>
-        <%= text_field_tag :menu_search_term %>
-        <div id="search_results">
+<div id="page_editor">
+  <%= form_for @menu_item do |f| %>
+  <div id="content">
+    <div class="node_description">Search</div>
+    <div class="node_content">
+      <%= text_field_tag :menu_search_term %>
+      <div id="search_results"></div>
+    </div>
 
-        </div>
-      </td>
-    </tr>
-    <tr>
-      <td class="description">Node Id</td>
-      <td><%= f.text_field :node_id %></td>
-    </tr>
-    <tr>
-      <td class="description">Path</td>
-      <td><%= f.text_field :path %></td>
-    </tr>
-    <tr>
-      <td class="description">Title</td>
-      <td><%= f.text_field :title %></td>
-    </tr>
-    <tr>
-      <td></td>
-      <td class="right"><%= f.submit 'Update' %></td>
-    </tr>
-  </table>
-<% end %>
+    <div class="node_description">Node Id</div>
+    <div class="node_content"><%= f.text_field :node_id %></div>
+
+    <div class="node_description">Path</div>
+    <div class="node_content"><%= f.text_field :path %></div>
+
+    <div class="node_description">Title</div>
+    <div class="node_content"><%= f.text_field :title %></div>
+
+    <div class="node_description"></div>
+    <div class="node_content"><%= f.submit 'Update' %></div>
+  <% end %>
+  </div>
+</div>
diff --git a/app/views/menu_items/index.html.erb b/app/views/menu_items/index.html.erb
index c52c150..e84199b 100644
--- a/app/views/menu_items/index.html.erb
+++ b/app/views/menu_items/index.html.erb
@@ -1,10 +1,7 @@
-<% content_for :subnavigation do %>
-  <%= link_to "new", new_menu_item_path %>
-<% end %>
-
-
 <h1>Menu Items</h1>
 
+<%= link_to 'New menu item', new_menu_item_path, class: 'action_button' %>
+
 <table id="menu_item_list">
   <% @menu_items.each do |menu_item| %>
   <tr id="menu_items-<%= menu_item.id %>">
@@ -16,7 +13,7 @@
     <td>
       <%= button_to "Delete", menu_item_path(menu_item),
             method: :delete,
-            form: { data: { confirm: "Are you sure?" } } %>
+            form: { data: { confirm: "Are you sure?" }, class: 'button_to destructive' } %>
     </td>
   </tr>
   <% end %>
-- 
cgit v1.3