@actuate-media/cms-admin 0.66.0 → 0.68.0

package/src/views/page-builder/PageTemplates.tsx

@@ -1,7 +1,20 @@
 'use client'
 
-import { AlertTriangle, Layers, Loader2, RefreshCw } from 'lucide-react'
+import { useMemo, useState } from 'react'
+import { AlertTriangle, Layers, Pencil, RefreshCw, Trash2 } from 'lucide-react'
+import { toast } from 'sonner'
+import { cmsApi } from '../../lib/api.js'
 import { useApiData } from '../../lib/useApiData.js'
+import { ConfirmDialog } from '../../components/ui/ConfirmDialog.js'
+import { EmptyState } from '../../components/ui/EmptyState.js'
+import { FilterPill } from '../../components/ui/FilterPill.js'
+import { Input } from '../../components/ui/Input.js'
+import { Modal } from '../../components/ui/Modal.js'
+import { Button } from '../../components/ui/Button.js'
+import { RowActions, RowActionButton } from '../../components/ui/RowActions.js'
+import { SearchInput } from '../../components/ui/SearchInput.js'
+import { Select } from '../../components/ui/Select.js'
+import { Skeleton } from '../../components/ui/Skeleton.js'
 
 interface PageTemplate {
   id: string
@@ -10,6 +23,25 @@ interface PageTemplate {
   category?: string
   builtIn?: boolean
   updatedAt?: string
+  thumbnail?: string | null
+  tree?: { children?: unknown[] } | null
+}
+
+type CategoryId = 'content' | 'landing' | 'utility'
+
+const CATEGORIES: Record<CategoryId, { label: string; dot: string; text: string }> = {
+  content: { label: 'Content', dot: 'bg-[var(--acc)]', text: 'text-[var(--acc)]' },
+  landing: { label: 'Landing', dot: 'bg-[var(--suc)]', text: 'text-[var(--suc)]' },
+  utility: { label: 'Utility', dot: 'bg-[var(--warn)]', text: 'text-[var(--warn)]' },
+}
+
+function categoryOf(template: PageTemplate): CategoryId {
+  const cat = template.category
+  return cat === 'landing' || cat === 'utility' ? cat : 'content'
+}
+
+function sectionCountOf(template: PageTemplate): number {
+  return Array.isArray(template.tree?.children) ? template.tree.children.length : 0
 }
 
 export interface PageTemplatesProps {
@@ -18,94 +50,345 @@ export interface PageTemplatesProps {
 
 export function PageTemplates({ onNavigate }: PageTemplatesProps) {
   const { data, loading, error, refetch } = useApiData<PageTemplate[]>('/page-templates')
-  const templates = data ?? []
+  const templates = useMemo(() => data ?? [], [data])
 
-  if (loading) {
-    return (
-      <div className="flex h-64 items-center justify-center p-4" role="status" aria-live="polite">
-        <Loader2 className="text-primary h-6 w-6 animate-spin" />
-        <span className="sr-only">Loading page templates</span>
-      </div>
-    )
+  const [search, setSearch] = useState('')
+  const [category, setCategory] = useState<CategoryId | 'all'>('all')
+  const [editTarget, setEditTarget] = useState<PageTemplate | null>(null)
+  const [deleteTarget, setDeleteTarget] = useState<PageTemplate | null>(null)
+
+  const filtered = useMemo(() => {
+    const q = search.trim().toLowerCase()
+    return templates.filter((t) => {
+      const matchesQuery =
+        !q ||
+        (t.name ?? '').toLowerCase().includes(q) ||
+        (t.description ?? '').toLowerCase().includes(q)
+      const matchesCategory = category === 'all' || categoryOf(t) === category
+      return matchesQuery && matchesCategory
+    })
+  }, [templates, search, category])
+
+  async function confirmDelete() {
+    if (!deleteTarget) return
+    const res = await cmsApi(`/page-templates/${deleteTarget.id}`, { method: 'DELETE' })
+    if (res.error) {
+      toast.error(res.error)
+    } else {
+      toast.success('Template deleted')
+      refetch()
+    }
   }
 
   return (
     <div className="p-4 pr-8">
       {error && (
-        <div className="border-border bg-card mb-4 flex items-center gap-3 rounded-lg border p-3">
-          <AlertTriangle className="text-muted-foreground h-5 w-5 shrink-0" />
-          <span className="text-foreground flex-1 text-sm">{error}</span>
+        <div className="mb-4 flex items-center gap-3 rounded-[var(--r)] border border-[var(--bdr)] bg-[var(--card)] p-3">
+          <AlertTriangle className="h-5 w-5 shrink-0 text-[var(--muted)]" aria-hidden />
+          <span className="flex-1 text-sm text-[var(--txt)]">{error}</span>
           <button
             type="button"
             onClick={refetch}
-            className="border-border text-foreground hover:bg-accent rounded-md border px-3 py-1 text-sm"
+            className="rounded-md border border-[var(--bdr)] px-3 py-1 text-sm text-[var(--txt)] hover:bg-[var(--acc-l)]"
           >
             Retry
           </button>
         </div>
       )}
 
-      <div className="mb-4 flex items-center justify-between">
+      <div className="mb-4 flex items-center justify-between gap-4">
         <div>
           <h1 className="text-foreground mb-1">Page Templates</h1>
-          <p className="text-muted-foreground text-sm">
+          <p className="text-sm text-[var(--sub)]">
             {templates.length} saved template{templates.length === 1 ? '' : 's'}
           </p>
         </div>
         <button
           type="button"
           onClick={refetch}
-          className="border-border text-foreground hover:bg-accent inline-flex items-center gap-2 rounded-md border px-3 py-2 text-sm font-medium"
+          className="inline-flex items-center gap-2 rounded-md border border-[var(--bdr)] px-3 py-2 text-sm font-medium text-[var(--txt)] transition-colors duration-[var(--motion-fast)] hover:border-[var(--acc-m)] hover:bg-[var(--acc-l)] hover:text-[var(--acc)]"
         >
-          <RefreshCw className="h-4 w-4" />
+          <RefreshCw className="h-4 w-4" aria-hidden />
           Refresh
         </button>
       </div>
 
-      {templates.length === 0 ? (
-        <div className="border-border bg-card rounded-lg border p-8 text-center">
-          <Layers className="text-muted-foreground mx-auto mb-3 h-8 w-8" />
-          <h2 className="text-foreground mb-1 text-lg font-medium">No page templates yet</h2>
-          <p className="text-muted-foreground mb-4 text-sm">
-            Built-in templates are seeded by the CMS when the templates API is available.
-          </p>
-          <button
-            type="button"
-            onClick={() => onNavigate?.('/saved-sections')}
-            className="bg-primary text-primary-foreground rounded-md px-4 py-2 text-sm font-medium hover:opacity-90"
-          >
-            View Saved Sections
-          </button>
+      {!loading && templates.length > 0 && (
+        <div className="mb-4 flex flex-wrap items-center gap-2">
+          <div className="max-w-xs min-w-52 flex-1">
+            <SearchInput value={search} onChange={setSearch} placeholder="Search templates…" />
+          </div>
+          <div className="flex gap-1.5" role="group" aria-label="Filter templates by category">
+            <FilterPill active={category === 'all'} onClick={() => setCategory('all')}>
+              All
+            </FilterPill>
+            {(Object.keys(CATEGORIES) as CategoryId[]).map((cat) => (
+              <FilterPill key={cat} active={category === cat} onClick={() => setCategory(cat)}>
+                {CATEGORIES[cat].label}
+              </FilterPill>
+            ))}
+          </div>
+          <span className="ml-auto text-[12.5px] whitespace-nowrap text-[var(--muted)]">
+            {filtered.length} of {templates.length}
+          </span>
         </div>
-      ) : (
-        <div className="grid grid-cols-1 gap-4 md:grid-cols-2 xl:grid-cols-3">
-          {templates.map((template) => (
-            <div key={template.id} className="border-border bg-card rounded-lg border p-4">
-              <div className="mb-3 flex items-start justify-between gap-3">
-                <div>
-                  <h2 className="text-foreground text-base font-medium">
-                    {template.name ?? 'Untitled template'}
-                  </h2>
-                  <p className="text-muted-foreground mt-1 text-sm">
-                    {template.description ?? 'No description provided.'}
-                  </p>
-                </div>
-                {template.builtIn && (
-                  <span className="bg-muted text-muted-foreground rounded-full px-2 py-0.5 text-xs">
-                    Built-in
-                  </span>
-                )}
-              </div>
-              <div className="text-muted-foreground flex items-center justify-between text-sm">
-                <span>{template.category ?? 'content'}</span>
-                <span>
-                  {template.updatedAt ? new Date(template.updatedAt).toLocaleDateString() : ''}
-                </span>
-              </div>
-            </div>
+      )}
+
+      {loading ? (
+        <div
+          className="grid grid-cols-1 gap-4 md:grid-cols-2 xl:grid-cols-3"
+          role="status"
+          aria-live="polite"
+          aria-label="Loading page templates"
+        >
+          {Array.from({ length: 6 }, (_, i) => (
+            <Skeleton key={i} variant="card" />
           ))}
         </div>
+      ) : templates.length === 0 ? (
+        <EmptyState
+          icon={<Layers size={24} aria-hidden />}
+          title="No page templates yet"
+          description="Built-in templates are seeded by the CMS when the templates API is available."
+          actionLabel="View Saved Sections"
+          onAction={() => onNavigate?.('/saved-sections')}
+        />
+      ) : filtered.length === 0 ? (
+        <p className="py-12 text-center text-sm text-[var(--muted)]">
+          No templates match your search.
+        </p>
+      ) : (
+        <ul
+          className="grid grid-cols-1 gap-4 md:grid-cols-2 xl:grid-cols-3"
+          aria-label="Page templates"
+        >
+          {filtered.map((template) => (
+            <TemplateCard
+              key={template.id}
+              template={template}
+              onEdit={() => setEditTarget(template)}
+              onDelete={() => setDeleteTarget(template)}
+            />
+          ))}
+        </ul>
       )}
+
+      <TemplateEditModal
+        template={editTarget}
+        onClose={() => setEditTarget(null)}
+        onSaved={() => {
+          setEditTarget(null)
+          refetch()
+        }}
+      />
+
+      <ConfirmDialog
+        open={deleteTarget !== null}
+        onClose={() => setDeleteTarget(null)}
+        onConfirm={() => void confirmDelete()}
+        title="Delete template?"
+        description={`"${deleteTarget?.name ?? 'This template'}" will be permanently removed. Pages already created from it are not affected.`}
+        confirmLabel="Delete"
+        destructive
+      />
     </div>
   )
 }
+
+function TemplateCard({
+  template,
+  onEdit,
+  onDelete,
+}: {
+  template: PageTemplate
+  onEdit: () => void
+  onDelete: () => void
+}) {
+  const cat = CATEGORIES[categoryOf(template)]
+  const sections = sectionCountOf(template)
+  const name = template.name ?? 'Untitled template'
+  const updated = template.updatedAt ? new Date(template.updatedAt).toLocaleDateString() : null
+
+  return (
+    <li className="group relative flex flex-col overflow-hidden rounded-[var(--r)] border border-[var(--bdr)] bg-[var(--card)] shadow-[var(--shd)] transition-[border-color,box-shadow,transform] duration-[var(--motion-fast)] focus-within:border-[var(--acc-m)] focus-within:shadow-[var(--shd2)] hover:-translate-y-0.5 hover:border-[var(--acc-m)] hover:shadow-[var(--shd2)] motion-reduce:hover:translate-y-0">
+      <TemplatePreview template={template} sections={sections} />
+
+      <div className="flex flex-1 flex-col p-4 pb-0">
+        <div className="flex items-start justify-between gap-3">
+          <h2 className="text-[15px] leading-snug font-medium text-[var(--txt)]">{name}</h2>
+          {template.builtIn && (
+            <span className="shrink-0 rounded-full border border-[var(--bdr)] bg-[var(--bg)] px-2 py-0.5 text-[10.5px] font-medium whitespace-nowrap text-[var(--sub)]">
+              Built-in
+            </span>
+          )}
+        </div>
+        <p className="mt-2 min-h-10 text-[12.5px] leading-relaxed text-[var(--sub)]">
+          {template.description ?? 'No description provided.'}
+        </p>
+
+        <div className="relative mt-3 flex h-[46px] items-center justify-between border-t border-[var(--bdr)] py-2.5">
+          <span
+            className={`inline-flex items-center gap-1.5 text-[11.5px] font-medium ${cat.text}`}
+          >
+            <span aria-hidden className={`h-[7px] w-[7px] shrink-0 rounded-full ${cat.dot}`} />
+            {cat.label}
+          </span>
+          <span className="text-[11.5px] text-[var(--muted)] transition-opacity duration-[var(--motion-fast)] group-focus-within:opacity-0 group-hover:opacity-0">
+            {sections} section{sections === 1 ? '' : 's'}
+            {updated ? ` · ${updated}` : ''}
+          </span>
+          {!template.builtIn && (
+            <RowActions className="absolute top-1/2 right-0 -translate-y-1/2">
+              <RowActionButton aria-label={`Edit ${name}`} onClick={onEdit}>
+                <Pencil size={14} aria-hidden />
+              </RowActionButton>
+              <RowActionButton aria-label={`Delete ${name}`} danger onClick={onDelete}>
+                <Trash2 size={14} aria-hidden />
+              </RowActionButton>
+            </RowActions>
+          )}
+        </div>
+      </div>
+    </li>
+  )
+}
+
+/**
+ * Card preview area. Uses the stored thumbnail when one exists; otherwise
+ * renders an honest wireframe placeholder built from the template's section
+ * count (a hero band plus one bar per section) — no fake screenshots.
+ */
+function TemplatePreview({ template, sections }: { template: PageTemplate; sections: number }) {
+  if (template.thumbnail) {
+    return (
+      <div className="h-28 shrink-0 overflow-hidden border-b border-[var(--bdr)] bg-[var(--bg)]">
+        <img
+          src={template.thumbnail}
+          alt={`Preview of ${template.name ?? 'template'}`}
+          className="h-full w-full object-cover"
+        />
+      </div>
+    )
+  }
+
+  const bars = Math.max(0, Math.min(sections - 1, 3))
+  return (
+    <div
+      aria-hidden
+      className="flex h-28 shrink-0 flex-col gap-1.5 border-b border-[var(--bdr)] bg-[var(--bg)] p-3"
+    >
+      <div className="h-9 rounded-[5px] bg-[var(--acc-l)]" />
+      {Array.from({ length: bars }, (_, i) => (
+        <div
+          key={i}
+          className="h-3.5 rounded-[4px] bg-[var(--bdr)]"
+          style={{ width: `${88 - i * 14}%` }}
+        />
+      ))}
+    </div>
+  )
+}
+
+function TemplateEditModal({
+  template,
+  onClose,
+  onSaved,
+}: {
+  template: PageTemplate | null
+  onClose: () => void
+  onSaved: () => void
+}) {
+  // Keyed remount seeds the form state from the selected template each time
+  // the modal opens, without effect-based state syncing.
+  if (!template) return null
+  return (
+    <TemplateEditForm key={template.id} template={template} onClose={onClose} onSaved={onSaved} />
+  )
+}
+
+function TemplateEditForm({
+  template,
+  onClose,
+  onSaved,
+}: {
+  template: PageTemplate
+  onClose: () => void
+  onSaved: () => void
+}) {
+  const [name, setName] = useState(template.name ?? '')
+  const [description, setDescription] = useState(template.description ?? '')
+  const [category, setCategory] = useState<CategoryId>(categoryOf(template))
+  const [saving, setSaving] = useState(false)
+
+  async function handleSave() {
+    if (!name.trim()) {
+      toast.error('Template name is required')
+      return
+    }
+    setSaving(true)
+    const res = await cmsApi(`/page-templates/${template.id}`, {
+      method: 'PUT',
+      body: JSON.stringify({
+        name: name.trim(),
+        description: description.trim() || null,
+        category,
+      }),
+    })
+    setSaving(false)
+    if (res.error) {
+      toast.error(res.error)
+    } else {
+      toast.success('Template updated')
+      onSaved()
+    }
+  }
+
+  return (
+    <Modal
+      open
+      onClose={onClose}
+      title="Edit template"
+      actions={
+        <>
+          <Button variant="ghost" onClick={onClose} disabled={saving}>
+            Cancel
+          </Button>
+          <Button variant="primary" onClick={() => void handleSave()} loading={saving}>
+            Save changes
+          </Button>
+        </>
+      }
+    >
+      <div className="flex flex-col gap-4">
+        <label className="flex flex-col gap-1.5">
+          <span className="text-sm font-medium text-[var(--txt)]">Name</span>
+          <Input
+            value={name}
+            onChange={(e) => setName(e.target.value)}
+            placeholder="Template name"
+          />
+        </label>
+        <label className="flex flex-col gap-1.5">
+          <span className="text-sm font-medium text-[var(--txt)]">Description</span>
+          <textarea
+            value={description}
+            onChange={(e) => setDescription(e.target.value)}
+            rows={3}
+            placeholder="What this template is for…"
+            className="w-full resize-y rounded-[var(--r-control)] border border-[var(--bdr)] bg-[var(--input-background)] px-3 py-2 text-sm text-[var(--txt)] placeholder:text-[var(--muted)] focus-visible:ring-2 focus-visible:ring-[var(--ring)] focus-visible:outline-none"
+          />
+        </label>
+        <label className="flex flex-col gap-1.5">
+          <span className="text-sm font-medium text-[var(--txt)]">Category</span>
+          <Select value={category} onChange={(e) => setCategory(e.target.value as CategoryId)}>
+            {(Object.keys(CATEGORIES) as CategoryId[]).map((cat) => (
+              <option key={cat} value={cat}>
+                {CATEGORIES[cat].label}
+              </option>
+            ))}
+          </Select>
+        </label>
+      </div>
+    </Modal>
+  )
+}

package/src/views/page-editor/PageSectionEditor.tsx

@@ -48,6 +48,12 @@ import { resolveDefaultSitePreview } from '../../lib/editor-preview.js'
 interface PageSectionEditorProps {
   /** undefined when creating a new page (`/pages/new`) */
   documentId?: string
+  /**
+   * Parent page to preselect on a new page — set when arriving via
+   * "Add sub-page" in the Pages tree view (`/pages/new?parent=<id>`).
+   * Ignored when `documentId` is present.
+   */
+  initialParentId?: string | null
   config?: any
   session?: any
   onNavigate: (path: string) => void
@@ -73,6 +79,7 @@ function deriveCan(session: any): { canEdit: boolean; canPublish: boolean } {
 
 export function PageSectionEditor({
   documentId,
+  initialParentId,
   config,
   session,
   onNavigate,
@@ -136,7 +143,7 @@ export function PageSectionEditor({
     }
 
     if (!documentId) {
-      setPage(emptyPage())
+      setPage(emptyPage(initialParentId ?? null))
       setLoading(false)
       return
     }
@@ -160,7 +167,7 @@ export function PageSectionEditor({
     return () => {
       cancelled = true
     }
-  }, [documentId, schemaError])
+  }, [documentId, initialParentId, schemaError])
 
   // ─── Unsaved-changes guard (browser unload) ──────────────────────────
   useEffect(() => {

package/src/views/page-editor/PageSettingsModal.tsx

@@ -5,6 +5,7 @@ import { normalizeSlug, validateSlug } from '@actuate-media/cms-core/client'
 import { Modal } from '../../components/ui/Modal.js'
 import { Button } from '../../components/ui/Button.js'
 import { SeoCharCount } from '../../components/seo/SeoCharCount.js'
+import { cmsApi } from '../../lib/api.js'
 import type { EditorPage } from '../../lib/page-editor-service.js'
 
 const SEO_TITLE_MIN = 30
@@ -26,7 +27,12 @@ const INPUT =
 
 const slugify = normalizeSlug
 
-/** Edits page-level fields: title, slug, path, and SEO. */
+interface ParentOption {
+  id: string
+  title: string
+}
+
+/** Edits page-level fields: title, slug, path, parent page, and SEO. */
 export function PageSettingsModal({
   open,
   page,
@@ -37,21 +43,52 @@ export function PageSettingsModal({
   const [title, setTitle] = useState(page.title)
   const [slug, setSlug] = useState(page.slug)
   const [path, setPath] = useState(page.path)
+  const [parentPageId, setParentPageId] = useState(page.parentPageId ?? '')
   const [seoTitle, setSeoTitle] = useState(page.seoTitle)
   const [seoDescription, setSeoDescription] = useState(page.seoDescription)
   // Auto-derive slug/path from title until the user edits slug manually.
   const [slugTouched, setSlugTouched] = useState(Boolean(page.slug))
+  const [parentOptions, setParentOptions] = useState<ParentOption[] | null>(null)
 
   useEffect(() => {
     if (!open) return
     setTitle(page.title)
     setSlug(page.slug)
     setPath(page.path)
+    setParentPageId(page.parentPageId ?? '')
     setSeoTitle(page.seoTitle)
     setSeoDescription(page.seoDescription)
     setSlugTouched(Boolean(page.slug))
   }, [open, page])
 
+  // Load the parent-page choices lazily the first time the modal opens.
+  // The page itself is excluded (a page can't be its own parent); deeper
+  // cycle protection lives in the tree builder, which breaks bad chains.
+  useEffect(() => {
+    if (!open || parentOptions !== null) return
+    let cancelled = false
+    cmsApi<{ docs: Array<{ id: string; title: string | null }> }>(
+      '/collections/pages?pageSize=500&sort=updatedAt&order=desc',
+    )
+      .then((res) => {
+        if (cancelled) return
+        const docs = res.data?.docs ?? []
+        const options = docs
+          .filter((d) => d.id !== page.id)
+          .map((d) => ({ id: d.id, title: d.title ?? '(untitled)' }))
+          .sort((a, b) => a.title.localeCompare(b.title))
+        setParentOptions(options)
+      })
+      .catch(() => {
+        // Leave the picker in its loading state note; the rest of the
+        // settings form stays fully usable.
+        if (!cancelled) setParentOptions([])
+      })
+    return () => {
+      cancelled = true
+    }
+  }, [open, parentOptions, page.id])
+
   function handleTitle(next: string) {
     setTitle(next)
     if (!slugTouched) {
@@ -65,7 +102,14 @@ export function PageSettingsModal({
 
   function handleSave() {
     if (slugError) return
-    onSave({ title, slug, path: path || (slug ? `/${slug}` : '/'), seoTitle, seoDescription })
+    onSave({
+      title,
+      slug,
+      path: path || (slug ? `/${slug}` : '/'),
+      parentPageId: parentPageId || null,
+      seoTitle,
+      seoDescription,
+    })
     onClose()
   }
 
@@ -135,6 +179,35 @@ export function PageSettingsModal({
             />
           </div>
         </div>
+        <div>
+          <label className={LABEL} htmlFor="ps-parent">
+            Parent page
+          </label>
+          <select
+            id="ps-parent"
+            className={INPUT}
+            value={parentPageId}
+            disabled={!canEdit || parentOptions === null}
+            onChange={(e) => setParentPageId(e.target.value)}
+          >
+            <option value="">None — top level</option>
+            {/* Keep a preselected parent (e.g. from "Add sub-page") valid
+                while the options load or if it's missing from the list. */}
+            {parentPageId && !(parentOptions ?? []).some((p) => p.id === parentPageId) && (
+              <option value={parentPageId}>
+                {parentOptions === null ? 'Loading…' : 'Current parent'}
+              </option>
+            )}
+            {(parentOptions ?? []).map((p) => (
+              <option key={p.id} value={p.id}>
+                {p.title}
+              </option>
+            ))}
+          </select>
+          <p className="text-muted-foreground mt-1 text-xs">
+            Nests this page under a parent in the Pages tree view.
+          </p>
+        </div>
         <div>
           <div className="mb-1 flex items-center justify-between">
             <label className={`${LABEL} mb-0`} htmlFor="ps-seo-title">