Skip to content

fix: extend breadcrumb#568

Open
sharanyavinod wants to merge 4 commits into
mainfrom
ewbrd
Open

fix: extend breadcrumb#568
sharanyavinod wants to merge 4 commits into
mainfrom
ewbrd

Conversation

@sharanyavinod

@sharanyavinod sharanyavinod commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Extend breadcrumb to support actions at leaf level, suffix elements, and custom font size

Preview: https://da.live/canvas#/exp-workspace/frescopa/index

@aem-code-sync

aem-code-sync Bot commented Jun 29, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@sharanyavinod sharanyavinod marked this pull request as draft June 29, 2026 09:29
@sharanyavinod sharanyavinod marked this pull request as ready for review June 29, 2026 14:03
@hannessolo

Copy link
Copy Markdown
Contributor

Where can I see the suffix? I don't understand where it's being used

@sharanyavinod

Copy link
Copy Markdown
Contributor Author

Where can I see the suffix? I don't understand where it's being used

Just added a slot at the end, so we can include things if needed - like a badge/extra separator etc. For eg. in browse there is an additional separator and button which we stitch together, but for things that are semantically and visually supposed to be part of the breadcrumb, makes sense its included here. Wdyt?

@hannessolo

Copy link
Copy Markdown
Contributor

Ok, but we don't have a use case for it yet?

I think generally we should add stuff as we need it, but since you already wrote it let's keep it.

@sharanyavinod

Copy link
Copy Markdown
Contributor Author

Ok, but we don't have a use case for it yet?

We do - gear action button, separator after breadcrumb etc are already on the browse breadcrumb. These are currently absolutely positioned to look like they belong together.

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