Changeset 851239c in Klonkt

Timestamp:

06/15/2026 01:37:31 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

1907a18

Parents:

e6888ca

Message:

fix: topnav home button (home icon) + post-edit save/cancel with siteUrlBase prefix

• Hub brand 'Studio Noord' in the topnav replaced by a home icon button (-> /).
• post-edit form-action + Cancel were missing the /user/<slug>/ prefix: in hub the save went to a bare URL -> redirect without prefix -> post renders headerless (bareChrome). Now with _base prefix; header stays intact after saving.

Co-Authored-By: Claude <noreply@…>

Location:

src/views

Files:

• pages/post-edit.ejs (modified) (2 diffs)
• partials/topnav.ejs (modified) (2 diffs)

src/views/pages/post-edit.ejs

@@ -9 +9 @@
 // and styling changed.
 const _hasCover = !!post.cover_image_url;
+// In hub-modus moeten save/create/cancel de /user/<slug>/-prefix dragen, anders
+// valt de request terug op de primaire site (siteUrlBase leeg) en rendert de
+// post na opslaan headerless (bareChrome). In solo is _base leeg.
+const _base = (typeof siteUrlBase !== 'undefined' && siteUrlBase) ? siteUrlBase : '';
 %>
 <div class="container post-edit-page">
   <h1><%= isNew ? 'Nieuwe post' : 'Post bewerken' %></h1>
 
-  <form method="post" action="<%= isNew ? '/posts/create' : '/posts/' + post.slug + '/save' %>" class="post-edit-form">
+  <form method="post" action="<%= _base %><%= isNew ? '/posts/create' : '/posts/' + post.slug + '/save' %>" class="post-edit-form">
 
     <%# ── IDENTITY ─────────────────────────────────────────────── %>
@@ -184 +188 @@
     <%# ── ACTIONS (sticky at bottom) ──────────────────────────── %>
     <div class="pe-actions">
-      <a href="<%= isNew ? '/' : '/' + post.slug %>" class="pe-btn">Annuleren</a>
+      <a href="<%= _base %><%= isNew ? '/' : '/' + post.slug %>" class="pe-btn">Annuleren</a>
       <div class="pe-actions-spacer"></div>
       <% if (!isNew && post.status !== 'published') { %>

src/views/partials/topnav.ejs

@@ -31 +31 @@
             HTMX, zodat de headerloze overview correct rendert). Eigen class want
             .site-title is overal display:none. %>
-        <a class="nav-hub-home" href="/" aria-label="Naar de hub-home"><%= (typeof hubTitle !== 'undefined' && hubTitle) ? hubTitle : 'Home' %></a>
+        <a class="nav-hub-home" href="/" aria-label="Home" title="Home">
+          <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M3 9l9-7 9 7v11a2 2 0 0 1-2 2H5a2 2 0 0 1-2-2z"/><polyline points="9 22 9 12 15 12 15 22"/></svg>
+        </a>
       <% } else if (site) { %>
         <a class="site-title" href="<%= _siteUrlBase %>/"
@@ -236 +238 @@
 /* Hub-home brand is WÉL zichtbaar — de thuisknop op artiest-/hub-pagina's. */
 .masthead-brand .nav-hub-home {
-  display: inline-block; vertical-align: middle;
+  display: inline-flex; align-items: center; justify-content: center;
+  width: 40px; height: 40px;
   color: var(--ink); text-decoration: none;
-  font-family: var(--font-display, serif); font-weight: 700; font-size: 1.05rem;
-  transition: color 120ms;
-}
-.masthead-brand .nav-hub-home:hover { color: var(--accent); }
+  border-radius: 8px;
+  transition: color 120ms, background 120ms;
+}
+.masthead-brand .nav-hub-home:hover { color: var(--accent); background: var(--paper-2); }
+.masthead-brand .nav-hub-home svg { width: 22px; height: 22px; }
 
 /* On mobile, hide the entire masthead except for admin pages. The brand