@prenta/admin 0.98.1 → 1.1.0

package/src/__tests__/lib/view-site-chrome.test.ts

@@ -0,0 +1,65 @@
+import { describe, expect, it } from 'vitest'
+import { NO_PUBLIC_SITE_URL_REASON, resolveViewSiteChrome } from '../../lib/view-site-chrome.js'
+
+const base = {
+  dirty: false,
+  hasPublicSiteUrl: true,
+  hasIdentity: true,
+}
+
+describe('resolveViewSiteChrome', () => {
+  it('previews a draft and shows Share', () => {
+    expect(resolveViewSiteChrome({ ...base, status: 'DRAFT' })).toEqual({
+      viewEnabled: true,
+      primaryIntent: 'preview-draft',
+      showViewLive: false,
+      shareVisible: true,
+    })
+  })
+
+  it('treats scheduled and archived like a draft', () => {
+    for (const status of ['SCHEDULED', 'ARCHIVED'] as const) {
+      expect(resolveViewSiteChrome({ ...base, status }).primaryIntent).toBe('preview-draft')
+      expect(resolveViewSiteChrome({ ...base, status }).shareVisible).toBe(true)
+    }
+  })
+
+  it('opens live on a clean published document and hides Share', () => {
+    expect(resolveViewSiteChrome({ ...base, status: 'PUBLISHED' })).toEqual({
+      viewEnabled: true,
+      primaryIntent: 'view-live',
+      showViewLive: false,
+      shareVisible: false,
+    })
+  })
+
+  it('previews unpublished changes when published and dirty, and keeps View live + Share', () => {
+    expect(resolveViewSiteChrome({ ...base, status: 'PUBLISHED', dirty: true })).toEqual({
+      viewEnabled: true,
+      primaryIntent: 'preview-draft',
+      showViewLive: true,
+      shareVisible: true,
+    })
+  })
+
+  it('disables View site and Share when the public site URL is missing', () => {
+    const chrome = resolveViewSiteChrome({
+      ...base,
+      status: 'DRAFT',
+      hasPublicSiteUrl: false,
+    })
+    expect(chrome.viewEnabled).toBe(false)
+    expect(chrome.viewDisabledReason).toBe(NO_PUBLIC_SITE_URL_REASON)
+    expect(chrome.shareVisible).toBe(true)
+  })
+
+  it('hides View site and Share when the document is unsaved', () => {
+    const chrome = resolveViewSiteChrome({
+      ...base,
+      status: 'DRAFT',
+      hasIdentity: false,
+    })
+    expect(chrome.viewEnabled).toBe(false)
+    expect(chrome.shareVisible).toBe(false)
+  })
+})

package/src/__tests__/router/admin-routes.test.ts

@@ -71,4 +71,9 @@ describe('ADMIN_STATIC_ROUTE_PATTERNS stays in sync with AdminRoot', () => {
     const missing = [...new Set(literals)].filter((p) => !known.has(p))
     expect(missing).toEqual([])
   })
+
+  it('does not keep the retired admin-renderer preview routes', () => {
+    expect(ADMIN_STATIC_ROUTE_PATTERNS).not.toContain('/pages/:id/preview')
+    expect(ADMIN_STATIC_ROUTE_PATTERNS).not.toContain('/posts/:type/:id/preview')
+  })
 })

package/src/__tests__/views/canvas-surface-shared.test.ts

@@ -58,23 +58,4 @@ describe('canvas surface is shared, not copied', () => {
       expect(read(rel), `${rel} re-implements the scale`).not.toMatch(/fitScale/)
     }
   })
-
-  /**
-   * `SitePreviewFrame` was the third copy of `clientWidth - 48` and the one
-   * followup 2 left alone, because it scales an IFRAME rather than a rendered
-   * surface and cannot render `CanvasSurface`. It can still measure the same
-   * way, and now does: the padding arithmetic is gone even though the
-   * component is not shared.
-   *
-   * Kept apart from CANVASES on purpose — the assertion here is about the
-   * measurement, not about rendering the shared surface, and folding it in
-   * would make the other two tests claim something untrue about it.
-   */
-  it('the site preview measures from a sentinel too, though it cannot share the surface', () => {
-    const preview = read('src/views/page-editor/SitePreviewFrame.tsx')
-    expect(preview, 'SitePreviewFrame still does padding arithmetic').not.toMatch(
-      /clientWidth\s*-\s*\d/,
-    )
-    expect(preview, 'SitePreviewFrame does not measure from a sentinel').toMatch(/sentinelRef/)
-  })
 })

package/src/__tests__/views/editor-top-bar.render.test.tsx

@@ -15,7 +15,6 @@ const baseProps = {
   viewport: 'wide' as const,
   onViewport: vi.fn(),
   onBack: vi.fn(),
-  onPreview: vi.fn(),
   onOpenSettings: vi.fn(),
   onSaveDraft: vi.fn(),
   onPublish: vi.fn(),
@@ -88,25 +87,93 @@ describe('EditorTopBar publish gating', () => {
   })
 })
 
-describe('EditorTopBar view-on-site vs view-live', () => {
-  it('labels the external-link button "View draft on site" for a draft', () => {
-    render(<EditorTopBar {...baseProps} status="DRAFT" onViewOnSite={vi.fn()} />)
-    expect(screen.getByRole('button', { name: 'View draft on site' })).toBeTruthy()
-    expect(screen.queryByRole('button', { name: 'View live page' })).toBeNull()
+describe('EditorTopBar view site / share', () => {
+  it('does not render the eye or the globe', () => {
+    render(
+      <EditorTopBar {...baseProps} onViewOnSite={vi.fn()} onShare={vi.fn()} hasPublicSiteUrl />,
+    )
+    expect(screen.queryByRole('button', { name: /Preview page|Preview structure/ })).toBeNull()
   })
 
-  it('labels it "View live page" for a published document and calls the handler', () => {
+  it('labels View site on a draft and calls with preview-draft', () => {
     const onViewOnSite = vi.fn()
-    render(<EditorTopBar {...baseProps} status="PUBLISHED" onViewOnSite={onViewOnSite} />)
-    const btn = screen.getByRole('button', { name: 'View live page' })
-    fireEvent.click(btn)
-    expect(onViewOnSite).toHaveBeenCalledOnce()
-    expect(screen.queryByRole('button', { name: 'View draft on site' })).toBeNull()
+    render(
+      <EditorTopBar
+        {...baseProps}
+        status="DRAFT"
+        onViewOnSite={onViewOnSite}
+        onShare={vi.fn()}
+        hasPublicSiteUrl
+      />,
+    )
+    fireEvent.click(screen.getByRole('button', { name: 'View site' }))
+    expect(onViewOnSite).toHaveBeenCalledWith('preview-draft')
+    expect(screen.getByRole('button', { name: 'Share' })).toBeTruthy()
+  })
+
+  it('labels View live on a clean published document and hides Share', () => {
+    const onViewOnSite = vi.fn()
+    render(
+      <EditorTopBar
+        {...baseProps}
+        status="PUBLISHED"
+        onViewOnSite={onViewOnSite}
+        onShare={vi.fn()}
+        hasPublicSiteUrl
+      />,
+    )
+    fireEvent.click(screen.getByRole('button', { name: 'View live' }))
+    expect(onViewOnSite).toHaveBeenCalledWith('view-live')
+    expect(screen.queryByRole('button', { name: 'Share' })).toBeNull()
+  })
+
+  it('keeps View live as a second action when published and dirty', () => {
+    const onViewOnSite = vi.fn()
+    render(
+      <EditorTopBar
+        {...baseProps}
+        status="PUBLISHED"
+        dirty
+        onViewOnSite={onViewOnSite}
+        onShare={vi.fn()}
+        hasPublicSiteUrl
+      />,
+    )
+    fireEvent.click(screen.getByRole('button', { name: 'View site' }))
+    expect(onViewOnSite).toHaveBeenCalledWith('preview-draft')
+    // Radix DropdownMenu opens on pointerdown (not click), which happy-dom
+    // doesn't synthesize from fireEvent.click — open via keyboard instead.
+    fireEvent.keyDown(screen.getByRole('button', { name: 'More view options' }), {
+      key: 'Enter',
+    })
+    fireEvent.click(screen.getByRole('menuitem', { name: 'View live' }))
+    expect(onViewOnSite).toHaveBeenCalledWith('view-live')
+    expect(screen.getByRole('button', { name: 'Share' })).toBeTruthy()
+  })
+
+  it('disables View site and Share when the public site URL is missing', () => {
+    render(
+      <EditorTopBar
+        {...baseProps}
+        onViewOnSite={vi.fn()}
+        onShare={vi.fn()}
+        hasPublicSiteUrl={false}
+      />,
+    )
+    const view = screen.getByRole('button', { name: 'View site' }) as HTMLButtonElement
+    const share = screen.getByRole('button', { name: 'Share' }) as HTMLButtonElement
+    expect(view.disabled).toBe(true)
+    expect(share.disabled).toBe(true)
+    expect(view.getAttribute('title')).toBe(
+      'Set the public site URL in Settings → SEO to open the live design.',
+    )
   })
 
-  it('hides the button when onViewOnSite is omitted (unsaved doc)', () => {
+  it('hides View site and Share when onViewOnSite is omitted (unsaved doc)', () => {
     render(<EditorTopBar {...baseProps} status="PUBLISHED" />)
-    expect(screen.queryByRole('button', { name: /View (live|draft)/ })).toBeNull()
+    expect(screen.queryByRole('button', { name: 'View site' })).toBeNull()
+    expect(screen.queryByRole('button', { name: 'View live' })).toBeNull()
+    expect(screen.queryByRole('button', { name: 'Share' })).toBeNull()
   })
 })
 
@@ -133,19 +200,15 @@ describe('EditorTopBar history button', () => {
     render(
       <EditorTopBar
         {...baseProps}
+        status="PUBLISHED"
+        dirty
+        hasPublicSiteUrl
         onHistory={vi.fn()}
         onViewOnSite={vi.fn()}
-        onToggleSitePreview={vi.fn()}
       />,
     )
 
-    for (const name of [
-      'Page settings',
-      'Version history',
-      'Preview page',
-      'Inline site preview',
-      'View draft on site',
-    ]) {
+    for (const name of ['Page settings', 'Version history', 'More view options']) {
       const button = screen.getByRole('button', { name })
       expect(button.className, `${name} has no touch target`).toContain('prenta-touch-target')
     }

package/src/__tests__/views/page-section-editor.test.tsx

@@ -3,11 +3,20 @@ import { afterEach, describe, expect, it, vi } from 'vitest'
 import { fireEvent, render, screen, waitFor } from '@testing-library/react'
 
 const fetchPageForEditor = vi.fn()
+const savePageDraft = vi.fn(async (page: unknown) => page)
 vi.mock('../../lib/page-editor-service.js', async (importOriginal) => {
   const actual = await importOriginal<typeof import('../../lib/page-editor-service.js')>()
   return {
     ...actual,
-    fetchPageForEditor: (...args: unknown[]) => fetchPageForEditor(...args),
+    fetchPageForEditor,
+    savePageDraft,
+  }
+})
+vi.mock('../../lib/seo-service.js', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('../../lib/seo-service.js')>()
+  return {
+    ...actual,
+    updateSeoFields: vi.fn(async () => ({})),
   }
 })
 
@@ -32,6 +41,7 @@ const validConfig = {
 
 afterEach(() => {
   fetchPageForEditor.mockReset()
+  savePageDraft.mockClear()
 })
 
 describe('PageSectionEditor schema guard', () => {
@@ -98,7 +108,6 @@ function loadedPage(sections: Array<{ id: string; sectionType: string; name: str
     seoDescription: '',
     publishedAt: null,
     updatedAt: null,
-    builderOwned: false,
   }
 }
 
@@ -113,9 +122,8 @@ describe('PageSectionEditor structure mode', () => {
 
     await screen.findByText('Sections')
     expect(screen.queryByText(/Structure view/)).toBeNull()
-    // "View on site" stays an icon-only button.
-    expect(screen.getByRole('button', { name: 'View draft on site' })).toBeTruthy()
-    expect(screen.getByRole('button', { name: 'Preview page' })).toBeTruthy()
+    expect(screen.getByRole('button', { name: 'View site' })).toBeTruthy()
+    expect(screen.getByRole('button', { name: 'Share' })).toBeTruthy()
   })
 
   it('shows the structure notice and promotes "View on site" for custom section types', async () => {
@@ -131,10 +139,7 @@ describe('PageSectionEditor structure mode', () => {
 
     await screen.findByText('Sections')
     expect(screen.getByText(/Structure view/)).toBeTruthy()
-    // Promoted to a labeled button — the primary design preview.
-    expect(screen.getByRole('button', { name: /View on site/ })).toBeTruthy()
-    // Eye preview is relabeled to set expectations.
-    expect(screen.getByRole('button', { name: /Preview structure/ })).toBeTruthy()
+    expect(screen.getByRole('button', { name: 'View site' })).toBeTruthy()
   })
 
   it('stays out of structure mode when a custom renderer is registered for the type', async () => {
@@ -162,7 +167,7 @@ describe('PageSectionEditor structure mode', () => {
     // structural for this page.
     expect(screen.getByTestId('opal-canvas')).toBeTruthy()
     expect(screen.queryByText(/Structure view/)).toBeNull()
-    expect(screen.getByRole('button', { name: 'View draft on site' })).toBeTruthy()
+    expect(screen.getByRole('button', { name: 'View site' })).toBeTruthy()
   })
 
   it('exposes a version history button for saved pages', async () => {
@@ -234,3 +239,30 @@ describe('PageSectionEditor section disclosure', () => {
     expect(screen.getByRole('button', { name: 'Close Hero' })).toBeTruthy()
   })
 })
+
+describe('PageSectionEditor Share', () => {
+  const configWithSite = { ...validConfig, seo: { siteUrl: 'https://example.com' } }
+
+  it('saves a dirty draft before opening Share', async () => {
+    const page = loadedPage([{ id: 's1', sectionType: 'hero', name: 'Hero' }])
+    fetchPageForEditor.mockResolvedValue(page)
+    savePageDraft.mockResolvedValue(page)
+
+    render(
+      <PageSectionEditor
+        documentId="p1"
+        config={configWithSite}
+        onNavigate={() => {}}
+        session={{}}
+      />,
+    )
+    await screen.findByText('Sections')
+
+    fireEvent.click(screen.getByRole('button', { name: 'Edit Hero' }))
+    fireEvent.change(await screen.findByLabelText('Section name'), { target: { value: 'Intro' } })
+    fireEvent.click(screen.getByRole('button', { name: 'Share' }))
+
+    await waitFor(() => expect(savePageDraft).toHaveBeenCalledOnce())
+    expect(await screen.findByRole('dialog', { name: 'Share preview link' })).toBeTruthy()
+  })
+})

package/src/__tests__/views/page-tab.render.test.tsx

@@ -21,7 +21,6 @@ const page: EditorPage = {
   scheduledUnpublishAt: null,
   authorName: '',
   authorId: '',
-  builderOwned: false,
 }
 
 function renderTab(

package/src/__tests__/views/post-section-editor.test.tsx

@@ -4,12 +4,14 @@ import { fireEvent, render, screen, waitFor } from '@testing-library/react'
 
 const fetchPostForEditor = vi.fn()
 const fetchTemplateForEditor = vi.fn()
+const savePostDraft = vi.fn()
 vi.mock('../../lib/post-editor-service.js', async (importOriginal) => {
   const actual = await importOriginal<typeof import('../../lib/post-editor-service.js')>()
   return {
     ...actual,
     fetchPostForEditor: (...args: unknown[]) => fetchPostForEditor(...args),
     fetchTemplateForEditor: (...args: unknown[]) => fetchTemplateForEditor(...args),
+    savePostDraft: (...args: unknown[]) => savePostDraft(...args),
   }
 })
 
@@ -62,6 +64,7 @@ function loadedPost(
 afterEach(() => {
   fetchPostForEditor.mockReset()
   fetchTemplateForEditor.mockReset()
+  savePostDraft.mockReset()
 })
 
 describe('PostSectionEditor schema guard', () => {
@@ -126,6 +129,9 @@ describe('PostSectionEditor new post', () => {
     )
     await screen.findByText('Sections')
     expect(screen.queryByRole('dialog', { name: 'Post details' })).toBeNull()
+    expect(screen.getByRole('button', { name: 'View site' })).toBeTruthy()
+    expect(screen.getByRole('button', { name: 'Share' })).toBeTruthy()
+    expect(screen.queryByRole('button', { name: /Preview page/ })).toBeNull()
   })
 })
 
@@ -201,6 +207,35 @@ describe('PostSectionEditor section disclosure', () => {
   })
 })
 
+describe('PostSectionEditor Share', () => {
+  it('saves a dirty draft before opening Share', async () => {
+    const post = loadedPost([{ id: 's1', sectionType: 'quote', name: 'Quote' }])
+    fetchTemplateForEditor.mockRejectedValue(new Error('no template'))
+    fetchPostForEditor.mockResolvedValue(post)
+    savePostDraft.mockResolvedValue(post)
+
+    render(
+      <PostSectionEditor
+        postType="blog"
+        documentId="p1"
+        config={{ ...validConfig, seo: { siteUrl: 'https://example.com' } }}
+        onNavigate={() => {}}
+        session={{}}
+      />,
+    )
+    await screen.findByText('Sections')
+
+    fireEvent.click(screen.getByRole('button', { name: 'Edit Quote' }))
+    fireEvent.change(await screen.findByLabelText('Section name'), {
+      target: { value: 'Pull quote' },
+    })
+    fireEvent.click(screen.getByRole('button', { name: 'Share' }))
+
+    await waitFor(() => expect(savePostDraft).toHaveBeenCalledOnce())
+    expect(await screen.findByRole('dialog', { name: 'Share preview link' })).toBeTruthy()
+  })
+})
+
 /**
  * The post editor was the last screen still on the pre-handoff panel: a fixed
  * 288px aside with no resize, no collapse and no tabs, with SEO in a

package/src/components/BreakpointPicker.tsx

@@ -19,11 +19,12 @@ export interface BreakpointPickerProps {
 }
 
 /**
- * Preview-breakpoint selector for the editor header.
+ * Canvas-width selector for the editor header.
  *
  * A dropdown rather than a segmented row: six presets with names like "Tablet
  * Landscape" do not fit a header strip, and the grouped menu is what makes the
- * DESKTOP / TABLET / MOBILE structure legible.
+ * DESKTOP / TABLET / MOBILE structure legible. This only sizes the first-party
+ * canvas — it is not a live-site viewport.
  *
  * Radix owns the menu behaviour — outside-click, Escape, roving focus, and
  * `aria-expanded` on the trigger — rather than a hand-rolled popover.
@@ -32,6 +33,7 @@ export function BreakpointPicker({ value, onChange, disabled }: BreakpointPicker
   const [open, setOpen] = useState(false)
   const active = getBreakpoint(value)
   const ActiveIcon = breakpointIcon(active.icon)
+  const canvasWidthLabel = `Canvas width: ${active.name}, ${active.width} pixels. First-party layout only — use View site for the live design.`
 
   return (
     <DropdownMenu.Root open={open} onOpenChange={setOpen}>
@@ -39,7 +41,8 @@ export function BreakpointPicker({ value, onChange, disabled }: BreakpointPicker
         <button
           type="button"
           disabled={disabled}
-          aria-label={`Preview breakpoint: ${active.name}, ${active.width} pixels`}
+          aria-label={canvasWidthLabel}
+          title={canvasWidthLabel}
           className={`border-border flex h-[34px] items-center gap-1.5 rounded-lg border px-2.5 text-[12.5px] font-medium transition-colors disabled:opacity-50 ${
             open
               ? 'border-brand bg-accent text-accent-foreground'

package/src/components/CanvasEditTip.tsx

@@ -27,8 +27,8 @@ export function CanvasEditTip() {
         <span className="text-foreground font-medium">Editing tip</span>
         {' — '}
         Click a section on the canvas, or in the list, to open its copy and images in the section
-        list. Use <span className="text-foreground font-medium">Site preview</span> to click text
-        and images in place.
+        list. Use <span className="text-foreground font-medium">View site</span> for the live
+        design.
       </p>
       <button
         type="button"

package/src/index.ts

@@ -52,7 +52,7 @@ export type {
 export { Dashboard } from './views/Dashboard.js'
 /** @deprecated Superseded by `PagesListView` routing in `AdminRoot`; will be removed in the next major. */
 export { Pages } from './views/Pages.js'
-/** @deprecated Superseded by the visual page builder (`PageBuilder`); will be removed in the next major. */
+/** @deprecated Superseded by `PageSectionEditor`; will be removed in the next major. */
 export { PageEditor } from './views/PageEditor.js'
 export { MediaBrowser } from './views/MediaBrowser.js'
 export { Forms } from './views/Forms.js'
@@ -78,16 +78,6 @@ export type { ResetPasswordProps } from './views/ResetPassword.js'
 export { CollectionList } from './views/CollectionList.js'
 export { DocumentEdit } from './views/DocumentEdit.js'
 
-export { PageBuilder } from './views/page-builder/PageBuilder.js'
-export { useBuilderState } from './hooks/useBuilderState.js'
-export type {
-  BuilderState,
-  BuilderActions,
-  DeviceMode,
-  PanelTab,
-  PageSettings as BuilderPageSettings,
-} from './hooks/useBuilderState.js'
-
 export { Breadcrumbs } from './components/Breadcrumbs.js'
 export { CommandPalette } from './components/CommandPalette.js'
 export { ErrorBoundary } from './components/ErrorBoundary.js'

package/src/layout/Sidebar.tsx

@@ -26,9 +26,7 @@ import {
   PanelTop,
   PanelBottom,
   Layers,
-  LayoutGrid,
   Code2,
-  Library,
   ListTree,
   Menu as MenuIcon,
   PanelLeftClose,
@@ -194,37 +192,16 @@ const POSTS_CHILDREN: NavItem[] = [
   { path: '/posts/types', label: 'Post Types', icon: Layers },
 ]
 
-/**
- * Children of the Templates parent. Templates is its own top-level item
- * (page building tools) — Saved Sections are reusable content blocks and
- * Widgets are embeddable components. These are deliberately NOT children
- * of Pages: Pages is content, Templates/Sections/Widgets are structure.
- */
-const TEMPLATES_CHILDREN: NavItem[] = [
-  { path: '/saved-sections', label: 'Saved Sections', icon: Library },
-  { path: '/widgets', label: 'Widgets', icon: LayoutGrid },
-]
-
 /**
  * Nav tree for the default (config-less) install. Dashboard sits on its
  * own at the top, then a CONTENT group holding Pages (no children) above
- * Posts (with All Posts + Post Types) and Templates (with Saved Sections
- * + Widgets), followed by Media / Forms / SEO / Script Tags, then a
- * SETTINGS group (Users / API Keys / Settings). The group ordering is
- * meaningful — the renderer emits a section header the first time it
- * sees a new group, so "CONTENT" renders once, above Pages.
+ * Posts (with All Posts + Post Types), followed by Media / Forms / SEO /
+ * Script Tags, then a SETTINGS group (Users / API Keys / Settings).
  */
 const defaultNavItems: NavItem[] = [
   { path: '/', label: 'Dashboard', icon: LayoutDashboard },
   { path: '/pages', label: 'Pages', icon: File, group: 'Content' },
   { path: '/posts', label: 'Posts', icon: FileText, group: 'Content', children: POSTS_CHILDREN },
-  {
-    path: '/page-templates',
-    label: 'Templates',
-    icon: Layers,
-    group: 'Content',
-    children: TEMPLATES_CHILDREN,
-  },
   { path: '/media', label: 'Media', icon: Image, group: 'Content' },
   { path: '/forms', label: 'Forms', icon: ClipboardList, group: 'Content' },
   { path: '/seo', label: 'SEO', icon: SearchIcon, group: 'Content' },
@@ -773,10 +750,9 @@ export interface NavItem {
 /**
  * Build the sidebar nav tree from the CMS config. If no collections are
  * configured, returns the curated `defaultNavItems` tree. When
- * collections ARE configured, page-type collections inherit the
- * Pages / Saved Sections / Templates child block, and every static
- * destination (Media, Forms, SEO, Script Tags, Users, API Keys,
- * Settings) is anchored under the right Content / Settings group.
+ * collections ARE configured, page-type collections render as Content
+ * entries, and every static destination (Media, Forms, SEO, Script Tags,
+ * Users, API Keys, Settings) is anchored under the right group.
  */
 export function buildNavItems(config: any): NavItem[] {
   if (!config?.collections) return defaultNavItems
@@ -817,8 +793,7 @@ export function buildNavItems(config: any): NavItem[] {
   // Script Tags).
 
   // Pages-style collections render as plain Content entries (no nested
-  // children). Saved Sections / Widgets now live under the dedicated
-  // Templates parent below, not under Pages.
+  // children).
   pages.forEach((collection) => {
     items.push({
       label: collection.labels?.plural ?? collection.slug,
@@ -843,16 +818,6 @@ export function buildNavItems(config: any): NavItem[] {
     })
   }
 
-  // Templates — its own parent holding Saved Sections + Widgets (page
-  // building structure, distinct from Pages content).
-  items.push({
-    path: '/page-templates',
-    label: 'Templates',
-    icon: Layers,
-    group: 'Content',
-    children: TEMPLATES_CHILDREN,
-  })
-
   // Standard Content destinations (always present — they map to CMS
   // features, not collections).
   items.push(

package/src/lib/editor-preview.ts

@@ -1,6 +1,9 @@
 import type { AdminEditorConfig } from '@prenta/core'
 
-/** Resolve whether a section editor should open with Site preview active. */
+/**
+ * Ignored by the editors; returns the old boolean for compatibility only.
+ * The page and post editors no longer mount an inline site-preview iframe.
+ */
 export function resolveDefaultSitePreview(
   editor: AdminEditorConfig | undefined,
   kind: 'pages' | 'posts',

package/src/lib/open-public-site-tab.ts

@@ -0,0 +1,57 @@
+import { toast } from 'sonner'
+import { buildPublicPreviewUrl, buildPublicUrl, createPreviewToken } from './preview-link.js'
+import { NO_PUBLIC_SITE_URL_REASON, type ViewSiteIntent } from './view-site-chrome.js'
+
+export async function openPublicSiteTab(opts: {
+  intent: ViewSiteIntent
+  siteUrl: string
+  collection: string
+  documentId: string
+  slug: string
+  path?: string | null
+  urlPrefix?: string
+  data?: Record<string, unknown> | null
+  dirty: boolean
+  saveDraft: () => Promise<boolean>
+}): Promise<void> {
+  const siteUrl = opts.siteUrl.trim()
+  if (!siteUrl) throw new Error(NO_PUBLIC_SITE_URL_REASON)
+
+  const parts = {
+    siteUrl,
+    urlPrefix: opts.urlPrefix ?? '',
+    slug: opts.slug,
+    path: opts.path,
+    data: opts.data,
+  }
+
+  if (opts.intent === 'view-live') {
+    window.open(buildPublicUrl(parts), '_blank', 'noopener')
+    return
+  }
+
+  const tab = window.open('about:blank', '_blank')
+  if (!tab) {
+    toast.error('Enable pop-ups for this site to open the preview in a new tab.')
+    return
+  }
+  try {
+    if (opts.dirty) {
+      const ok = await opts.saveDraft()
+      if (!ok) {
+        tab.close()
+        return
+      }
+    }
+    const { token } = await createPreviewToken(opts.collection, opts.documentId)
+    try {
+      tab.opener = null
+    } catch {
+      /* same-origin guard */
+    }
+    tab.location.replace(buildPublicPreviewUrl({ ...parts, token }))
+  } catch (err) {
+    tab.close()
+    throw err
+  }
+}