Skip to content

Conversation

@AndyButland
Copy link
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

#20455

Description

Currently the initial item in the content and media breadcrumb trial isn't correct and doesn't take you to the content or media root when you click on it.

image

With this update the link is correct such that it does.

Testing

Click on "Content" or "Media" in the breadcrumb trial shown when on a document or media item and check you end up at the expected root page.

Copilot AI review requested due to automatic review settings October 10, 2025 12:19
Copy link
Contributor

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

Fixes the breadcrumb trail links in the back office so that clicking on the root "Content" or "Media" item in the breadcrumb correctly navigates to the workspace root instead of generating invalid URLs.

  • Modified the #getHref method in both breadcrumb components to handle root items without unique identifiers
  • Added conditional logic to return section-only URLs for root items and full workspace URLs for specific items

Reviewed Changes

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

File Description
workspace-variant-menu-breadcrumb.element.ts Updated href generation to handle root items by checking for unique identifier before building workspace path
workspace-menu-breadcrumb.element.ts Applied same fix to handle root items in the standard breadcrumb component

@AndyButland
Copy link
Contributor Author

Thanks @madsrasmussen - I'll pick up merging, labelling and cherry-picking.

@AndyButland AndyButland merged commit e71f36d into main Oct 15, 2025
28 checks passed
@AndyButland AndyButland deleted the v16/bugfix/link-to-workspace-root branch October 15, 2025 08:06
AndyButland added a commit that referenced this pull request Oct 15, 2025
…: #20455) (#20459)

Fixes link to workspace root from breadcrumb trail.
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.

3 participants