Skip to content
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

Navigation to deeper levels #505

Closed
fulv opened this issue Feb 23, 2018 · 3 comments
Closed

Navigation to deeper levels #505

fulv opened this issue Feb 23, 2018 · 3 comments

Comments

@fulv
Copy link
Member

@fulv fulv commented Feb 23, 2018

Question:
Is there any way to get a navigation menu structure that goes deeper than just the top level in a single call? I'm thinking here of something that would allow a client to build a nav menu similar to what webcoutourier.dropdownmenu provides. In order to build the dropdowns, you need to recurse to deeper levels, which ideally should be configurable.

Or is the only option right now to get @navigation and then make another call for each of the items returned in the first call?

@tisto
Copy link
Member

@tisto tisto commented Feb 24, 2018

Not right now. This would be possible but I am not sure it is really needed. Retrieving the full content tree for the navigation portlet or a dropdown menu is possible but it does not really scale. I implemented a PoC for AngularJS long time ago and the calls that I used were for single levels (lazy loading the next level when the users clicks on a navigation item) and retrieving the full path to the portal root (for first time accessing a deeply nested page and getting an "expanded" navigation tree).

Bottom line is I am not opposed to adding such a feature. Maybe I am just missing the use case here though...

@fulv
Copy link
Member Author

@fulv fulv commented Feb 26, 2018

@fulv
Copy link
Member Author

@fulv fulv commented Apr 10, 2018

Resolved by #523.
Also, for the record, I did implement my POC in https://github.com/collective/collective.restapi.navigationtree.

@fulv fulv closed this Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants