Skip to content

Comments

Add Breadcrumbs#25

Open
0xlukem wants to merge 5 commits intomainfrom
0xlukem/breadcrumbs
Open

Add Breadcrumbs#25
0xlukem wants to merge 5 commits intomainfrom
0xlukem/breadcrumbs

Conversation

@0xlukem
Copy link
Contributor

@0xlukem 0xlukem commented Feb 23, 2026

This PR adds breadcrumb navigation to improve site navigation
image

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds breadcrumb navigation to the documentation site by enabling the MkDocs Material theme's navigation.path feature and applying custom styling to match the site's design system.

Changes:

  • Enabled the navigation.path feature in the MkDocs configuration
  • Added template logic to conditionally include breadcrumb navigation
  • Applied custom CSS styling for breadcrumb separators and hover states using established color variables

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
mkdocs.yml Enables the navigation.path feature in the theme configuration
material-overrides/main.html Adds conditional include for breadcrumb navigation partial when feature is enabled
material-overrides/assets/stylesheets/datahaven.css Defines custom styling for breadcrumb separators and hover states using project color variables

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xlukem 0xlukem requested a review from eshaben February 23, 2026 18:34
Copy link
Collaborator

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

the hover colors are diff than what we use for the rest of links. Also, is there a way to remove Home? in the beginning of the path?

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