-
-
Notifications
You must be signed in to change notification settings - Fork 852
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
[PUI] Make breadcrumbs adjustable #8027
[PUI] Make breadcrumbs adjustable #8027
Conversation
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
see #7962 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #8027 +/- ##
==========================================
- Coverage 85.54% 85.50% -0.05%
==========================================
Files 1177 1177
Lines 52024 52176 +152
Branches 2118 2120 +2
==========================================
+ Hits 44503 44611 +108
- Misses 6989 7024 +35
- Partials 532 541 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@matmair it'd be great to get this one closed out soon if we can :) |
…pui-adjust-breadcrumbs
…pui-adjust-breadcrumbs
…pui-adjust-breadcrumbs
@SchrodingersGat ready for review and merge |
@matmair looks very good, just a small change requested regarding variable naming |
Thanks @matmair :) |
Makes usage of breadcrumbs more uniform and adds a user setting to optionally show the current item in the breadcrumbs too.