summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorerdgeist <erdgeist@erdgeist.org>2026-07-05 21:52:11 +0200
committererdgeist <erdgeist@erdgeist.org>2026-07-05 21:52:11 +0200
commit9a7ce465b06462443ee82e8418f3c070de080ab2 (patch)
treebf68ecc1536af990604cd58e75417073a07f0d1a /app
parent51d491a3d67e8c9efde8019ecb8a8e1a8195ec99 (diff)
Surface node-kind conventions in the admin UI
nodes#new: each kind's radio button now shows a label plus a smaller, muted .field_hint explaining what happens automatically (parent, tags, template) - previously a single undifferentiated string mixing a short label with a long parenthetical, cluttering the smaller kinds (erfa/chaostreff) worst. Title field gets the same hint treatment, telling editors up front that the slug is auto-generated and where to adjust it later - the previous silence here meant editors only discovered slug generation after already committing to a title, with no visible way to fix it. New "Resulting path" row live-previews the full URL (origin + kind's path prefix + a debounced call to the new parameterize_preview route, reusing the exact same slug_for helper "create" now calls, so the preview can never diverge from what actually gets created) as the editor types, plus a copy-to-clipboard button. For "generic", the prefix comes from whichever parent gets chosen via the search widget, not the registry - parent_search.link_closure now stashes the chosen node's unique_name as a data attribute for exactly this. New read-only admin/conventions view dumps the NODE_KINDS registry as a reference table for whoever's confused later about how a given kind behaves - deliberately just renders the same label/hint text used on nodes#new rather than re-describing parent/tags/template separately, since decomposing the parent Proc would mean either invoking it (Update's has a real side effect - creates the year folder) or re-stating its logic a second time in a different shape, either of which risks drifting from the actual behavior. nodes#edit's Template field gets a one-line hint noting the value may already be pre-filled based on how the node was created - the one place the previous commit's new inheritance mechanism actually surfaces on this view.
Diffstat (limited to 'app')
-rw-r--r--app/helpers/application_helper.rb4
-rw-r--r--app/views/nodes/edit.html.erb5
-rw-r--r--app/views/nodes/new.html.erb34
3 files changed, 27 insertions, 16 deletions
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 0be66e9..87aa82c 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -11,4 +11,8 @@ module ApplicationHelper
11 end 11 end
12 end 12 end
13 end 13 end
14
15 def resolve_kind_text(value)
16 value.respond_to?(:call) ? value.call : value
17 end
14end 18end
diff --git a/app/views/nodes/edit.html.erb b/app/views/nodes/edit.html.erb
index 31f585c..2d0edc1 100644
--- a/app/views/nodes/edit.html.erb
+++ b/app/views/nodes/edit.html.erb
@@ -44,7 +44,10 @@
44 <div class="node_content"><%= d.datetime_select :published_at, :value => @draft.published_at %></div> 44 <div class="node_content"><%= d.datetime_select :published_at, :value => @draft.published_at %></div>
45 45
46 <div class="node_description">Template</div> 46 <div class="node_description">Template</div>
47 <div class="node_content"><%= d.select :template_name, custom_page_templates, {:prompt => 'Select Template'} %></div> 47 <div class="node_content">
48 <%= d.select :template_name, custom_page_templates, {:prompt => 'Select Template'} %>
49 <span class="field_hint">Set automatically based on how this node was created - change it if needed.</span>
50 </div>
48 51
49 <div class="node_description">Author</div> 52 <div class="node_description">Author</div>
50 <div class="node_content"><%= d.select :user_id, user_list %></div> 53 <div class="node_content"><%= d.select :user_id, user_list %></div>
diff --git a/app/views/nodes/new.html.erb b/app/views/nodes/new.html.erb
index 028d727..f557454 100644
--- a/app/views/nodes/new.html.erb
+++ b/app/views/nodes/new.html.erb
@@ -13,27 +13,23 @@
13 <tr> 13 <tr>
14 <td class="description">Type</td> 14 <td class="description">Type</td>
15 <td> 15 <td>
16 <div> 16 <% CccConventions::NODE_KINDS.each do |kind, config| %>
17 <%= radio_button_tag :kind, "top_level" %>
18 Top Level
19 </div>
20 <p>
21 <%= radio_button_tag :kind, "generic", :selected => true %>
22 Generic ( can be created anywhere )
23 </p>
24 <p>
25 <%= radio_button_tag :kind, "update" %>
26 Update ( is automatically created in /updates/<%= Time.now.year.to_s %>/ and gets tag "update" )
27 </p>
28 <p> 17 <p>
29 <%= radio_button_tag :kind, "press_release" %> 18 <%= radio_button_tag :kind, kind, kind == "generic",
30 Pressemitteilung ( is automatically created in /updates/<%= Time.now.year.to_s %>/ and gets tags "update, pressemitteilung" ) 19 data: { path_prefix: resolve_kind_text(config[:path_prefix]) } %>
20 <%= resolve_kind_text(config[:label]) %>
21 <% if config[:hint] %>
22 <span class="field_hint"><%= resolve_kind_text(config[:hint]) %></span>
23 <% end %>
31 </p> 24 </p>
25 <% end %>
32 </td> 26 </td>
33 </tr> 27 </tr>
34 <tr> 28 <tr>
35 <td class="description">Title</td> 29 <td class="description">Title</td>
36 <td><%= text_field_tag :title %></td> 30 <td><%= text_field_tag :title %>
31 <span class="field_hint">A URL slug will be generated from this automatically. You can review or adjust it afterward under the "metadata" section's Slug field.</span>
32 </td>
37 </tr> 33 </tr>
38 <tr id="parent_search_field"> 34 <tr id="parent_search_field">
39 <td class="description">Parent</td> 35 <td class="description">Parent</td>
@@ -46,6 +42,14 @@
46 </td> 42 </td>
47 </tr> 43 </tr>
48 <tr> 44 <tr>
45 <td class="description">Resulting path</td>
46 <td>
47 <span id="resulting_path">—</span>
48 <button type="button" id="copy_resulting_path" class="unselected">copy</button>
49 <span class="field_hint">This preview updates as you type. The final path can still be changed afterward by editing the title (for the last segment) or moving the node to a different parent (for everything before it).</span>
50 </td>
51 </tr>
52 <tr>
49 <td></td> 53 <td></td>
50 <td class="right"><%= submit_tag "Create" %></td> 54 <td class="right"><%= submit_tag "Create" %></td>
51 </tr> 55 </tr>