-
-
Notifications
You must be signed in to change notification settings - Fork 101
Improve breadcrumbs component #1425
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
Comments
@arkid15r can i work on this? |
@arkid15r i would like to work on it, can you assign it to me please |
@arkid15r could you please assign it to me? I'd like to work on the issue! |
Thanks for taking this over! |
@codic-yeeshu could you provide an update on this? thank you! |
hey @arkid15r ! I've made the changes but am stuck due to failing test cases. I will raise the PR as soon as it's resolved. |
@codic-yeeshu it's been 10 days since the last update -- any progress? |
Is your feature request related to a problem? Please describe.
We have pretty simple and straight-forward breadcrumbs component. However it lacks some flexibility due to its simplicity.
Describe the solution you'd like
Improve the breadcrumbs component to display each project/chapter/user/etc page H1 header content, e.g. for OWASP Nest it should be
Home > Projects > OWASP Nest
instead ofHome > Project > Nest
. Another example -- https://nest.owasp.org/members/arkid15r should showHome > Members > Arkadii Yakovets
instead ofHome > Members > Arkid15r
.Implement changes for all pages and add a couple of simple tests.
Are you going to work on implementing this?
Additional context
The text was updated successfully, but these errors were encountered: