Skip to content

Conversation

@irees
Copy link
Contributor

@irees irees commented Dec 19, 2025

No description provided.

@irees irees marked this pull request as ready for review December 19, 2025 12:01
Copilot AI review requested due to automatic review settings December 19, 2025 12:01
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

This PR restores a breadcrumbs navigation component to the codebase. The component provides dynamic breadcrumb navigation based on the current route, supporting customizable route names, tags, and parameter handling.

Key Changes

  • Adds a new Vue 3 breadcrumbs component with TypeScript support
  • Registers the component globally as TlBreadcrumbs in the type declarations
  • Implements dynamic breadcrumb generation from route paths with support for nested routes and custom labels

Reviewed changes

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

File Description
src/runtime/components/breadcrumbs.vue New breadcrumb navigation component with route-based navigation, customizable labels, and dynamic path generation
src/runtime/components.d.ts Adds type declaration for the TlBreadcrumbs global component

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

- Remove unused mergedParams Map
- Remove unused forceUpdate ref and simplify updateKey computed
- Change href='#' anchor to span for non-navigable items
- Replace deprecated substr() with slice()
- Fix CSS spacing in margin-left property
@irees irees merged commit 98a5330 into main Dec 20, 2025
5 checks passed
@irees irees deleted the breadcrumbs branch December 20, 2025 00:32
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