-
-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(breadcrumbs): added ellipsis #1301
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1301 +/- ##
==========================================
+ Coverage 91.96% 91.97% +0.01%
==========================================
Files 405 405
Lines 61432 61470 +38
Branches 4060 4064 +4
==========================================
+ Hits 56496 56540 +44
+ Misses 4928 4922 -6
Partials 8 8
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR.
Could you fix the spelling error in line 29 and 45 in test?
propertie
-> property
We don't need template literals if interpolation and special-character handling are not needed.
Line 21 in breadcrumb.tsx
name: `BreadcrumbContext`,
should be name: "BreadcrumbContext",
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closes #987
Description
made it possible to select
BreadcrumbEllipsis(ellipsis)
.New behavior
The places where ellipsis is used will display as
•••
instead of textIs this a breaking change (Yes/No):
No