Skip to content

fix(breadcrumbs): preserve policy name casing by building crumbs from…#110

Merged
mkaply merged 2 commits intomozilla:mainfrom
mkaply:108_breadcrumb_camelcase
Apr 13, 2026
Merged

fix(breadcrumbs): preserve policy name casing by building crumbs from…#110
mkaply merged 2 commits intomozilla:mainfrom
mkaply:108_breadcrumb_camelcase

Conversation

@mkaply
Copy link
Copy Markdown
Contributor

@mkaply mkaply commented Apr 6, 2026

… frontmatter title - Fixes #108

Description:

Fix CamelCase in breadcrumb navicaton

Motivation:

breadcrumb navigation had the wrong case

Related issues and pull requests:

This fully resolves Github issues 108

@mkaply mkaply requested a review from bsmth as a code owner April 6, 2026 18:52

{
showBreadcrumbs && (
<Breadcrumbs customizeListElements={[{ index: 0, remove: true }]} linkTextFormat="sentence">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It works as expected, thanks. There should be a "passthrough" config for linkTextFormat, IMO: https://docs.astro-breadcrumbs.kasimir.dev/configuration/properties/#linktextformat

Copy link
Copy Markdown
Collaborator

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

LGTM, thanks 👍🏻

@mkaply mkaply merged commit 3e5e3bb into mozilla:main Apr 13, 2026
1 check passed
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.

No camel case in breadcrumbs

2 participants