From 1384756b81c719f899359efc9125e80fae8614c0 Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Sat, 1 Aug 2026 14:46:04 +0200
Subject: Allow aggregate to order by slug and paint chapters list more nicely

---
 app/helpers/content_helper.rb               |  3 +++
 app/helpers/link_helper.rb                  |  5 ++--
 app/models/page.rb                          |  5 ++++
 app/views/custom/partials/_chapter.html.erb | 38 ++++++++++++++---------------
 4 files changed, 29 insertions(+), 22 deletions(-)

(limited to 'app')

diff --git a/app/helpers/content_helper.rb b/app/helpers/content_helper.rb
index 9d52c110..b2603481 100644
--- a/app/helpers/content_helper.rb
+++ b/app/helpers/content_helper.rb
@@ -98,6 +98,9 @@ module ContentHelper
   #   order_by="published_at"
   #   order_direction="DESC"
   # ]
+  #
+  # order_by currently takes id, published_at, created_at, updated_at, title
+  #                      and slug
 
 
   def aggregate? content
diff --git a/app/helpers/link_helper.rb b/app/helpers/link_helper.rb
index 2121133e..21e4da59 100644
--- a/app/helpers/link_helper.rb
+++ b/app/helpers/link_helper.rb
@@ -21,8 +21,9 @@ module LinkHelper
       active = (page_path == path.sub(/^\//, ""))
     end
 
-    active_class = active ? {:class => 'active'} : {:class => 'inactive'}
-    html_options = html_options.merge(active_class)
+    html_options = html_options.merge(
+      :class => [html_options[:class], active ? "active" : "inactive"].compact.join(" ")
+    )
     locale = (params[:locale] || I18n.locale).to_sym == I18n.default_locale ? nil : (params[:locale] || I18n.locale)
 
     link_to(
diff --git a/app/models/page.rb b/app/models/page.rb
index a66527da..8313b1d4 100644
--- a/app/models/page.rb
+++ b/app/models/page.rb
@@ -92,6 +92,11 @@ class Page < ApplicationRecord
         .paginate(:page => page, :per_page => options[:limit])
     end
 
+    if options[:order_by] == "slug"
+      return scope.order(Arel.sql("MIN(LOWER(nodes.slug)) #{direction}"))
+                  .paginate(:page => page, :per_page => options[:limit])
+    end
+
     column = options[:order_by].to_s.sub(/\Apages\./, "")
     column = "id" unless %w[id published_at created_at updated_at].include?(column)
 
diff --git a/app/views/custom/partials/_chapter.html.erb b/app/views/custom/partials/_chapter.html.erb
index 5d47679e..8a92b46f 100644
--- a/app/views/custom/partials/_chapter.html.erb
+++ b/app/views/custom/partials/_chapter.html.erb
@@ -1,26 +1,24 @@
 <div class="article_partial chapter_partial" lang="<%= page.effective_lang %>">
   <div class="chapter_partial_layout">
     <% if page.headline_asset %>
-      <%= link_to_path image_tag(page.headline_asset.upload.url(:thumb), :alt => ""), page.node.unique_name, class: "chapter_thumbnail" %>
+      <%= link_to_path image_tag(page.headline_asset.upload.url(:medium), :alt => ""), page.node.unique_name, class: "chapter_thumbnail" %>
     <% end %>
-    <div class="chapter_partial_content">
-      <h2 class="headline"><%= link_to_path page.title, page.node.unique_name %></h2>
-      <% if page.abstract.present? %>
-        <div class="chapter_location"><%= page.abstract %></div>
-      <% end %>
-      <% if page.node.external_url.present? %>
-        <div class="chapter_url"><%= link_to page.node.external_url, page.node.external_url, target: '_blank', rel: 'noopener' %></div>
-      <% end %>
-      <% open_days = page.node.events.tagged_with('open-day').order(:start_time) %>
-      <% if open_days.any? %>
-        <div class="chapter_open_days">
-          <span class="open_days_label"><%= t(:open_days_label) %>: </span>
-          <% open_days.each do |event| %>
-            <span class="event_schedule"><%= event_schedule_text(event) %></span>
-          <% end %>
-        </div>
-      <% end %>
-      <p class="excerpt"><%= sanitize page.body %></p>
-    </div>
+    <h2 class="headline"><%= link_to_path page.title, page.node.unique_name %></h2>
+    <% if page.abstract.present? %>
+      <div class="chapter_location"><%= page.abstract %></div>
+    <% end %>
+    <% if page.node.external_url.present? %>
+      <div class="chapter_url"><%= link_to page.node.external_url, page.node.external_url, target: '_blank', rel: 'noopener' %></div>
+    <% end %>
+    <% open_days = page.node.events.tagged_with('open-day').order(:start_time) %>
+    <% if open_days.any? %>
+      <div class="chapter_open_days">
+        <span class="open_days_label"><%= t(:open_days_label) %>: </span>
+        <% open_days.each do |event| %>
+          <span class="event_schedule"><%= event_schedule_text(event) %></span>
+        <% end %>
+      </div>
+    <% end %>
+    <p class="excerpt"><%= sanitize page.body %></p>
   </div>
 </div>
-- 
cgit v1.3