Skip to content

fix(docs): make middle breadcrumb segments clickable (TH-3860)#659

Open
SuhaniNagpal7 wants to merge 1 commit into
devfrom
suhani/th-3860-middle-quickstart-not-clickable
Open

fix(docs): make middle breadcrumb segments clickable (TH-3860)#659
SuhaniNagpal7 wants to merge 1 commit into
devfrom
suhani/th-3860-middle-quickstart-not-clickable

Conversation

@SuhaniNagpal7
Copy link
Copy Markdown
Contributor

Summary

  • Middle breadcrumb segments for intermediate URLs with no page of their own (e.g. /docs/quickstart, /docs/agent-playground/concepts) were rendered as non-clickable spans, which made the breadcrumb look broken next to the linked siblings.
  • Fix: when no page exists at the intermediate URL but child pages exist in the nav tree, fall back to the first child href so the segment stays clickable.
  • Adds getFirstChildHrefForPath to src/lib/navigation.ts and wires it into the breadcrumb in src/layouts/DocsLayout.astro.

Linear: https://linear.app/future-agi/issue/TH-3860

Test plan

  • /docs/quickstart/setup-mcp-server — middle "Quickstart" links to /docs/quickstart/prompts (first child) and navigates correctly.
  • /docs/agent-playground/concepts/understanding-agent-playground — middle "Concepts" links to the first concept page.
  • /docs/self-hosting/docker-compose — regression check: middle "Self Hosting" still links to its own /docs/self-hosting index page (real index.mdx), not the fallback.
  • Last crumb still renders as plain page-title text.

Breadcrumb segments for intermediate URL paths without their own page
(e.g. /docs/quickstart, /docs/agent-playground/concepts) were rendered as
non-clickable spans, which made the breadcrumb look broken next to the
linked siblings. When no page exists at the intermediate URL but child
pages exist in the nav tree, fall back to the first child href so the
segment stays clickable.

Linear: https://linear.app/future-agi/issue/TH-3860
@SuhaniNagpal7 SuhaniNagpal7 requested a review from NVJKKartik May 20, 2026 06:55
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