Skip to content

Storyboard review page with inline re-render and UX improvements#17

Merged
amoghbanta merged 1 commit intomainfrom
increment-5-pipeline-ux
Feb 11, 2026
Merged

Storyboard review page with inline re-render and UX improvements#17
amoghbanta merged 1 commit intomainfrom
increment-5-pipeline-ux

Conversation

@amoghbanta
Copy link
Collaborator

Summary

  • Storyboard review page: Two-panel layout with page sidebar and rendered HTML preview, inline re-render button with success/error banners
  • Accept Storyboard: Dialog with pending count badge showing how many pages still need rendering
  • Page edit UX: "Back to Storyboard" button, "Save & Re-render" combined action, success banner after re-render
  • Disabled button tooltips: All disabled buttons now explain why (no API key, no pipeline data, unsaved changes)
  • Settings sheet: Rebuild warning showing how many pages will be affected, pruning controls for text/section types

Test plan

  • Storyboard: select a rendered page → see "Re-render" button in preview header
  • Click Re-render → spinner → success banner appears
  • Accept button shows "N pending" badge when not all pages are rendered
  • Page edit: see "← Storyboard" button at top left, navigates back
  • Edit mode: "Save & Re-render" button available when changes exist
  • Click Save & Re-render → saves then re-renders, success banner appears
  • Hover disabled buttons → tooltip explains why they're disabled
  • Settings sheet: warning text before Save & Rebuild says "all N pages"
  • pnpm typecheck — clean
  • pnpm vitest run — 289 tests pass

- Storyboard two-panel layout: page sidebar + rendered HTML preview
- Re-render button in preview header with success/error banners
- Accept Storyboard dialog with pending count badge
- Settings sheet with pruning controls and rebuild warning
- Page edit: Back to Storyboard button, Save & Re-render combined action
- Disabled buttons show tooltips explaining why they're disabled
- Extract shared config constants to lib/config-constants.ts
- Guide dialogs with dismissable state (use-guide-dismissed hook)
@amoghbanta amoghbanta merged commit 240a936 into main Feb 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant