Skip to content

feat: Sidebar components and individual phase detail pages#727

Open
scazan wants to merge 2 commits intonav-architecturefrom
sidebar-components
Open

feat: Sidebar components and individual phase detail pages#727
scazan wants to merge 2 commits intonav-architecturefrom
sidebar-components

Conversation

@scazan
Copy link
Collaborator

@scazan scazan commented Mar 9, 2026

Summary

  • Add SidebarNavItems unified sidebar navigation component
  • Add CornerDownRight custom SVG icon for phase sub-items
  • Refactor ProcessBuilderSectionNav to use SidebarNavItems instead of Tabs
  • Add PhaseDetailPage and PhaseDetailSection for individual phase editing
  • Refactor PhasesSectionContent into a phases overview with "Configure" buttons per phase
  • Register PhaseDetailSection in the content registry

Stack

This is PR 2 of 3 in the Process Builder sidebar refactor:

  1. Navigation Architecture (nav-architecture) ← base
  2. → Sidebar Components & Phase Detail Pages (this PR)
  3. Summary, Footer & Integration (process-builder-integration)

Merge add-sheet before merging PR 3.

Test plan

  • Type check passes: pnpm w:app typecheck
  • Sidebar nav renders with correct sections and phase sub-items
  • Clicking a phase sub-item navigates to the individual phase editing page
  • Phases overview shows all phases with "Configure" buttons

@vercel
Copy link
Contributor

vercel bot commented Mar 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview Mar 9, 2026 3:20pm

Request Review

- Add SidebarNavItems unified sidebar navigation component
- Add CornerDownRight custom SVG icon for phase sub-items
- Refactor ProcessBuilderSectionNav to use SidebarNavItems instead of Tabs
- Add PhaseDetailSection to content registry
- Add PhaseDetailPage for individual phase editing
- Add PhaseDetailSection as section registry entry
- Refactor PhasesSectionContent to phases overview with Configure buttons
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.

2 participants