Changeset 74544fb in Klonkt

Timestamp:

06/18/2026 06:48:40 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

8e29f77

Parents:

fee35fa

Message:

New sites: require login to comment on by default + post nav at top

• require_login_to_comment default 0 → 1 (siteEditableFields, 001-init.sql, ensurePrimarySite). Moderation was already set to 'moderate' by default. Existing sites unchanged; applies to newly created sites.
• Post navigation (← Newer / Older →) is now ABOVE the post instead of below, with the separator line below it. Applies to all sites.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@…>

Location:

src

Files:

• db/migrations/001-init.sql (modified) (1 diff)
• routes/admin-sites.js (modified) (1 diff)
• services/ensurePrimarySite.js (modified) (1 diff)
• views/pages/post.ejs (modified) (3 diffs)

src/db/migrations/001-init.sql

@@ -62 +62 @@
     closed_circle_mode INTEGER DEFAULT 0,
     robots_index INTEGER DEFAULT 1,
-    require_login_to_comment INTEGER DEFAULT 0,
+    require_login_to_comment INTEGER DEFAULT 1,
     enable_audio_player INTEGER DEFAULT 1,
     profile_photo TEXT,

src/routes/admin-sites.js

@@ -116 +116 @@
     is_public: 1,
     robots_index: 1,
-    require_login_to_comment: 0,
+    require_login_to_comment: 1,
     enable_audio_player: 1,
     enable_prutter: 1,

src/services/ensurePrimarySite.js

@@ -41 +41 @@
       is_public, robots_index, require_login_to_comment, enable_audio_player,
       feed_view_default, comments_moderation_mode, is_primary
-    ) VALUES (?, ?, ?, '', '', ?, 'nl', 'sage', '#c2410c', NULL, 1, 1, 0, 1, 'grid', 'moderate', 1)
+    ) VALUES (?, ?, ?, '', '', ?, 'nl', 'sage', '#c2410c', NULL, 1, 1, 1, 1, 'grid', 'moderate', 1)
   `).run(siteId, slug, defaultTitle(), owner.id);
 

src/views/pages/post.ejs

@@ -6 +6 @@
 %>
 <article class="container post-page">
+
+  <%# Navigatie over ALLE posts — BOVEN de post (Newer links, Older rechts). %>
+  <% if (newerPost || olderPost) { %>
+    <nav class="post-nav post-nav--top">
+      <% if (newerPost) { %>
+        <a class="post-nav-prev" href="<%= newerPost._urlBase || '' %>/<%= newerPost.slug %>"<% if (!newerPost._urlBase) { %>
+           hx-get="/<%= newerPost.slug %>?partial=1" hx-target="#pcms-main"
+           hx-push-url="/<%= newerPost.slug %>" hx-indicator="#pcms-loading"<% } %>>
+          <span class="post-nav-label">← Newer</span>
+          <span class="post-nav-title"><%= newerPost.title %></span>
+        </a>
+      <% } else { %><span></span><% } %>
+      <% if (olderPost) { %>
+        <a class="post-nav-next" href="<%= olderPost._urlBase || '' %>/<%= olderPost.slug %>"<% if (!olderPost._urlBase) { %>
+           hx-get="/<%= olderPost.slug %>?partial=1" hx-target="#pcms-main"
+           hx-push-url="/<%= olderPost.slug %>" hx-indicator="#pcms-loading"<% } %>>
+          <span class="post-nav-label">Older →</span>
+          <span class="post-nav-title"><%= olderPost.title %></span>
+        </a>
+      <% } %>
+    </nav>
+  <% } %>
+
   <% if (post.cover_image_url) { %>
     <figure class="post-cover">
@@ -233 +256 @@
   <% } %>
 
-  <%# ── Navigatie over ALLE posts (pinned + niet-pinned in de feed-volgorde) ──
-      Newer links (← omhoog in de lijst), Older rechts (→ omlaag). Vervangt de
-      vroegere pinned-only stack-navigatie: pinned posts zitten gewoon in de
-      doorlopende reeks. .post-nav-prev = linker kolom, .post-nav-next = rechter. %>
-  <% if (newerPost || olderPost) { %>
-    <nav class="post-nav">
-      <% if (newerPost) { %>
-        <a class="post-nav-prev" href="<%= newerPost._urlBase || '' %>/<%= newerPost.slug %>"<% if (!newerPost._urlBase) { %>
-           hx-get="/<%= newerPost.slug %>?partial=1" hx-target="#pcms-main"
-           hx-push-url="/<%= newerPost.slug %>" hx-indicator="#pcms-loading"<% } %>>
-          <span class="post-nav-label">← Newer</span>
-          <span class="post-nav-title"><%= newerPost.title %></span>
-        </a>
-      <% } else { %><span></span><% } %>
-      <% if (olderPost) { %>
-        <a class="post-nav-next" href="<%= olderPost._urlBase || '' %>/<%= olderPost.slug %>"<% if (!olderPost._urlBase) { %>
-           hx-get="/<%= olderPost.slug %>?partial=1" hx-target="#pcms-main"
-           hx-push-url="/<%= olderPost.slug %>" hx-indicator="#pcms-loading"<% } %>>
-          <span class="post-nav-label">Older →</span>
-          <span class="post-nav-title"><%= olderPost.title %></span>
-        </a>
-      <% } %>
-    </nav>
-  <% } %>
 </article>
 <style>
@@ -282 +281 @@
 .post-actions { display: flex; gap: 0.5rem; margin: 2rem 0; padding: 1rem; background: var(--paper-2); border-radius: 6px; flex-wrap: wrap; }
 .post-nav { display: grid; grid-template-columns: 1fr 1fr; gap: 1rem; margin-top: 3rem; padding-top: 2rem; border-top: 1px solid var(--rule); }
+/* Boven de post: geen lijn/gap erboven, maar een scheidingslijn ERONDER (naar de content). */
+.post-nav--top { margin-top: 0; padding-top: 0; border-top: 0; margin-bottom: 2rem; padding-bottom: 2rem; border-bottom: 1px solid var(--rule); }
 .post-nav-prev, .post-nav-next { display: flex; flex-direction: column; padding: 1rem; background: var(--paper-2); border-radius: 6px; text-decoration: none; color: var(--ink); }
 .post-nav-next { text-align: right; align-items: flex-end; }