Skip to content

feat(docs): clickable breadcrumb for ancestor navigation#161

Merged
ajianaz merged 2 commits into
developfrom
feat/doc-breadcrumb-clickable
Jul 9, 2026
Merged

feat(docs): clickable breadcrumb for ancestor navigation#161
ajianaz merged 2 commits into
developfrom
feat/doc-breadcrumb-clickable

Conversation

@ajianaz

@ajianaz ajianaz commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

What

Document breadcrumbs become clickable so you can jump to any parent/sub-doc in the chain.

Before

Breadcrumbs were display-only and showed truncated UUID segments (part.slice(0, 8)) — not useful and not clickable.

After

  • Each ancestor crumb shows the real title and navigates to that doc on click.
  • The Documents root crumb still clears the selection.

How

  • Build a flat id → DocEntry lookup (docById) in loadRootDocs from the already-loaded full doc list — no extra network requests.
  • getBreadcrumb() resolves selectedDoc.path (materialized ancestor UUID chain like /uuid/uuid/) to a real DocEntry[] via docById.
  • Each crumb calls selectDoc(ancestor).

Checks

  • svelte-check — 0 errors ✅
  • vitest — 61 passed ✅
  • npm run build

Document breadcrumbs were display-only (truncated UUID segments).
Now each ancestor crumb is clickable and navigates to that parent doc.

- Build a flat id→DocEntry lookup (docById) in loadRootDocs from the
  already-loaded full doc list — no extra network requests.
- getBreadcrumb() resolves selectedDoc.path (materialized ancestor UUID
  chain) to real DocEntry[] via docById.
- Each crumb renders the ancestor title and calls selectDoc(ancestor)
  on click. 'Documents' root crumb still clears the selection.
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

🔍 Cora AI Code Review

No issues found. Code looks good!


Review powered by cora-cli · BYOK · MIT

@ajianaz
ajianaz merged commit 0d44e7b into develop Jul 9, 2026
8 checks passed
@ajianaz
ajianaz deleted the feat/doc-breadcrumb-clickable branch July 9, 2026 16:25
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