Skip to content

[FIX] Add tab page name to breadcrumbs when navigating to inner page#3670

Open
amit-noy wants to merge 4 commits into
mlrun:feature/refactor-navbarfrom
amit-noy:ML-12584
Open

[FIX] Add tab page name to breadcrumbs when navigating to inner page#3670
amit-noy wants to merge 4 commits into
mlrun:feature/refactor-navbarfrom
amit-noy:ML-12584

Conversation

@amit-noy
Copy link
Copy Markdown
Collaborator

@amit-noy amit-noy commented May 20, 2026

📝 Description

When navigating to an inner page (job, function, artifact, alert, etc.), the breadcrumbs now display the item name as the last step.

Refactored Breadcrumbs to accept an itemName prop and render it as a final typed crumb. Updated all relevant page components to pass the appropriate route param.


🛠️ Changes Made

  • Breadcrumbs now shows the inner page item name as the last crumb when navigating to a specific item
  • Refactored breadcrumb internals to use typed path objects instead of raw path strings
  • Updated all page components to pass the relevant route param to <Breadcrumbs itemName={...} />
  • Improved dropdown with pre-computed links, includes search, and an "All Projects" footer link
  • Minor styling polish: ellipsis truncation, smaller max-width, redesigned dropdown

✅ Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I added or updated unit / integration tests (if needed)
  • I checked that this change doesn’t introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

🔗 References


🚨 Potentially Breaking Changes

  • Yes
  • No

Includes DRC change

  • Yes
  • No

If yes -> requires bump NPM version


🔍 Additional Notes


📸 Screenshots / Demos

image

Copy link
Copy Markdown
Collaborator

@Taras-Hlukhovetskyi Taras-Hlukhovetskyi left a comment

Choose a reason for hiding this comment

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

will discuss in call

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