Skip to content

feat(tangle-cloud): render the route breadcrumb trail in the top nav#3246

Merged
tangletools merged 1 commit into
developfrom
feat/nav-breadcrumb-trail
May 25, 2026
Merged

feat(tangle-cloud): render the route breadcrumb trail in the top nav#3246
tangletools merged 1 commit into
developfrom
feat/nav-breadcrumb-trail

Conversation

@drewstone
Copy link
Copy Markdown
Contributor

Per feedback: the nav should show the "Blueprints / Trading" breadcrumb trail (what was meant by "pills").

The Header already computed breadcrumbs but never rendered them. Now the nav left slot shows the trail by default — leading "Cloud" dropped (the sidebar already names the section), and section roots (Blueprints/Operators/Instances) link back. Pages can still override via useTopNavSlot; the blueprint detail injects a matching "Blueprints / " trail next to its Create/Details actions (replacing the earlier "← Catalog" link).

Test plan

  • tsc --noEmit -p tsconfig.app.json clean; full pre-push (lint/prettier/test/build) passed
  • Visual: every page shows its trail; blueprint detail reads "Blueprints / "

The Header computed breadcrumbs but never rendered them. Show the trail
("Blueprints / Trading") in the nav left slot by default, with the leading
"Cloud" dropped and section roots (Blueprints/Operators/Instances) linking
back. Pages can still override via useTopNavSlot; the blueprint detail now
injects a matching "Blueprints / <name>" trail alongside its Create/Details
actions instead of a "← Catalog" link.
@drewstone drewstone requested a review from AtelyPham as a code owner May 25, 2026 21:42
Copy link
Copy Markdown
Contributor

@tangletools tangletools left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renders the existing computed trail; section roots link back; detail override matches. Approving.

@tangletools tangletools merged commit cd6d969 into develop May 25, 2026
@tangletools tangletools deleted the feat/nav-breadcrumb-trail branch May 25, 2026 21:42
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