Merge the Layout and Design tabs into one form with a single Save

The look and the layout of a blog were split over two tabs that cross-linked
each other, and the Layout tab saved every toggle, move and add straight
away through a dozen tiny forms. Now one Design tab holds it all, organised
by part of the blog (colours & fonts, header, menu, content, side columns,
footer, logo & icon) behind a sticky side index, and one Save stores the
theme, every module and the menu atomically (BlogStore.SaveDesign). Module
settings are edited inline; Discard changes throws the edits away; a
validation error shows the form again exactly as it was sent.

The image pickers no longer render the whole library four times over: a
select of the newest images, and a script-driven panel that fetches the
library page by page (GET /files as JSON) with lazy thumbnails.

New options: posts per page, site title size, footer alignment.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Sd8UPWrvyYCLj97JexNw3A

1 files changed, 11 insertions, 13 deletions

diff --git a/internal/web/handlers_blog.go b/internal/web/handlers_blog.go
index 6538c34..53ab891 100644
--- a/internal/web/handlers_blog.go
+++ b/internal/web/handlers_blog.go
@@ -13,8 +13,6 @@ import (
 	"github.com/gramanas/blogspace/internal/store"
 )
 
-const postsPerPage = 10
-
 // blogView gathers what every public page needs: theme css, the layout with
 // its modules and their data, the menu, announcements, the blog.
 func (s *Server) blogView(r *http.Request) (map[string]any, error) {
@@ -102,13 +100,13 @@ func (s *Server) renderPage(w http.ResponseWriter, r *http.Request, page *store.
 		s.serverError(w, err)
 		return
 	}
-	n := pageNum(r)
-	posts, total, err := blogStore(r).PublishedPosts(r.Context(), page.ID, postsPerPage, (n-1)*postsPerPage)
+	n, per := pageNum(r), v["theme"].(Theme).PostsPerPage
+	posts, total, err := blogStore(r).PublishedPosts(r.Context(), page.ID, per, (n-1)*per)
 	if err != nil {
 		s.serverError(w, err)
 		return
 	}
-	last := (total + postsPerPage - 1) / postsPerPage
+	last := (total + per - 1) / per
 	v["page"], v["posts"], v["pageNum"], v["lastPage"] = page, posts, n, last
 	// The pager appends ?p=N to base: no trailing slash, which /{page} would not match.
 	v["base"] = "/" + page.Slug
@@ -126,22 +124,22 @@ func (s *Server) handleBlogTag(w http.ResponseWriter, r *http.Request) {
 		s.blogNotFound(w, r)
 		return
 	}
-	n := pageNum(r)
-	posts, total, err := blogStore(r).PublishedPostsByTag(r.Context(), tag.ID, postsPerPage, (n-1)*postsPerPage)
+	v, err := s.blogView(r)
 	if err != nil {
 		s.serverError(w, err)
 		return
 	}
-	if total == 0 {
-		s.blogNotFound(w, r)
-		return
-	}
-	v, err := s.blogView(r)
+	n, per := pageNum(r), v["theme"].(Theme).PostsPerPage
+	posts, total, err := blogStore(r).PublishedPostsByTag(r.Context(), tag.ID, per, (n-1)*per)
 	if err != nil {
 		s.serverError(w, err)
 		return
 	}
-	last := (total + postsPerPage - 1) / postsPerPage
+	if total == 0 {
+		s.blogNotFound(w, r)
+		return
+	}
+	last := (total + per - 1) / per
 	v["tag"], v["posts"], v["pageNum"], v["lastPage"], v["base"] = tag, posts, n, last, "/tag/"+tag.Slug
 	s.render(w, r, "blog/tag.html", v)
 }