fix: Breadcrumb itemRender
not remove link even path is provided
#42049
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
fix #41994
📝 Changelog
itemRender
withpath
render additional a element.itemRender
并且配置path
时,渲染元素会额外包一层 a 元素的问题。☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 52f7978
The pull request refactors the
Breadcrumb
andBreadcrumbItem
components and their types, deprecates some props and introduces new ones, and extracts the rendering logic to a separate hook and function. The pull request also adds a test case and updates the documentation for theitemRender
prop. These changes aim to simplify the code, improve type safety, and allow more flexibility and customization of the breadcrumb component.🔍 Walkthrough
🤖 Generated by Copilot at 52f7978
Breadcrumb
andBreadcrumbItem
components (link, link, link, link, link, link)cloneElement
andConfigContext
in theBreadcrumb.tsx
file to follow the alphabetical order (link)items
prop in theBreadcrumb
component (link, link)