flask-menu
flask-menu copied to clipboard
Breadcrumb functionality
:heart: Thank you for your contribution!
Description
Added minor changes that enabled Breadcrumb functionality.
There was issue with active_item property - it failed to create url for endpoint with parameter which was in children's collection but not active. Therefore _expected_args are not found in the args
After this changes I can use list_path to get list of items in the active path.
{%- for breadcrumb in current_menu.list_path(current_menu.root, current_menu.active_item.path) -%}
I have added additional example with Bootstrap 5 navbar.
I did somehow succeed to run pytest which fail on three of test concerning classy
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
- [x] I'm aware of the code of conduct.
- [x] I've created logical separate commits and followed the commit message format.
- [x] I've added relevant test cases.
- [x] I've added relevant documentation.
- [x] I've marked translation strings (for relevant code).
- [x] I've followed the CSS/JS and React guidelines (for relevant code).
- [x] I've followed the web accessibility guidelines (for relevant code).
- [x] I've followed the user interface guidelines (for relevant code).
- [x] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
- [x] I've NOT included third-party code (copy/pasted source code or new dependencies).
Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:
- You license your contribution under the same terms as the current repository’s license.
- You agree that you have the right to license your contribution under the current repository’s license.