Skip to content

feat(oc-docs): page-based routing with navigation#51

Open
sundram-bruno wants to merge 1 commit into
opencollection-dev:mainfrom
sundram-bruno:feat/bru-3188-page-routing
Open

feat(oc-docs): page-based routing with navigation#51
sundram-bruno wants to merge 1 commit into
opencollection-dev:mainfrom
sundram-bruno:feat/bru-3188-page-routing

Conversation

@sundram-bruno

@sundram-bruno sundram-bruno commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Page-based navigation for the docs renderer (BRU-3188): replaces the single long scroll with one page per item, each at a stable, shareable slug URL.

What's included

  • Routing module (routing/): slug (path-based slugify + sibling dedupe), navModel (ordered DFS of the collection honouring folder hierarchy + seq), resolve/hooks (active entry + prev/next derived from the URL).
  • AppShell: composes Topbar + sidebar + routed page content (+ playground drawer).
  • PageRouter: renders one page by type — Overview, Request, Folder, Script, Environments.
  • PrevNext footer: hierarchy/seq-ordered pagination — Fira Code method labels, edge chevrons, ellipsis on long names, responsive padding; aligned to the 1200px content gutter.
  • Sidebar wired to the route: active item derived from the URL + expand-only auto-reveal of the active node's ancestors.
  • Routing state lives in the existing store/slices/docs (expandFolders) — no parallel store.
routing-demo1.mov

@sundram-bruno sundram-bruno changed the title feat(oc-docs): page-based routing with slug navigation feat(oc-docs): page-based routing with navigation Jun 26, 2026
@sundram-bruno sundram-bruno force-pushed the feat/bru-3188-page-routing branch 3 times, most recently from 468ba72 to c9dbd84 Compare June 26, 2026 20:13
@sundram-bruno sundram-bruno force-pushed the feat/bru-3188-page-routing branch from c9dbd84 to 5f01b63 Compare June 26, 2026 20:34
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